On Tue, Jan 7, 2020 at 12:18 PM Jesper Nilsson <jesper.nilsson@xxxxxxxx> wrote: > 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. OK I did that and applied, let's see if it works :) Yours, Linus Walleij