Re: [PATCH v2 2/4] devm: fix ioremap_cache() usage

From: Arnd Bergmann
Date: Sat May 30 2015 - 16:53:24 EST


On Saturday 30 May 2015, Dan Williams wrote:
> @@ -154,7 +148,7 @@ void __iomem *devm_ioremap_resource(struct device *dev, struct resource *res)
> }
>
> if (res->flags & IORESOURCE_CACHEABLE)
> - dest_ptr = devm_ioremap(dev, res->start, size);
> + dest_ptr = devm_ioremap_cache(dev, res->start, size);
> else
> dest_ptr = devm_ioremap_nocache(dev, res->start, size);

I think the existing uses of IORESOURCE_CACHEABLE are mostly bugs, so changing
the behavior here may cause more problems than it solves.

Arnd
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/