diff mbox

Patch Thinkpad X201T backlight

Message ID 8737p8aq7k.fsf@quengel.org (mailing list archive)
State Accepted, archived
Delegated to: Rafael Wysocki
Headers show

Commit Message

Ralf Gerbig May 22, 2016, 5:23 p.m. UTC
Hi,

the X201 Tablet needs the same treatmant as the X201s,
otherwise you can't change levels.

The patch is against 4.5.5 stable, but should apply to all recent
kernels.

Please be gentle, I am neither a programmer nor a developer.

If you have questions, feel free to ask, but don't expect an answer in
the next 10 to 20 days (going to hospital).

Regards,

Ralf Gerbig

---
commit a10b5a05eae1a990682ce238b8e0ac6ad791a8a9
Author: Ralf Gerbig <rge@quengel.org>
Date:   Sun May 22 18:56:22 2016 +0200

    The Thinkpad X201 Tablet also needs video_detect_force_video
    
    Signed-off-by: Ralf Gerbig <rge@quengel.org>

--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Aaron Lu June 3, 2016, 6:16 a.m. UTC | #1
On Sun, May 22, 2016 at 07:23:59PM +0200, Ralf Gerbig wrote:
> Hi,
> 
> the X201 Tablet needs the same treatmant as the X201s,
> otherwise you can't change levels.

I guess /sys/class/backlight/intel_backlight interface doesn't work?

> 
> The patch is against 4.5.5 stable, but should apply to all recent
> kernels.
> 
> Please be gentle, I am neither a programmer nor a developer.
> 
> If you have questions, feel free to ask, but don't expect an answer in
> the next 10 to 20 days (going to hospital).

Hope you are well.
Can you please, when you have time, attach your acpidump?
Perhaps file a new bug @https://bugzilla.kernel.org under
ACPI/Power-Video category and put the acpidump there, thanks.

Regards,
Aaron
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Rafael J. Wysocki June 23, 2016, 12:31 a.m. UTC | #2
On Sunday, May 22, 2016 07:23:59 PM Ralf Gerbig wrote:
> Hi,
> 
> the X201 Tablet needs the same treatmant as the X201s,
> otherwise you can't change levels.
> 
> The patch is against 4.5.5 stable, but should apply to all recent
> kernels.
> 
> Please be gentle, I am neither a programmer nor a developer.
> 
> If you have questions, feel free to ask, but don't expect an answer in
> the next 10 to 20 days (going to hospital).
> 
> Regards,
> 
> Ralf Gerbig
> 
> ---
> commit a10b5a05eae1a990682ce238b8e0ac6ad791a8a9
> Author: Ralf Gerbig <rge@quengel.org>
> Date:   Sun May 22 18:56:22 2016 +0200
> 
>     The Thinkpad X201 Tablet also needs video_detect_force_video
>     
>     Signed-off-by: Ralf Gerbig <rge@quengel.org>
> 
> diff --git a/drivers/acpi/video_detect.c b/drivers/acpi/video_detect.c
> index 1316ddd..d8d7bd3 100644
> --- a/drivers/acpi/video_detect.c
> +++ b/drivers/acpi/video_detect.c
> @@ -167,6 +167,14 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
>  		DMI_MATCH(DMI_PRODUCT_VERSION, "ThinkPad X201s"),
>  		},
>  	},
> +        {
> +         .callback = video_detect_force_video,
> +         .ident = "ThinkPad X201T",
> +         .matches = {
> +                DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
> +                DMI_MATCH(DMI_PRODUCT_VERSION, "ThinkPad X201T"),
> +                },
> +        },
>  
>  	/* The native backlight controls do not work on some older machines */
>  	{

Patch applied, thanks!

--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/acpi/video_detect.c b/drivers/acpi/video_detect.c
index 1316ddd..d8d7bd3 100644
--- a/drivers/acpi/video_detect.c
+++ b/drivers/acpi/video_detect.c
@@ -167,6 +167,14 @@  static const struct dmi_system_id video_detect_dmi_table[] = {
 		DMI_MATCH(DMI_PRODUCT_VERSION, "ThinkPad X201s"),
 		},
 	},
+        {
+         .callback = video_detect_force_video,
+         .ident = "ThinkPad X201T",
+         .matches = {
+                DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
+                DMI_MATCH(DMI_PRODUCT_VERSION, "ThinkPad X201T"),
+                },
+        },
 
 	/* The native backlight controls do not work on some older machines */
 	{