On Saturday 27 May 2017 07:15:28 Darren Hart wrote: > From: Andy Lutomirski <[email protected]> > > The hotkey table is 0xb2, add a comment for clarity. > > Suggested-by: Darren Hart <[email protected]> > Signed-off-by: Andy Lutomirski <[email protected]> > Cc: Matthew Garrett <[email protected]> > Cc: "Pali Rohár" <[email protected]> > Cc: Andy Shevchenko <[email protected]> > Signed-off-by: Darren Hart (VMware) <[email protected]> > --- > drivers/platform/x86/dell-wmi.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/platform/x86/dell-wmi.c > b/drivers/platform/x86/dell-wmi.c index 8a64c79..24467b1 100644 > --- a/drivers/platform/x86/dell-wmi.c > +++ b/drivers/platform/x86/dell-wmi.c > @@ -449,6 +449,7 @@ static void __init handle_dmi_entry(const struct > dmi_header *dm, if (results->err || results->keymap) > return; /* We already found the hotkey table. */ > > + /* The Dell hotkey table is type 0xB2. Scan until we find it. */ > if (dm->type != 0xb2) > return;
Good. Reviewed-by: Pali Rohár <[email protected]> (Maybe in future we should add a #define name for such magic constants) -- Pali Rohár [email protected]
signature.asc
Description: This is a digitally signed message part.
