[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20111115101817.GA6907@avionic-0098.mockup.avionic-design.de>
Date: Tue, 15 Nov 2011 11:18:17 +0100
From: Thierry Reding <thierry.reding@...onic-design.de>
To: Paul Bolle <pebolle@...cali.nl>
Cc: Richard Purdie <rpurdie@...ux.intel.com>,
Russell King <rmk+kernel@....linux.org.uk>,
Wim Van Sebroeck <wim@...ana.be>,
linux-kernel@...r.kernel.org
Subject: Re: ARCH_PXA_ADX
* Paul Bolle wrote:
> 0) Since v2.6.32 the mainline tree supports the "Avionic Design Xanthos
> Backlight Driver" (BACKLIGHT_ADX) and the "Avionic Design Xanthos
> watchdog" driver (ADX_WATCHDOG). These were added in commit 3b96ea9ef8
> ("[...] Add support for the Avionic Design Xanthos backlight device.")
> and 38461c5c08 ("[...] Add support for the Avionic Design Xanthos
> watchdog timer.").
>
> 1) Both drivers' Kconfig symbols depend on the Kconfig symbol
> ARCH_PXA_ADX. At that time this symbol didn't exist. It still doesn't. I
> haven't been able to find anything remotely related (ie, I even did
> things like "git grep -i PXA | grep -i ADX").
>
> 2) So it seems these two drivers could never have been built from the
> mainline tree. (I'm starting to think that the config tools should
> somehow check stuff like this.)
>
> 3) Is the above correct? If it is, why shouldn't I submit the trivial
> patch to remove both these drivers?
Yes, all of that is correct. Unfortunately, I never got around to mainlining
the rest of the code. Feel free to remove both drivers and add my Acked-by.
Thierry
Content of type "application/pgp-signature" skipped
Powered by blists - more mailing lists