On Tue, Jan 07, 2020 at 10:39:34AM +0100, Linus Walleij wrote: > On Wed, Dec 18, 2019 at 6:32 PM Ben Dooks <ben.dooks@xxxxxxxxxxxxxxx> wrote: > > On 18/12/2019 16:26, Jesper Nilsson wrote: > > > On Wed, Dec 18, 2019 at 11:16:02AM +0100, Ben Dooks (Codethink) wrote: > > > >> - unsigned int *reg; > > >> + unsigned int __iomem *reg; > > > > > > void __iomem *reg; > > > > > > We're using as an argument to readl()? > > > > yes, readl() shoud take an __iomem attributed pointer. > > Shall I change it to void as well when applying? Please do, sorry that my comment wasn't clear above. > Yours, > Linus Walleij /^JN - Jesper Nilsson -- Jesper Nilsson -- jesper.nilsson@xxxxxxxx