Subject: Re: [PATCH] isdn: disable HiSax NetJet driver on microblaze arch
From: Jean Delvare <jdelvare@suse.de>
To: Nicolai Stange <nicstange@gmail.com>
Cc: David Miller <davem@davemloft.net>, linux-kernel@vger.kernel.org,
        netdev@vger.kernel.org, isdn@linux-pingi.de,
        Chen Gang <gang.chen.5i5j@gmail.com>,
        Ralf Baechle <ralf@linux-mips.org>
In-Reply-To: <87y4jdc58w.fsf@gmail.com>
References: <87pp4vw3ep.fsf@gmail.com>
	 <20150621.095400.890337700038231482.davem@davemloft.net>
	 <87y4jdc58w.fsf@gmail.com>
Content-Type: text/plain; charset="UTF-8"
Organization: Suse Linux
Date: Mon, 22 Jun 2015 13:27:33 +0200
Message-ID: <1434972453.4568.8.camel@chaos.site>
Mime-Version: 1.0
Content-Transfer-Encoding: 8bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1177
Lines: 34

Le Sunday 21 June 2015 à 19:59 +0200, Nicolai Stange a écrit :
> David Miller <davem@davemloft.net> writes:
> >> Note that endianess on microblaze is not determined through Kconfig,
> >> but by means of a compiler provided CPP macro, namely __MICROBLAZEEL__.
> >> However, gcc defaults to big endianess on that platform.
> > Applied, but we're long overdue for an across-the-board-available
> > endianness Kconfig option that can flat out be used in these situations.
> > The current way this is handled is at best, a mess.

I fully agree.

> Thanks!
> 
> Regarding the general Kconfig endianess symbol, I will do another patch
> based on the information of
>   git grep '_endian\.h' -- arch/
> and supply every arch with either of CONFIG_CPU_{LITTLE,BIG}_ENDIAN if
> lacking.

Note that Chen Gang (Cc'd) attempted the same almost a year ago:
http://patchwork.linux-mips.org/patch/7540/
then:
http://marc.info/?t=140958656400002&r=1&w=2

The idea was great IMHO but it did not work out, and I can't remember
why.

-- 
Jean Delvare
SUSE L3 Support

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
Please read the FAQ at  http://www.tux.org/lkml/