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, ®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? _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel