[PATCH] [POWERPC] Add the PC speaker only when requested so
grant.likely at secretlab.ca
Fri May 23 14:44:06 EST 2008
On Thu, May 22, 2008 at 9:34 PM, Kumar Gala <galak at kernel.crashing.org> wrote:
> On May 22, 2008, at 6:27 PM, Grant Likely wrote:
>> On Thu, May 22, 2008 at 4:40 PM, Emil Medve <Emilian.Medve at freescale.com>
>>> This will cause this minor boot-time debugging error message to go away:
>>> [ 1.316451] calling add_pcspkr+0x0/0x84
>>> [ 1.316478] initcall add_pcspkr+0x0/0x84 returned -19 after 0 msecs
>> What situation are you hitting this in? The code should only run if
>> there is a "pnpPNP,100" compatible node in the device tree.
> The code always runs, the -19 is from the fact that the code returns -ENODEV
> when it doesn't find the device in the tree.
> I don't see any reason we should be ALWAYS be probing for a PC speaker.
> Seems like a reasonable patch.
Fair enough; I was just wondering if it was a complete fix. But now
that I dig into the code I see that the calling...initcall returned...
messages are just debug stuff that isn't always turned on.
I have no problem with the patch.
>> Also, where is CONFIG_PCSPKR_PLATFORM defined? I don't see it
>> anywhere in powerpc code and only a reference to it in an x86
>> Makefile. As it stands, it looks like this patch unconditionally
>> disables the pcspkr code.
> Its defined in init/Kconfig.
Apparently my grep-foo isn't what it used to be...
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.
More information about the Linuxppc-dev