Re: [PATCH v2 0/3] Support INT3406 Display thermal device

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

 



On Wednesday, April 13, 2016 03:32:48 PM Aaron Lu wrote:
> The display thermal device represents the LED/LCD display panel
> that may or may not include touch support. The main function of
> the display thermal device is to allow control of the display
> brightness in order to address a thermal condition or to reduce
> power consumed by display device.
> 
> Due to the way this thermal device changes brightness level is said
> to be deprecated so we are using the raw interface to do the actual
> backlight change. This requires the backlight core support so two
> new APIs are added and exported in patch 1/3. With this, the previous
> API backlight_device_registered can be removed and this is done in
> patch 2/3. Patch 3/3 adds the new int3406 thermal driver.
> 
> The 1st version is here:
> http://thread.gmane.org/gmane.linux.acpi.devel/72619
> 
> This whole series should go through the thermal tree if applied.
> 
> Aaron Lu (3):
>   video / backlight: add two APIs for drivers to use
>   video / backlight: remove the backlight_device_registered API
>   Thermal: add INT3406 thermal driver

So I have nothing against [1-2/3], but the [3/3] tries to do too many things
in one go.  It should be at least two separate patches to my eyes, one making
changes to the ACPI video driver and the other adding the INT3406 one.

Thanks,
Rafael

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



[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux