http://bugzilla.kernel.org/show_bug.cgi?id=15054


Zhang Rui <rui.zh...@intel.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |acpi-bugzi...@lists.sourcef
                   |                            |orge.net




--- Comment #14 from Zhang Rui <rui.zh...@intel.com>  2010-02-25 05:42:35 ---
(In reply to comment #13)
> Created an attachment (id=24744)
 --> (http://bugzilla.kernel.org/attachment.cgi?id=24744) [details]
> dmesg with nomodeset boot option after pressing fn+brightness up/down
> 
> brightness keys did have effect

I see.
On some HP laptops, DIDL is required to be initialized before _DOD is invoked,
thus we add the code to initialize DIDL in i915_opregion.c.
And now, on this laptop, it seems that the AML code in _DOD method
1. check if DIDL is initialized or not
2. If yes, follow the initialized value
3. If no, initialize DIDL in the AML code.

The problem is that
1. the graphics driver initialized DIDL value can not work on this laptop.
2. According to the spec, SBIOS(AML code) should not write to the DIDL field at
all. Only the graphics driver can set this field...

IMO, this is a firmware bug, not sure if we should workaround it in the
Linux/kernel,

-- 
Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

------------------------------------------------------------------------------
Download Intel&#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
_______________________________________________
acpi-bugzilla mailing list
acpi-bugzilla@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla

Reply via email to