Re: [ibm-acpi-devel] [PATCH] video: backlight: Remove backlight sysfs uevent

2013-12-18 Thread Henrique de Moraes Holschuh
On Sun, 15 Dec 2013, Andrew Morton wrote: > On Sun, 24 Nov 2013 01:53:11 -0200 Henrique de Moraes Holschuh > wrote: > > On Sun, 24 Nov 2013, Matthew Garrett wrote: > > > On Sat, Nov 23, 2013 at 10:40:15PM -0200, Henrique de Moraes Holschuh > > > wrote: > > > > On Fri, 22 Nov 2013, Matthew Garret

Re: [ibm-acpi-devel] [PATCH] video: backlight: Remove backlight sysfs uevent

2013-12-16 Thread Andrew Morton
On Sun, 24 Nov 2013 01:53:11 -0200 Henrique de Moraes Holschuh wrote: > On Sun, 24 Nov 2013, Matthew Garrett wrote: > > On Sat, Nov 23, 2013 at 10:40:15PM -0200, Henrique de Moraes Holschuh wrote: > > > On Fri, 22 Nov 2013, Matthew Garrett wrote: > > > > We have userspace that relies on uevents

Re: [ibm-acpi-devel] [PATCH] video: backlight: Remove backlight sysfs uevent

2013-12-02 Thread Kyungmin Park
On Tue, Nov 12, 2013 at 10:19 AM, Kay Sievers wrote: > On Tue, Nov 12, 2013 at 1:56 AM, Henrique de Moraes Holschuh > wrote: >> On Tue, 12 Nov 2013, Jingoo Han wrote: >>> On Tuesday, November 12, 2013 8:57 AM, Kyungmin Park wrote: >>> > From: Kyungmin Park >>> > >>> > The most mobile phones have

Re: [ibm-acpi-devel] [PATCH] video: backlight: Remove backlight sysfs uevent

2013-12-02 Thread Kay Sievers
On Tue, Nov 12, 2013 at 3:08 AM, Kyungmin Park wrote: > On Tue, Nov 12, 2013 at 10:19 AM, Kay Sievers wrote: >> On Tue, Nov 12, 2013 at 1:56 AM, Henrique de Moraes Holschuh >> wrote: >>> On Tue, 12 Nov 2013, Jingoo Han wrote: On Tuesday, November 12, 2013 8:57 AM, Kyungmin Park wrote:

Re: [ibm-acpi-devel] [PATCH] video: backlight: Remove backlight sysfs uevent

2013-12-02 Thread Kay Sievers
On Tue, Nov 12, 2013 at 1:56 AM, Henrique de Moraes Holschuh wrote: > On Tue, 12 Nov 2013, Jingoo Han wrote: >> On Tuesday, November 12, 2013 8:57 AM, Kyungmin Park wrote: >> > From: Kyungmin Park >> > >> > The most mobile phones have Ambient Light Sensors and it changes >> > brightness accordin

Re: [ibm-acpi-devel] [PATCH] video: backlight: Remove backlight sysfs uevent

2013-12-02 Thread Jingoo Han
On Tuesday, November 12, 2013 8:57 AM, Kyungmin Park wrote: > > From: Kyungmin Park > > The most mobile phones have Ambient Light Sensors and it changes brightness > according lux. > It means it changes backlight brightness frequently by just writing sysfs > node, so it generates uevent. > >

Re: [ibm-acpi-devel] [PATCH] video: backlight: Remove backlight sysfs uevent

2013-12-02 Thread Kyungmin Park
On Tue, Nov 12, 2013 at 9:56 AM, Henrique de Moraes Holschuh wrote: > On Tue, 12 Nov 2013, Jingoo Han wrote: >> On Tuesday, November 12, 2013 8:57 AM, Kyungmin Park wrote: >> > From: Kyungmin Park >> > >> > The most mobile phones have Ambient Light Sensors and it changes >> > brightness accordin

Re: [ibm-acpi-devel] [PATCH] video: backlight: Remove backlight sysfs uevent

2013-11-23 Thread Henrique de Moraes Holschuh
On Sun, 24 Nov 2013, Matthew Garrett wrote: > On Sat, Nov 23, 2013 at 10:40:15PM -0200, Henrique de Moraes Holschuh wrote: > > On Fri, 22 Nov 2013, Matthew Garrett wrote: > > > We have userspace that relies on uevents of type > > > BACKLIGHT_UPDATE_HOTKEY. I don't know that we have userspace that

Re: [ibm-acpi-devel] [PATCH] video: backlight: Remove backlight sysfs uevent

2013-11-23 Thread Matthew Garrett
On Sat, Nov 23, 2013 at 10:40:15PM -0200, Henrique de Moraes Holschuh wrote: > On Fri, 22 Nov 2013, Matthew Garrett wrote: > > We have userspace that relies on uevents of type > > BACKLIGHT_UPDATE_HOTKEY. I don't know that we have userspace that relies > > on uevents of type BACKLIGHT_UPDATE_SYSF

Re: [ibm-acpi-devel] [PATCH] video: backlight: Remove backlight sysfs uevent

2013-11-23 Thread Henrique de Moraes Holschuh
On Fri, 22 Nov 2013, Matthew Garrett wrote: > On Fri, Nov 22, 2013 at 09:36:01AM -0200, Henrique de Moraes Holschuh wrote: > > On Thu, 21 Nov 2013, Matthew Garrett wrote: > > > The uevent support was initially added to handle systems where pressing > > > a hotkey generates an event (good) but the

Re: [ibm-acpi-devel] [PATCH] video: backlight: Remove backlight sysfs uevent

2013-11-22 Thread Matthew Garrett
On Fri, Nov 22, 2013 at 09:36:01AM -0200, Henrique de Moraes Holschuh wrote: > On Thu, 21 Nov 2013, Matthew Garrett wrote: > > The uevent support was initially added to handle systems where pressing > > a hotkey generates an event (good) but the firmware automatically > > changes the brightness (

Re: [ibm-acpi-devel] [PATCH] video: backlight: Remove backlight sysfs uevent

2013-11-22 Thread Henrique de Moraes Holschuh
On Thu, 21 Nov 2013, Matthew Garrett wrote: > On Thu, Nov 21, 2013 at 09:43:32AM -0200, Henrique de Moraes Holschuh wrote: > > With this patchset applied, as far as I can tell anything that used to be > > uevent-driven by the backlight class will break: when a process changes the > > backlight usin

Re: [ibm-acpi-devel] [PATCH] video: backlight: Remove backlight sysfs uevent

2013-11-21 Thread Matthew Garrett
On Thu, Nov 21, 2013 at 09:43:32AM -0200, Henrique de Moraes Holschuh wrote: > With this patchset applied, as far as I can tell anything that used to be > uevent-driven by the backlight class will break: when a process changes the > backlight using sysfs, other processes will not be notified of th

Re: [ibm-acpi-devel] [PATCH] video: backlight: Remove backlight sysfs uevent

2013-11-21 Thread Henrique de Moraes Holschuh
On Wed, 20 Nov 2013, Matthew Garrett wrote: > On Mon, 2013-11-11 at 22:56 -0200, Henrique de Moraes Holschuh wrote: > > On Tue, 12 Nov 2013, Jingoo Han wrote: > > > 'thinkpad_acpi.c' uses the 'BACKLIGHT_UPDATE_SYSFS'. > > > Henrique, can we remove it? > > > > Can't you fix this by rate-limiting, o

Re: [ibm-acpi-devel] [PATCH] video: backlight: Remove backlight sysfs uevent

2013-11-20 Thread Matthew Garrett
On Mon, 2013-11-11 at 22:56 -0200, Henrique de Moraes Holschuh wrote: > On Tue, 12 Nov 2013, Jingoo Han wrote: > > 'thinkpad_acpi.c' uses the 'BACKLIGHT_UPDATE_SYSFS'. > > Henrique, can we remove it? > > Can't you fix this by rate-limiting, or otherwise adding an attribute that > backlight devices

Re: [ibm-acpi-devel] [PATCH] video: backlight: Remove backlight sysfs uevent

2013-11-11 Thread Henrique de Moraes Holschuh
On Tue, 12 Nov 2013, Jingoo Han wrote: > On Tuesday, November 12, 2013 8:57 AM, Kyungmin Park wrote: > > From: Kyungmin Park > > > > The most mobile phones have Ambient Light Sensors and it changes brightness > > according lux. > > It means it changes backlight brightness frequently by just writ