On Tue, Jun 02, 2015 at 12:59:10PM +0300, Jani Nikula wrote: > On Mon, 01 Jun 2015, Hans de Goede <hdegoede@xxxxxxxxxx> wrote: > > Hi All, > > > > I'm working on cleaning up the currently somewhat convoluted logic to > > select which backlight interfaces to register on x86 systems, see: > > http://lists.freedesktop.org/archives/dri-devel/2014-December/074687.html > > > > For a rought outline (details will change in the actual patch-set). > > > > These 3 patches are a preparation for that work, as the behavior of the > > current code is not always consistent (it changes depending on module > > loading order in some cases). These 3 patches remove this inconsistency > > which in some cases may result in a behavior change. > > > > This way the cleanup can have a consistent base to build upon, and I can > > ensure that the cleanup itself does not cause any functional changes. > > Thanks for doing this. > > Slightly unrelated, but you'll end up calling: > > acpi_video_unregister_backlight > acpi_video_bus_unregister_backlight > acpi_video_dev_unregister_backlight > thermal_cooling_device_unregister > > Uhm, err, what? What business does *that* have in the backlight > unregister call chain?! Should that be untangled from the mess too? I think it should be OK. We currently will create a thermal cooling device for the ACPI backlight device, its intention is that we can achieve some thermal purpose by increasing or decreasing the backlight level(using the same set of methods as the ACPI backlight one, though I'm not sure if it is being used by anyone). When we are going to unregister the ACPI video backlight interface, it most likely means that the interface doesn't work so it doesn't make much sense to keep that thermal interface and hence the thermal_cooling_device_unregister ends up being called here. Regards, Aaron > > BR, > Jani. > > > > > > I hope to post a v1 of the actual cleanup patch-set in 1-2 weeks. > > > > Regards, > > > > Hans > > _______________________________________________ > > dri-devel mailing list > > dri-devel@xxxxxxxxxxxxxxxxxxxxx > > http://lists.freedesktop.org/mailman/listinfo/dri-devel > > -- > Jani Nikula, Intel Open Source Technology Center -- 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
