Re: [9fans] Adjust Thinkpad Screen Brightness

[unobe@cpan.org]

Note the patch is for both the Ivy Bridge listed devices, but mine
specifically is 8086/0166.  I'm curious if there's someone with 0152.

Quoth unobe@cpan.org:
> Quoth Anthony Martin <ality@pbrane.org>:
> > unobe@cpan.org once said:
> > > I reviewed the data sheet and the scripts should
> > > work, but does not.
> > 
> > The above scripts can work for some Intel cards
> > but probably not for all of them.
> 
> Thank you, Anthony.  That script helped me review the volumes again
> and find out where the 'br' script is wrong.  It was just reading from
> the wrong register for Ivy Bridge.  Here's the patch for qwx/rc's
> branch, and it works like a charm now on my x230t.
> 
> From: Romano <unobe@cpan.org>
> Date: Thu, 08 Jun 2023 07:49:37 +0000
> Subject: [PATCH] Ivy Bridge uses South Display for reading brightness levels, per Vol. 3 Part 3 and 4 of Programmer's Reference
> 
> ---
> diff a9e9e008d4574c55fd9452de2cfc2cf9d964e7e5 d52fd7d1b16ce1e4f06607bfcec6859285077536
> --- a/bin/br
> +++ b/bin/br
> @@ -29,8 +29,8 @@
>         # works on x220, not on w520
>         did["8086/0126","r"] = "0xc8254";  did["8086/0126","w"] = "0x48254"
>         # ivy bridge
> -       did["8086/0152","r"] = "0x48254";  did["8086/0152","w"] = "0x48254"
> -       did["8086/0166","r"] = "0x48254";  did["8086/0166","w"] = "0x48254"
> +       did["8086/0152","r"] = "0xc8254";  did["8086/0152","w"] = "0x48254"
> +       did["8086/0166","r"] = "0xc8254";  did["8086/0166","w"] = "0x48254"
>         # haswell
> did["8086/0412","r"] = "0xc8254";  did["8086/0412","w"] = "0x48254"
> did["8086/041a","r"] = "0xc8254";  did["8086/041a","w"] = "0x48254"
> 

------------------------------------------
9fans: 9fans
Permalink: https://9fans.topicbox.com/groups/9fans/T4c4247ec1c0429f6-M2ee41d515969f35ca06223f8
Delivery options: https://9fans.topicbox.com/groups/9fans/subscription