On Fri, 04 Jul 2008, Matthew Garrett wrote: > Got it. The reason for the delay is that BRTO sits in a loop for no > obvious reason unless BRTF is cleared. The only thing that clears BRTF > is the PWMS function, which isn't called anywhere in the ACPI tables. > The NetBSD driver has the following code: PWMS: missing from X300, X61t, X61... is this a new thing that Lenovo recently added to the firmware? I don't have very new dumps, particularly, none taken without OSI(Linux) -- might do something to the SSDTs -- so I might not have the full view of this. Anyway, if it is that new AND we need it to work well, it means we need to warn the user to upgrade the firmware when it is missing (and of course, add your code to use it when it is not missing :p). BRTO, BRTF: these are around for a while already. But I can't find anything that calls BRTO. What is NetBSD doing with BRTO? Can you send me your acpidump (and censored dmidecode) output so that I can look at a new enough dump with PWMS support? > and doing this in thinkpad_acpi means my brightness control works in an > acceptably speedy manner with opregion. Henrique, any thoughts on this? I'd like to have newer versions of the tables, before giving an opinion. Can you send me the dump for your thinkpad? > If you're happy with it, we can get opregion support into .27 and add > the patch to remove duplicate ACPI video devices and everything will > work in a beautiful and harmonious manner. I didn't look much at the code for removing duplicate ACPI video yet, but I am fine with the basic idea. However, if we are going to be disabling the entire firmware brightness change system (which will include ACPI BCM, probably... need the tables to know), that's something the X server needs to be able to enable/disable. Otherwise, we break all such changes outside of a X display, which is not acceptable. -- "One disk to rule them all, One disk to find them. One disk to bring them all and in the darkness grind them. In the Land of Redmond where the shadows lie." -- The Silicon Valley Tarot Henrique Holschuh -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
