On Tue, Feb 19, 2019 at 8:45 AM Hugo Lefeuvre <hle@xxxxxxxxxx> wrote:
>
> Hi Stephen, Arnd,
>
> > After merging the asm-generic tree, today's linux-next build (powerpc
> > allnoconfig) failed like this:
> > ...
> > Caused by commit
> >
> > 8e074c243ed3 ("iomap: add missing const to ioread*/iowrite addr arg")
>
> I have prepared a patch addressing this issue, and am currently building it
> with a powerpc cross compiler. I'll submit it once it's done. Please tell
> me if I should update the initial patch instead.
>
> > The const qualifiers are also missing in:
> >
> > arch/parisc/lib/iomap.c
> > arch/sh/kernel/iomap.c
>
> I also have patches for these architectures, but it is more complicated for
> me to test them with a cross compiler. Should I still submit them?
I'm not sending a pull request for this if it breaks any architectures,
so I think we need to fix them all, and I suppose we also have to
change all architectures in the same patch that changes the architecture
independent declaration, so it doesn't break intermittently.
At this point, I'd probably drop your patches from my asm-generic
tree entirely to avoid the regression, and wait for you to resend
them all after 5.1-rc1, for inclusion in 5.2.
Can you elaborate on the original problem that you saw? Maybe
we can have a different workaround for it in the meantime.
Arnd
