Re: [PATCH 0/3] Use acpi_video_unregister_backlight instead of acpi_video_unregister in platfrom backlight drivers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Darren,

On 01-06-15 19:41, Darren Hart wrote:
On Mon, Jun 01, 2015 at 11:25:05AM +0200, 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.

I hope to post a v1 of the actual cleanup patch-set in 1-2 weeks.

That will likely mean we miss the 4.2 merge window and will target this for 4.3
(which is fine by me, but just so you know).

IMHO this patches are worthwhile to have by themselves, as said
they fix various inconsistencies, which is a good thing to fix
regardless.

I think that these going into 4.2, and the cleanup which depends on them
(but is other then that pretty much an orthogonal patch set) going into
4.3 is actually a good thing, because if people then experience any
problems due to my work on this, we can easily see if it is due to
the behavior change these 3 patches may introduce in some cases, or
if it is due to the actual cleanup causing a behavior change (which it
should not).

Regards,

Hans
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel





[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux