Re: [PATCH -next] thinkpad_acpi: fix build, HOTKEY_POLL is optional

From: Henrique de Moraes Holschuh
Date: Fri Oct 09 2009 - 20:10:10 EST


On Fri, 09 Oct 2009, Randy Dunlap wrote:
> From: Randy Dunlap <randy.dunlap@xxxxxxxxxx>
>
> Fix optional part of driver to be built optionally:

NAK!

This happens because of (linux-next) commit
7f6443f7238a0aa9014b11a0a31ab76825d75cdf
"thinkpad-acpi: Fix build on !CONFIG_THINKPAD_ACPI_HOTKEY_POLL" by
Ingo.

That commit is wrong, it must NOT be applied as explained in the thread
starting at http://lkml.org/lkml/2009/9/27/30.

Ingo, can you please check if you don't have that commit still in one of
your trees that are fed into linux-next? It broke the driver now that the
"backlight/thinkpad-acpi: issue backlight class events" patch landed in
-next, just like I said it would happen...

--
"One disk to rule them all, One disk to find them. One disk to bring
them all and in the darkness grind them. In the Land of Redmond
where the shadows lie." -- The Silicon Valley Tarot
Henrique Holschuh
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/