On 2/24/22 15:27, Thierry Reding wrote: > From: Thierry Reding <treding@xxxxxxxxxx> > > sparse is picky about casts between different address spaces. A cast to > plain void * needs to be accompanied by a __force modifier and casting > back to void __iomem * needs to be explicit to avoid warnings. > > Reported-by: kernel test robot <lkp@xxxxxxxxx> > Fixes: 88724b78a84c ("soc/tegra: fuse: Use resource-managed helpers") > Signed-off-by: Thierry Reding <treding@xxxxxxxxxx> > --- > drivers/soc/tegra/fuse/fuse-tegra.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/soc/tegra/fuse/fuse-tegra.c b/drivers/soc/tegra/fuse/fuse-tegra.c > index 10d2ae99babd..aa94fda282f4 100644 > --- a/drivers/soc/tegra/fuse/fuse-tegra.c > +++ b/drivers/soc/tegra/fuse/fuse-tegra.c > @@ -202,8 +202,8 @@ static const struct nvmem_cell_info tegra_fuse_cells[] = { > > static void tegra_fuse_restore(void *base) > { > + fuse->base = (void __iomem *)base; > fuse->clk = NULL; > - fuse->base = base; > } > > static int tegra_fuse_probe(struct platform_device *pdev) > @@ -213,7 +213,7 @@ static int tegra_fuse_probe(struct platform_device *pdev) > struct resource *res; > int err; > > - err = devm_add_action(&pdev->dev, tegra_fuse_restore, base); > + err = devm_add_action(&pdev->dev, tegra_fuse_restore, (void __force *)base); > if (err) > return err; > Thank you, I had the same change on my mind. Reviewed-by: Dmitry Osipenko <dmitry.osipenko@xxxxxxxxxxxxx>