DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=mime-version:date:message-id:subject:from:to:cc:content-type;
        b=BUmrH/fdbQnb+SJ+XPS15Bv300WzPXWhkoRwiIGyQixNuo+h+FpKu+zii42Lc97d1B
         mO+4k7d6PqNAobFNB3y2IUdRL0irPbMdg8Qg6K1lZ0Hcq7l6GzJEakk11p4gFAiGXmZ/
         Ogw6jTSfe5NN7YoUPP4fXULiwLUwknGFtnHoQ=
MIME-Version: 1.0
Date: Fri, 19 Feb 2010 00:52:05 -0700
Message-ID: <f9d2a5e11002182352ydb439c5j9684e03e6890257@mail.gmail.com>
Subject: drivers/backlight/lcd.c vs drivers/backlight/backlight.c
From: Russ Dill <russ.dill@gmail.com>
To: linux-kernel@vger.kernel.org
Cc: mjg@redhat.com
Content-Type: text/plain; charset=UTF-8
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 986
Lines: 20

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.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/