Thread (14 messages) 14 messages, 5 authors, 2014-08-25

Re: [PATCH v3] arch: Kconfig: Let all architectures set endian explicitly

From: Chen Gang <hidden>
Date: 2014-08-14 22:22:55
Also in: linux-alpha, linux-mips, linux-um

On 08/15/2014 02:27 AM, Lennox Wu wrote:
I don't think it's necessary, what's the benfit?

2014-08-15 2:21 GMT+08:00 Vineet Gupta [off-list ref]:
quoted
On Thursday 14 August 2014 09:55 AM, Chen Gang wrote:
[...]
quoted
quoted
diff --git a/arch/arc/Kconfig b/arch/arc/Kconfig
index 9596b0a..e939abd 100644
--- a/arch/arc/Kconfig
+++ b/arch/arc/Kconfig
@@ -35,6 +35,7 @@ config ARC
      select OF_EARLY_FLATTREE
      select PERF_USE_VMALLOC
      select HAVE_DEBUG_STACKOVERFLOW
+     select CPU_LITTLE_ENDIAN if !CPU_BIG_ENDIAN
It is not clear what exactly are you trying to fix. What doesn't work w/o
this
patch !
For many individual modules may need check CPU_LITTLE_ENDIAN or
CPU_BIG_ENDIAN, which is an architecture's attribute.

Or they have to list many architectures which they support, which they
don't support. And still, it is not precise.

For architecture API, endian is a main architecture's attribute which
may be used by outside, so every architecture need let outside know
about it, explicitly.


Thanks.
-- 
Chen Gang

Open share and attitude like air water and life which God blessed
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help