[PATCH v3 01/15] ARM: pxa: Don't rely on public mmc header to include leds.h
ulf.hansson at linaro.org
Fri Jan 20 00:27:29 PST 2017
On 20 January 2017 at 09:01, Robert Jarzmik <robert.jarzmik at free.fr> wrote:
> Ulf Hansson <ulf.hansson at linaro.org> writes:
>> Some of the pxa platforms, balloon3, colibri-pxa270-income, corgi,
>> trizeps4, vpac270, zeus and zylonite depends on leds.h. Explicitly include
>> it instead of relying on the public mmc header host.h.
>> Cc: Daniel Mack <daniel at zonque.org>
>> Cc: Haojian Zhuang <haojian.zhuang at gmail.com>
>> Cc: Robert Jarzmik <robert.jarzmik at free.fr>
>> Cc: <linux-arm-kernel at lists.infradead.org>
>> Signed-off-by: Ulf Hansson <ulf.hansson at linaro.org>
>> I am seeking an ack for this change as following changes for mmc in the
>> series, has build-dependencies to it.
> Acked-by: Robert Jarzmik <robert.jarzmik at free.fr>
> Actually I already have a patch for arch/arm/mach-pxa/idp.c which adds this
> include also in my -next tree, just in case you're tempted to modify this one,
> which would conflict with my pxa tree.
> An other point, I checked which files don't have a leds.h :
> - grep -ls _led arch/arm/mach-pxa/*.c > /tmp/a
> - grep -ls 'linux/leds.h' arch/arm/mach-pxa/*.c > /tmp/b
> - sdiff -s /tmp/a /tmp/b
> This gives me :
> So I was wondering how you made the choice of which files you add the include to
> and which you don't touch ?
By building the pxa_defconfig, see what errors it reports and then fix them.
Isn't pxa_defconfig building all the variants?
More information about the linux-arm-kernel