Re: [PATCH 2/4] platform/x86: asus-nb-wmi: Add Asus FX705GE support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux