From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pavel Machek Subject: Re: [patch] added brightness feature to lcd class. Date: Mon, 9 Nov 2009 21:15:46 +0100 Message-ID: <20091109201541.GA28710@elf.ucw.cz> References: <90b950fc0911020050q3bc3fae8j954ec6ed73d79b5@mail.gmail.com> <20091105192725.GE1392@ucw.cz> <90b950fc0911070443s57d2ec5ehcb5c9cf9edcf8d8c@mail.gmail.com> <20091107164830.GA1586@ucw.cz> <90b950fc0911090737tde32562qc878fac38cacabae@mail.gmail.com> Mime-Version: 1.0 Return-path: Content-Disposition: inline In-Reply-To: <90b950fc0911090737tde32562qc878fac38cacabae@mail.gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: InKi Dae Cc: linux-fbdev-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org, Kyungmin Park On Tue 2009-11-10 00:37:59, InKi Dae wrote: > are you saying me that user shouldn't know if it is TFT-LCD or AMOLD? > I agree your saying. Parse error. > if lcd class has brightness feature then sysfs file for controlling > brightness will be placed > in /sys/class/lcd/*/brightness. > it would be a problem because the path is no sysfs you expected. > > how about that symbolic link file is created by lcd class for user? > like this, > /sys/class/lcd/*/brightness -> /sys/class/backlight/*/brightness > The symlink would have to be backwards, but yes, that would be better than nothing. (But I still don't see why we should make it complex. Just pretend it is backlight.) Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html