Bug 9702

Summary:	backlight support for macbooks
Product:	hal	Reporter:	Allison Lortie (desrt) <desrt>
Component:	hald	Assignee:	David Zeuthen (not reading bugmail) <zeuthen>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	high	CC:	bugzilla, chris, stephan.michels
Version:	unspecified
Hardware:	x86 (IA32)
OS:	Linux (All)
Whiteboard:
i915 platform:		i915 features:
Attachments:	support for macbook backlight

Description Allison Lortie (desrt) 2007-01-18 14:53:48 UTC

here's the code.

i'd email the mailing list, but this conference has no smtp server :)

Comment 1 Allison Lortie (desrt) 2007-01-18 14:54:26 UTC

Created attachment 8441 [details] [review]
support for macbook backlight

first try.  comments, please.

Comment 2 David Zeuthen (not reading bugmail) 2007-03-05 19:46:38 UTC

Looks good; any chance you can

 1) rebase to HEAD
 2) fix up coding style to match the rest of the source code?

thanks!

Comment 3 David Zeuthen (not reading bugmail) 2007-03-05 19:47:06 UTC

Also, you mentioned somewhere else that this has issues with suspend? What were those?

Comment 4 Richard Hughes 2007-03-19 13:18:55 UTC

Pushed to master:

add support for the macbook backlight

Apply a patch from Ryan Lortie <desrt@desrt.ca> to support the Macbook
backlight.
The code was written after the AppleIntelIntegratedFramebuffer.kext kernel
extension was reverse engineered by Ryan.

Rebased to master, and CodingStyle corrected by me, no other changes.

Comment 5 Allison Lortie (desrt) 2007-03-19 13:23:06 UTC

Thanks for this, Richard.

cheers

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.