On Thu, Dec 9, 2010 at 8:56 AM, Ike Panhc <ike.pan@xxxxxxxxxxxxx> wrote: > Here are the enablement patches for hotkey events on ideapads and these patches > are made against current checkout of mainline kernel. > > After last time posting on LKML, there are several feedback and these patches > are modified. > > From Dmitry Torokhov, using markup on functions to let the compiler knows where > to put and passing private variable instead of using global vairable. The Great, > acpi_handle is still a global variable becuase we need to pass handle pointer > to rfk_set with its opcode but only one data argument available. Of course there is only one data argument available, but since it's a void *, you can use a struct with your handle and the opcode :). > From Dave Hansen, select INPUT_SPARSEKMAP to fill the dependency. > > These patches are available in the git repository at: > Âgit://kernel.ubuntu.com/ikepanhc/ideapad-laptop.git ideapad-laptop > > Ike Panhc (7): > Âideapad: add platform driver for ideapad > Âideapad: let camera power control entry under platform driver Maybe you could document that ? See Documentation/ABI/testing/sysfs-platform-* files. > Âideapad: add hotkey support > Âideapad: select INPUT_SPARSEKMAP > Âideapad: add markups, unify comments and return result when init > Âideapad: pass ideapad_priv as argument (part 1) > Âideapad: pass ideapad_priv as argument (part 2) > > Âdrivers/platform/x86/Kconfig     Â|  Â1 + > Âdrivers/platform/x86/ideapad-laptop.c | Â255 ++++++++++++++++++++++---------- > Â2 files changed, 176 insertions(+), 80 deletions(-) -- Corentin Chary http://xf.iksaif.net -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
