From: tomi.valkeinen@nokia.com (Tomi Valkeinen)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/2] OMAP: DSS2: Add new display panel
Date: Tue, 02 Nov 2010 11:19:06 +0200 [thread overview]
Message-ID: <1288689546.5614.60.camel@tubuntu> (raw)
In-Reply-To: <AANLkTi=fV58ApDqUE21YMK0_vhLdPbPr7F7YpYh0-2Vc@mail.gmail.com>
On Mon, 2010-11-01 at 17:45 +0100, ext Bryan Wu wrote:
> On Mon, Nov 1, 2010 at 10:15 AM, Tomi Valkeinen
> <tomi.valkeinen@nokia.com> wrote:
> > What we need is a common driver for all these dummy panels, and the
> > driver would contain configurations for all the supported panels. The
> > board file would pass the name of the panel used in that board to the
> > driver, and the driver would use the configurations for that panel.
> >
>
> It looks like that we just need one file name displays.c to do those
> generic things and pass the configuration from board files to this
> driver, such as timing or other things.
Not quite. If you pass timing information from the board files then
every board file has to define all the timings, even if they are using
the same panel.
That's why the configs should be in the panel driver, and the board file
only has to define which panel is actually connected.
> > This has been discussed a few times in l-o list, but I haven't found
> > time to implement it and nobody else has done it either.
> >
>
> I'm quite happy to do this, thanks,
Thanks, I appreciate that.
Tomi
prev parent reply other threads:[~2010-11-02 9:19 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-10-28 14:43 [PATCH 0/2] OMAP: DSS2: Add new display panel Enric Balletbo i Serra
2010-10-28 14:43 ` [PATCH 1/2] OMAP: DSS2: Add Powertip PH480272T " Enric Balletbo i Serra
2010-10-28 14:43 ` [PATCH 2/2] OMAP: DSS2: Add Seiko 70WVW1TZ3 " Enric Balletbo i Serra
2010-11-01 14:15 ` [PATCH 0/2] OMAP: DSS2: Add new " Tomi Valkeinen
2010-11-01 16:45 ` Bryan Wu
2010-11-02 9:19 ` Tomi Valkeinen [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1288689546.5614.60.camel@tubuntu \
--to=tomi.valkeinen@nokia.com \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).