public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* Advice on backlight support
@ 2007-02-21 16:02 Rodolfo Giometti
  2007-02-21 16:14 ` Paul Sokolovsky
  2007-02-25 10:43 ` Pavel Machek
  0 siblings, 2 replies; 3+ messages in thread
From: Rodolfo Giometti @ 2007-02-21 16:02 UTC (permalink / raw)
  To: linux-kernel

Hello,

I'd like to add backlight support for input devices since my custom
board has a backlighted mini keyboard.

It could be acceptable to move the code from drivers/video/backlight/
to drivers/backlight/ renaming the "Backlight & LCD" name into
"Backlight" and adding two new entries "LCD" and, maybe, "Input" (or
something like that?).

Thanks in advance,

Rodolfo

-- 

GNU/Linux Solutions                  e-mail:    giometti@enneenne.com
Linux Device Driver                             giometti@gnudd.com
Embedded Systems                     		giometti@linux.it
UNIX programming                     phone:     +39 349 2432127

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Advice on backlight support
  2007-02-21 16:02 Advice on backlight support Rodolfo Giometti
@ 2007-02-21 16:14 ` Paul Sokolovsky
  2007-02-25 10:43 ` Pavel Machek
  1 sibling, 0 replies; 3+ messages in thread
From: Paul Sokolovsky @ 2007-02-21 16:14 UTC (permalink / raw)
  To: Rodolfo Giometti; +Cc: linux-kernel

Hello Rodolfo,

Wednesday, February 21, 2007, 6:02:13 PM, you wrote:

> Hello,

> I'd like to add backlight support for input devices since my custom
> board has a backlighted mini keyboard.

  There's already generic indicator API, currently mostly known as
"[new] LED [classdev] API", even though it is already used for things
like buzzers, vibrators, etc. Why wouldn't it be suitable for your
case?

  LCD and LCD backlight are special cases, as they actually should
track state of (abstract) FB device, and keeping their states in sync
is pretty important for trivial PM and info accessibility, so worth
doing on kernel level.

  As for things like "light up keyboard if key was pressed, then light
down after timeout", or "light up backlight LCD on touchscreen touch,
light down on timeout", that can be pretty well done in userspace, no?

> It could be acceptable to move the code from drivers/video/backlight/
> to drivers/backlight/ renaming the "Backlight & LCD" name into
> "Backlight" and adding two new entries "LCD" and, maybe, "Input" (or
> something like that?).

  Yes, the fact that lcd.c (LCD power control) lives under backlight
dir, is a bit confusing. But they still has close relation to FB,
after all.

> Thanks in advance,

> Rodolfo




-- 
Best regards,
 Paul                            mailto:pmiscml@gmail.com


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Advice on backlight support
  2007-02-21 16:02 Advice on backlight support Rodolfo Giometti
  2007-02-21 16:14 ` Paul Sokolovsky
@ 2007-02-25 10:43 ` Pavel Machek
  1 sibling, 0 replies; 3+ messages in thread
From: Pavel Machek @ 2007-02-25 10:43 UTC (permalink / raw)
  To: Rodolfo Giometti; +Cc: linux-kernel

Hi!

> I'd like to add backlight support for input devices since my custom
> board has a backlighted mini keyboard.
> 
> It could be acceptable to move the code from drivers/video/backlight/
> to drivers/backlight/ renaming the "Backlight & LCD" name into
> "Backlight" and adding two new entries "LCD" and, maybe, "Input" (or
> something like that?).

Either that or using led class...

...keyboard backlight actually has selectable level in many cases, so
I'm not sure how possible it is to do with led class...?
									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2007-02-25 10:43 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-02-21 16:02 Advice on backlight support Rodolfo Giometti
2007-02-21 16:14 ` Paul Sokolovsky
2007-02-25 10:43 ` Pavel Machek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox