Re: [PATCH 2/2] include: warn for inconsistent endian config definition

From: Babu Moger
Date: Fri Jun 09 2017 - 10:40:55 EST



On 6/9/2017 9:11 AM, Geert Uytterhoeven wrote:
Hi Babu,

On Fri, Jun 9, 2017 at 3:55 PM, Babu Moger <babu.moger@xxxxxxxxxx> wrote:
On 6/9/2017 2:16 AM, Geert Uytterhoeven wrote:
On Fri, Jun 9, 2017 at 9:05 AM, Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>
wrote:
Here is the original discussion
http://www.spinics.net/lists/devicetree/msg178101.html

Signed-off-by: Babu Moger <babu.moger@xxxxxxxxxx>
Suggested-by: Arnd Bergmann <arnd@xxxxxxxx>
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@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