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-15 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-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: [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: [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: [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: [PATCH] video: backlight: Remove backlight sysfs uevent

2013-11-11 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: [PATCH] video: backlight: Remove backlight sysfs uevent

2013-11-11 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: [PATCH] video: backlight: Remove backlight sysfs uevent

2013-11-11 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: [PATCH] video: backlight: Remove backlight sysfs uevent

2013-11-11 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: [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

Re: [PATCH] video: backlight: Remove backlight sysfs uevent

2013-11-11 Thread Kyungmin Park
Please ignore it. strange mail system. After feedback from previous mail, I'll re-send it. Thank you, Kyungmin Park On Tue, Nov 12, 2013 at 9:44 AM, Kyungmin Park wrote: > From: Kyungmin Park > > The most mobile phones have Ambient Light Sensors and it changes brightness > according lux. > It

[PATCH] video: backlight: Remove backlight sysfs uevent

2013-11-11 Thread Kyungmin Park
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. Usually there's no user to use this backlight changes. But it forks udev worke

Re: [PATCH] video: backlight: Remove backlight sysfs uevent

2013-11-11 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. > >

[PATCH] video: backlight: Remove backlight sysfs uevent

2013-11-11 Thread Kyungmin Park
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. Usually there's no user to use this backlight changes. But it forks udev worke