From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matthew Garrett Subject: Re: [PATCH] backlight: add support for disabling backlights via sysfs Date: Thu, 28 Jun 2012 22:19:36 +0100 Message-ID: <20120628211936.GA14286@srcf.ucam.org> References: <1340910426-13269-1-git-send-email-seth.forshee@canonical.com> <20120628191043.GA9777@srcf.ucam.org> <20120628193017.GA29456@thinkpad-t410> <20120628193652.GB10945@srcf.ucam.org> <20120628211600.GB29456@thinkpad-t410> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20120628211600.GB29456@thinkpad-t410> Sender: linux-kernel-owner@vger.kernel.org To: Richard Purdie , linux-kernel@vger.kernel.org, platform-driver-x86@vger.kernel.org, Corentin Chary List-Id: platform-driver-x86.vger.kernel.org On Thu, Jun 28, 2012 at 04:16:00PM -0500, Seth Forshee wrote: > What I still don't understand is your reason for preferring > driver-specific interfaces for disabling backlights to a single generic > interface. Are you afraid that allowing it to be done from userspace > will make it too easy, and as a result quirks will be applied without > attempting to fix the problem (i.e. the quriking should go through a > kernel dev to ensure someone tries to fix the problem is made before > applying a quirk)? Yup, that. We'll just end up with three billion forum posts telling people to add a line to a text file and nobody will ever fix the real problem. -- Matthew Garrett | mjg59@srcf.ucam.org