From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jani Nikula Subject: Re: [PATCH 0/3] Use acpi_video_unregister_backlight instead of acpi_video_unregister in platfrom backlight drivers Date: Tue, 02 Jun 2015 12:59:10 +0300 Message-ID: <87zj4ie8i9.fsf@intel.com> References: <1433150708-8498-1-git-send-email-hdegoede@redhat.com> Mime-Version: 1.0 Content-Type: text/plain Return-path: In-Reply-To: <1433150708-8498-1-git-send-email-hdegoede@redhat.com> Sender: linux-acpi-owner@vger.kernel.org To: Hans de Goede , Darren Hart Cc: Aaron Lu , "Rafael J. Wysocki" , dri-devel@lists.freedesktop.org, platform-driver-x86@vger.kernel.org, linux-acpi@vger.kernel.org, Ben Skeggs List-Id: platform-driver-x86.vger.kernel.org On Mon, 01 Jun 2015, Hans de Goede 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? 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@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/dri-devel -- Jani Nikula, Intel Open Source Technology Center