Re: Display class
From: James Simmons <hidden>
Date: 2006-12-06 19:13:56
Also in:
linux-acpi, lkml
quoted
quoted
quoted
That patch was rought draft for feedback. I applied your comments. This patch actually works. It includes my backlight fix as well.Glad to hear it. I had to make the following changes in order for it to build. However, I still have build errors for aty.Ug. I see another problem. I had backlight completly compiled as a module! Thus it hid these compile errors. So we need also a CONFIG_BACKLIGHT_CLASS_DEVICE_MODULE check as well. Can sysfs handle this well or would it be better the the backlight class be a boolean instead?SCSI works as a module and it uses sysfs. See drivers/scsi/scsi_sysfs.c. Does that answer your question? I wasn't quite sure what the question was.
I'm scratching my head on how to configure a modular driver and two modular sysfs classes.
Next question, based on: drivers/built-in.o: In function `probe_edid': (.text.probe_edid+0x42): undefined reference to `fb_edid_to_monspecs' Should backlight and/or display support depend on CONFIG_FB? Right now they don't, so the above can happen...
I already sent a patch to Andrew to make backlight/lcd work independent of CONFIG_FB. Display is still in the alpha stage. In time it will work independent of CONFIG_FB.