Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752830AbdFLU6L (ORCPT ); Mon, 12 Jun 2017 16:58:11 -0400 Received: from mail-yw0-f194.google.com ([209.85.161.194]:35943 "EHLO mail-yw0-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752258AbdFLU6I (ORCPT ); Mon, 12 Jun 2017 16:58:08 -0400 MIME-Version: 1.0 In-Reply-To: References: <201706102219.7auwXKYx%fengguang.wu@intel.com> <002f7627-1ce5-eabb-967d-fe93a3660f03@oracle.com> From: Max Filippov Date: Mon, 12 Jun 2017 13:58:07 -0700 Message-ID: Subject: Re: [PATCH 2/2] include: warn for inconsistent endian config definition To: Arnd Bergmann Cc: Babu Moger , kbuild test robot , kbuild-all@01.org, Yoshinori Sato , Geert Uytterhoeven , Jonas Bonn , Stefan Kristiansson , Stafford Horne , "James E.J. Bottomley" , Helge Deller , David Miller , Al Viro , Michael Ellerman , Peter Zijlstra , Ingo Molnar , Linux Kernel Mailing List , "moderated list:H8/300 ARCHITECTURE" , linux-m68k@vger.kernel.org, openrisc@lists.librecores.org, Parisc List , sparclinux , Michal Simek Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 876 Lines: 19 On Mon, Jun 12, 2017 at 1:51 PM, Arnd Bergmann wrote: > That way, we don't have to guess what the toolchain does, but rather > tell it to do whatever is configured, like we do for most other architectures. > > Unfortunately we can't do the same thing on xtensa, as that no longer > supports the -mbig-endian/-mbig-endian flags in any recent gcc version > (a long time ago it had them, but they were removed along with many other > options). For xtensa we probably need to generate Kconfig fragment that would go in with the variant subdirectory. That will solve this, and clean up other options that we currently have for manual selection for xtensa, but there's actually no choice, i.e. the option has to be selected correctly, there's only one correct choice and otherwise the kernel either won't build or won't work. I'll look into it. -- Thanks. -- Max