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

From: Babu Moger
Date: Mon Jun 12 2017 - 17:32:40 EST



On 6/12/2017 3:58 PM, Max Filippov wrote:
On Mon, Jun 12, 2017 at 1:51 PM, Arnd Bergmann <arnd@xxxxxxxx> 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.
Max. Thanks. Please update us when you are done.