Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751596AbdFIOkq (ORCPT ); Fri, 9 Jun 2017 10:40:46 -0400 Received: from userp1040.oracle.com ([156.151.31.81]:21895 "EHLO userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751534AbdFIOko (ORCPT ); Fri, 9 Jun 2017 10:40:44 -0400 Subject: Re: [PATCH 2/2] include: warn for inconsistent endian config definition To: Geert Uytterhoeven Cc: Yoshinori Sato , Jonas Bonn , Stefan Kristiansson , Stafford Horne , "James E.J. Bottomley" , Helge Deller , "David S. Miller" , Al Viro , Michael Ellerman , Peter Zijlstra , Ingo Molnar , Max Filippov , "linux-kernel@vger.kernel.org" , uclinux-h8-devel@lists.sourceforge.jp, Linux/m68k , openrisc@lists.librecores.org, Parisc List , sparclinux References: <1496960243-196898-1-git-send-email-babu.moger@oracle.com> <1496960243-196898-3-git-send-email-babu.moger@oracle.com> <257e0d57-bf55-eb1b-26e2-ba50b1bd2676@oracle.com> From: Babu Moger Organization: Oracle Corporation Message-ID: Date: Fri, 9 Jun 2017 09:39:03 -0500 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.1.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Content-Language: en-US X-Source-IP: aserv0021.oracle.com [141.146.126.233] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1147 Lines: 30 On 6/9/2017 9:11 AM, Geert Uytterhoeven wrote: > Hi Babu, > > On Fri, Jun 9, 2017 at 3:55 PM, Babu Moger wrote: >> On 6/9/2017 2:16 AM, Geert Uytterhoeven wrote: >>> On Fri, Jun 9, 2017 at 9:05 AM, Geert Uytterhoeven >>> wrote: >>>>> Here is the original discussion >>>>> http://www.spinics.net/lists/devicetree/msg178101.html >>>>> >>>>> Signed-off-by: Babu Moger >>>>> Suggested-by: Arnd Bergmann >> Yes. Got it. So patch #1 is fine. But, patch #2 might cause regressions. >> Should I drop patch 2. > No, it should be applied, and regressions should be fixed. Geert, Ok. Sure. I will resubmit the patch mentioning all the files(base.c, fdt.c etc..) that are affected by this change. thanks > 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