Re: [PATCH] isdn: hisax: netjet requires VIRT_TO_BUS

2013-03-17 Thread David Miller
From: Arnd Bergmann Date: Thu, 14 Mar 2013 22:56:41 +0100 > Disabling CONFIG_VIRT_TO_BUS on ARM showed that the hisax netjet > driver depends on this deprecated functionality but is not > marked so in Kconfig. > > Rather than adding ARM to the already long list of architectures > that this drive

Re: [PATCH] isdn: hisax: netjet requires VIRT_TO_BUS

2013-03-15 Thread Arnd Bergmann
On Friday 15 March 2013, David Miller wrote: > The first usage seems reason, but the temtation is going to be quite > strong to misuse to block out drivers when there is "no value" in > spending time necessary to simply make them endian clean instead. Right. I'll let someone else start that discus

Re: [PATCH] isdn: hisax: netjet requires VIRT_TO_BUS

2013-03-15 Thread David Miller
From: Arnd Bergmann Date: Fri, 15 Mar 2013 12:29:52 + > On Friday 15 March 2013, David Miller wrote: >> I do not want to see us add such a Kconfig dependency knob. >> >> Then the real tendency will exist to make new drivers little-endian >> only, refuse to fix endian-broken old drivers, etc.

Re: [PATCH] isdn: hisax: netjet requires VIRT_TO_BUS

2013-03-15 Thread Arnd Bergmann
On Friday 15 March 2013, David Miller wrote: > I do not want to see us add such a Kconfig dependency knob. > > Then the real tendency will exist to make new drivers little-endian > only, refuse to fix endian-broken old drivers, etc. > > Which means that allmodconfig on my architecture will have b

Re: [PATCH] isdn: hisax: netjet requires VIRT_TO_BUS

2013-03-15 Thread David Miller
From: Arnd Bergmann Date: Fri, 15 Mar 2013 10:15:00 + > David Howells brought up the topic of endian checks recently, noting > that the way we define __BIG_ENDIAN and __LITTLE_ENDIAN in the kernel > is incompatible to how do it in user space, and not much better either. > > A few architectur

Re: [PATCH] isdn: hisax: netjet requires VIRT_TO_BUS

2013-03-15 Thread Arnd Bergmann
On Friday 15 March 2013, Geert Uytterhoeven wrote: > On Thu, Mar 14, 2013 at 10:56 PM, Arnd Bergmann wrote: > > Disabling CONFIG_VIRT_TO_BUS on ARM showed that the hisax netjet > > driver depends on this deprecated functionality but is not > > marked so in Kconfig. > > > > Rather than adding ARM t

Re: [PATCH] isdn: hisax: netjet requires VIRT_TO_BUS

2013-03-14 Thread Geert Uytterhoeven
On Thu, Mar 14, 2013 at 10:56 PM, Arnd Bergmann wrote: > Disabling CONFIG_VIRT_TO_BUS on ARM showed that the hisax netjet > driver depends on this deprecated functionality but is not > marked so in Kconfig. > > Rather than adding ARM to the already long list of architectures > that this driver is

[PATCH] isdn: hisax: netjet requires VIRT_TO_BUS

2013-03-14 Thread Arnd Bergmann
Disabling CONFIG_VIRT_TO_BUS on ARM showed that the hisax netjet driver depends on this deprecated functionality but is not marked so in Kconfig. Rather than adding ARM to the already long list of architectures that this driver is broken on, this patch adds 'depends on VIRT_TO_BUS' and removes the