Re: [9fans] Adjust Thinkpad Screen Brightness

[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-M7438e53aae9530c902ef75f6
Delivery options: https://9fans.topicbox.com/groups/9fans/subscription