Hi Thomas, On Thu, 10 Dec 2009 16:18:27 +0100, Thomas Renninger <trenn@xxxxxxx> wrote : > Hi Anisse, > > thanks for these! > A quick comment on each: > > > Anisse Astier (6): > > msi-wmi: remove useless includes > -> good catch, I probably C&P it from another acpi file. > > msi-wmi: rework init > -> might look a bit nicer now, thanks. > > msi-wmi: remove custom runtime debug implementation > -> fine. > > msi-wmi: remove unused field 'instance' in key_entry structure > -> nice cleanup > > msi-wmi: replace one-condition switch-case with if statement > -> might be nicer > > msi-wmi: switch to using input sparse keymap library > -> I didn't go into detail, but this looks like a very nice > cleanup and code reduction. > Thanks for taking the time to check this out. Anisse -- 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
