> > > > 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. - 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