From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matthew Garrett Date: Wed, 20 Nov 2013 23:40:59 +0000 Subject: Re: [PATCH] video: backlight: Remove backlight sysfs uevent Message-Id: <1384990859.20536.4.camel@x230> List-Id: References: <20131111235700.GA29987@july> <002901cedf3c$a7e77a00$f7b66e00$%han@samsung.com> <20131112005628.GA2914@khazad-dum.debian.net> In-Reply-To: <20131112005628.GA2914@khazad-dum.debian.net> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Henrique de Moraes Holschuh Cc: Jingoo Han , 'Kyungmin Park' , 'Henrique de Moraes Holschuh' , linux-fbdev@vger.kernel.org, linux-kernel@vger.kernel.org, kay@vrfy.org, 'Richard Purdie' , ibm-acpi-devel@lists.sourceforge.net, platform-driver-x86@vger.kernel.org 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 should set when they need to supress change events? It looks like this is just to force synchronisation to sysfs when using the /proc interface? In which case we should probably just kill the /proc interface. -- Matthew Garrett