Re: [PATCH] Keyboard backlight on Dell E7470 not adjustable from /sys entry

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

 



On 2017-04-23 15:41, Pali Rohár wrote:
Such macros are not needed in mainline code. They are probably useful
in time of writing patch, but not later to have them present in final
driver code.

I sent a new patch (you are in recipients list too) which fixes getting
and setting timeout value when on AC. It also handles detection of
timeout AC value based on existence of 0x0451 SMBIOS token like Mario
suggested.

Your patch is great, thanks!

Regarding the debug macros: there are quite a few additional reserved bits still left in the relevant data structures that may cause trouble in the future. IMO having those macros available may speed up developing fixes in the future.

Thanks again,

--
Arcadiy Ivanov
arcadiy@xxxxxxxxxx | @arcivanov | https://ivanov.biz
https://github.com/arcivanov





[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux