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





------- Comment #37 from [EMAIL PROTECTED]  2008-07-10 00:02 -------
Zhang, thanks a lot for testing and looking through this!

> So, IMO, IGD support should not be involved in the second patch.
Yes, I will remove it in the next version.
I thought the IGD support takes longer, but maybe now all works out.
It would still be nice to be able to switch backlight without IGD (e.g. if
parts of the BIOS are missing as Matthew stated), but this could be added on
top if needed, I like to make this as easy as possible for now to get it
upstream.

Be careful the patch has a bug which can lead to a kernel oops:
+       acpi_bus_get_device(handle, &acpi_dev);
in find_video() must be checked for it's return value.

I will send the next version on the list.


-- 
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, or are watching someone who is.

-------------------------------------------------------------------------
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
_______________________________________________
acpi-bugzilla mailing list
acpi-bugzilla@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla

Reply via email to