Re: [ibm-acpi-devel] [PATCH v5 2/6] leds: triggers: Add a keyboard backlight trigger

2016-11-24 Thread Pali Rohár
On Wednesday 23 November 2016 12:01:02 Jacek Anaszewski wrote: > I would also appreciate your opinion on the other solution to the > problem of notifying brightness changes originating from hardware, > i.e. hw_brightness_change{_ro} file, that would support POLLPRI events, > and reading brightness.

Re: [ibm-acpi-devel] [PATCH v5 2/6] leds: triggers: Add a keyboard backlight trigger

2016-11-24 Thread Hans de Goede
Hi Pali, On 24-11-16 10:15, Pali Rohár wrote: > On Wednesday 23 November 2016 12:01:02 Jacek Anaszewski wrote: >> I would also appreciate your opinion on the other solution to the >> problem of notifying brightness changes originating from hardware, >> i.e. hw_brightness_change{_ro} file, that wou

Re: [ibm-acpi-devel] [PATCH v5 2/6] leds: triggers: Add a keyboard backlight trigger

2016-11-24 Thread Pali Rohár
On Thursday 24 November 2016 15:21:36 Jacek Anaszewski wrote: > On 11/24/2016 10:15 AM, Pali Rohár wrote: > >On Wednesday 23 November 2016 12:01:02 Jacek Anaszewski wrote: > >>I would also appreciate your opinion on the other solution to the > >>problem of notifying brightness changes originating f

Re: [ibm-acpi-devel] [PATCH v5 2/6] leds: triggers: Add a keyboard backlight trigger

2016-11-24 Thread Pali Rohár
On Thursday 24 November 2016 16:32:06 Jacek Anaszewski wrote: > Since it has been reported that POLLPRI notifications on brightness > file can lead to increased power consumption, and having my above > statement I don't think that it is a good idea to use brightness > file for this. How is brightn

Re: [ibm-acpi-devel] [PATCH v5 2/6] leds: triggers: Add a keyboard backlight trigger

2016-11-24 Thread Pali Rohár
On Thursday 24 November 2016 17:21:19 Jacek Anaszewski wrote: > On 11/24/2016 04:36 PM, Pali Rohár wrote: > > On Thursday 24 November 2016 16:32:06 Jacek Anaszewski wrote: > >> Since it has been reported that POLLPRI notifications on > >> brightness file can lead to increased power consumption, and

Re: [ibm-acpi-devel] [PATCH v5 2/6] leds: triggers: Add a keyboard backlight trigger

2016-11-24 Thread Pali Rohár
On Thursday 24 November 2016 22:35:52 Jacek Anaszewski wrote: > > I understood that we cannot notify about changes done by CPU > > trigger due to high power usage... Or not? > > Exactly. So in this case exporting any new sysfs file (or using existing) which report POLLPRI events for LED devices