Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751511AbdIAI2I (ORCPT ); Fri, 1 Sep 2017 04:28:08 -0400 Received: from mail-pg0-f68.google.com ([74.125.83.68]:35379 "EHLO mail-pg0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751425AbdIAI2G (ORCPT ); Fri, 1 Sep 2017 04:28:06 -0400 X-Google-Smtp-Source: ADKCNb7wA5rO7UcKIXRq6ZyScDLGM5psX7EdFHEwEImowVQUhKtzDdb25M/MNllBhXB4yE1/kMX69A== Date: Fri, 1 Sep 2017 17:28:03 +0900 From: Stafford Horne To: Geert Uytterhoeven Cc: LKML , Jonas Bonn , Openrisc Subject: Re: [OpenRISC] [PATCH 02/13] openrisc: define CPU_BIG_ENDIAN as true Message-ID: <20170901082803.GI2609@lianli.shorne-pla.net> References: <65f12040f262906af8a599a85090f8482ef553cb.1504129273.git.shorne@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.8.3 (2017-05-23) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1591 Lines: 45 On Fri, Sep 01, 2017 at 10:06:49AM +0200, Geert Uytterhoeven wrote: > On Wed, Aug 30, 2017 at 11:58 PM, Stafford Horne 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 > > --- > > 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@linux-m68k.org > > 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