2012-08-01 21:06:01

by Ondrej Zary

Subject: [PATCH 2/2] leds-clevo-mail: Fix Clevo M5x0V DMI strings

There are more variants of Clevo M5x0V laptops and Positivo Mobile is just one of them.
Another one is UMAX VisionBook 2400WXC. Use only generic Clevo DMI strings to match all
variants.

Signed-off-by: Ondrej Zary <[email protected]>

--- a/drivers/leds/leds-clevo-mail.c
+++ b/drivers/leds/leds-clevo-mail.c
@@ -59,11 +59,10 @@ static struct dmi_system_id __initdata clevo_mail_led_dmi_table[] = {
},
{
.callback = clevo_mail_led_dmi_callback,
- .ident = "Positivo Mobile",
+ .ident = "Clevo M5x0V",
.matches = {
DMI_MATCH(DMI_BOARD_VENDOR, "CLEVO Co. "),
DMI_MATCH(DMI_BOARD_NAME, "M5X0V "),
- DMI_MATCH(DMI_PRODUCT_NAME, "Positivo Mobile"),
DMI_MATCH(DMI_PRODUCT_VERSION, "VT6198")
}
},




--
Ondrej Zary

2012-08-02 03:46:37

by Bryan Wu

Subject: Re: [PATCH 2/2] leds-clevo-mail: Fix Clevo M5x0V DMI strings

On Thu, Aug 2, 2012 at 5:04 AM, Ondrej Zary <[email protected]> wrote:
> There are more variants of Clevo M5x0V laptops and Positivo Mobile is just one of them.
> Another one is UMAX VisionBook 2400WXC. Use only generic Clevo DMI strings to match all
> variants.
>

Thanks, applied to my for-next branch
-Bryan

> Signed-off-by: Ondrej Zary <[email protected]>
>
> --- a/drivers/leds/leds-clevo-mail.c
> +++ b/drivers/leds/leds-clevo-mail.c
> @@ -59,11 +59,10 @@ static struct dmi_system_id __initdata clevo_mail_led_dmi_table[] = {
> },
> {
> .callback = clevo_mail_led_dmi_callback,
> - .ident = "Positivo Mobile",
> + .ident = "Clevo M5x0V",
> .matches = {
> DMI_MATCH(DMI_BOARD_VENDOR, "CLEVO Co. "),
> DMI_MATCH(DMI_BOARD_NAME, "M5X0V "),
> - DMI_MATCH(DMI_PRODUCT_NAME, "Positivo Mobile"),
> DMI_MATCH(DMI_PRODUCT_VERSION, "VT6198")
> }
> },
>
>
>
>
> --
> Ondrej Zary



--
Bryan Wu <[email protected]>
Kernel Developer +86.186-168-78255 Mobile
Canonical Ltd. http://www.canonical.com
Ubuntu - Linux for human beings | http://www.ubuntu.com