On Wed, Jul 3, 2019 at 6:41 PM <Nemcev_Aleksey@xxxxxxxx> wrote: > > From: Aleksey Nemcev <Nemcev_Aleksey@xxxxxxxx> > > This model has an AURA RGB backlight, > so wmi_needs_3_args quirk is required Thanks for the patch. Though it does two logical things, i.e. new module support and new key bindings. Please, split it to two changes and justify key bindings. P.S. To all your patches, please use punctuation, like period at the end of sentences and so on. > > Signed-off-by: Aleksey Nemcev <Nemcev_Aleksey@xxxxxxxx> > --- > drivers/platform/x86/asus-nb-wmi.c | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > > diff --git a/drivers/platform/x86/asus-nb-wmi.c b/drivers/platform/x86/asus-nb-wmi.c > index 4bed95357a32..bf937321f42e 100644 > --- a/drivers/platform/x86/asus-nb-wmi.c > +++ b/drivers/platform/x86/asus-nb-wmi.c > @@ -406,6 +406,16 @@ static const struct dmi_system_id asus_quirks[] = { > }, > .driver_data = &quirk_asus_forceals, > }, > + { > + .callback = dmi_matched, > + .ident = "ASUSTeK COMPUTER INC. FX705GE", > + .matches = { > + DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."), > + DMI_MATCH(DMI_PRODUCT_NAME, > + "TUF Gaming FX705GE_FX705GE"), > + }, > + .driver_data = &quirk_asus_wmi_needs_3_args, > + }, > {}, > }; > > @@ -441,6 +451,7 @@ static const struct key_entry asus_nb_wmi_keymap[] = { > { KE_KEY, 0x30, { KEY_VOLUMEUP } }, > { KE_KEY, 0x31, { KEY_VOLUMEDOWN } }, > { KE_KEY, 0x32, { KEY_MUTE } }, > + { KE_KEY, 0x33, { KEY_BRIGHTNESS_MIN } }, > { KE_KEY, 0x35, { KEY_SCREENLOCK } }, > { KE_KEY, 0x40, { KEY_PREVIOUSSONG } }, > { KE_KEY, 0x41, { KEY_NEXTSONG } }, > @@ -467,6 +478,7 @@ static const struct key_entry asus_nb_wmi_keymap[] = { > { KE_KEY, 0x6B, { KEY_TOUCHPAD_TOGGLE } }, > { KE_IGNORE, 0x6E, }, /* Low Battery notification */ > { KE_KEY, 0x7a, { KEY_ALS_TOGGLE } }, /* Ambient Light Sensor Toggle */ > + { KE_KEY, 0x7C, { KEY_MICMUTE } }, > { KE_KEY, 0x7D, { KEY_BLUETOOTH } }, /* Bluetooth Enable */ > { KE_KEY, 0x7E, { KEY_BLUETOOTH } }, /* Bluetooth Disable */ > { KE_KEY, 0x82, { KEY_CAMERA } }, > -- > 2.20.1 > -- With Best Regards, Andy Shevchenko