Re: [OpenRISC] [PATCH 02/13] openrisc: define CPU_BIG_ENDIAN as true

[Stafford Horne]

On Fri, Sep 01, 2017 at 10:06:49AM +0200, Geert Uytterhoeven wrote:
> On Wed, Aug 30, 2017 at 11:58 PM, Stafford Horne <shorne@xxxxxxxxx> wrote:
> > While working on SMP for OpenRISC I found this is needed for qrwlocks to
> > work correctly.  OpenRISC is big endian so this should have been here
> > already.
> >
> > Signed-off-by: Stafford Horne <shorne@xxxxxxxxx>
> > ---
> >  arch/openrisc/Kconfig | 3 +++
> >  1 file changed, 3 insertions(+)
> >
> > diff --git a/arch/openrisc/Kconfig b/arch/openrisc/Kconfig
> > index ba8ae92ec5fb..8ddfc92ab813 100644
> > --- a/arch/openrisc/Kconfig
> > +++ b/arch/openrisc/Kconfig
> > @@ -29,6 +29,9 @@ config OPENRISC
> >         select CPU_NO_EFFICIENT_FFS if !OPENRISC_HAVE_INST_FF1
> >         select NO_BOOTMEM
> >
> > +config CPU_BIG_ENDIAN
> > +       def_bool y
> > +
> 
> This is already in Andrew's tree and linux-next, as part of commit
> 410ae809fee6c792 ("arch: define CPU_BIG_ENDIAN for all fixed big endian
> archs").

Thank you,  Yes I am cc'd on those patches.  I had added this patch myself
some time before 10ae809fee6c792 ("arch: define CPU_BIG_ENDIAN for all
fixed big endian archs") appeared.

Once that gets merged I will rebase and this will disappear.

-Stafford

> Gr{oetje,eeting}s,
> 
>                         Geert
> 
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx
> 
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
>                                 -- Linus Torvalds