2010-02-19 07:52:10

by Russ Dill

[permalink] [raw]
Subject: drivers/backlight/lcd.c vs drivers/backlight/backlight.c

When these were initially checked in, they were basically the same,
except one was called backlight, the other lcd, one controlled
brightness, the other contrast.

Over time, they have diverged. backlight has sysfs_notify support,
provides uevent changes, and suspend/resume support. lcd has received
set_mode support.

Having a device that has variable brightness and needs info on video
timings, I'm confused at what to do. It seems other driver writers
chose to register and lcd device and a backlight device. It seems like
there is a lot of unnecessary duplication going on here. Should I
create my driver with both interfaces? Maybe with just use lcd and
pretend contrast is brightness, or maybe use backlight and add
set_mode support.