Re: [PATCH] staging: fieldbus: anybuss: force address space conversion

From: Sven Van Asbroeck
Date: Tue May 21 2019 - 11:22:46 EST


On Tue, May 21, 2019 at 11:13 AM Dan Carpenter <dan.carpenter@xxxxxxxxxx> 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, &regmap_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?