[PATCH] staging: fieldbus: anybuss: force address space conversion
gregkh at linuxfoundation.org
Tue May 21 15:42:41 UTC 2019
On Tue, May 21, 2019 at 11:19:59AM -0400, Sven Van Asbroeck wrote:
> On Tue, May 21, 2019 at 11:13 AM Dan Carpenter <dan.carpenter at oracle.com> wrote:
> > There is no need to use __force. Just:
> > void __iomem *base = (void __iomem *)context;
> > For the rest as well.
> Yes, that appears to work for 'void *' -> __iomem, but not the other way around:
> + return devm_regmap_init(dev, NULL, (void *)base, ®map_cfg);
> sparse generates:
> drivers/staging/fieldbus/anybuss/arcx-anybus.c:156:16: warning: cast
> removes address space of expression
> Would it be a ok solution to use __force in this instance only?
Ick, if you are using __force, almost always something is wrong.
More information about the devel