From: Tomi Valkeinen <tomi.valkeinen@ti.com>
To: Mayuresh Janorkar <mayur@ti.com>
Cc: linux-omap@vger.kernel.org
Subject: Re: [PATCH v2 0/7] picodlp projector driver
Date: Tue, 03 May 2011 22:06:22 +0300 [thread overview]
Message-ID: <1304449582.26126.41.camel@deskari> (raw)
In-Reply-To: <1304347971-1504-1-git-send-email-mayur@ti.com>
On Mon, 2011-05-02 at 20:22 +0530, Mayuresh Janorkar wrote:
> picodlp projector is supported by OMAP.
> OMAP4430 SDP and EVM boards have an on board projector called as picodlp projector.
> picodlp would be connected to display sub system as a display panel.
> It has a dlp processor dpp2600.
>
> The panel would be connected using 24 bit parallel interface.
> It is a WVGA panel with 864 X 480 resolution.
>
> To know more about picodlp please visit:
> http://omappedia.org/wiki/PicoDLP_projector_guide
I think patches 3, 4, 5, 7 can be squashed into one. panel-picodlp.c is
not that big of a file, and the division is quite artificial to my eyes.
In theory the dss part and the i2c part could be in separate patches,
but that doesn't make much sense here as the dss part won't function
without the i2c part.
It's much easier to review one whole patch, than pieces of the whole.
Tomi
prev parent reply other threads:[~2011-05-03 19:06 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-05-02 14:52 [PATCH v2 0/7] picodlp projector driver Mayuresh Janorkar
2011-05-02 14:52 ` [PATCH v2 1/7] OMAP: DSS: Adding a header file for picodlp panel data Mayuresh Janorkar
2011-05-03 18:33 ` Tomi Valkeinen
2011-05-02 14:52 ` [PATCH v2 2/7] OMAP: DSS: Adding a picodlp panel header file Mayuresh Janorkar
2011-05-02 14:52 ` [PATCH v2 3/7] OMAP: DSS: Adding a picodlp panel driver Mayuresh Janorkar
2011-05-03 18:26 ` Tomi Valkeinen
2011-05-04 9:24 ` Janorkar, Mayuresh
2011-05-02 14:52 ` [PATCH v2 4/7] OMAP: DSS: Add i2c client driver for picodlp Mayuresh Janorkar
2011-05-03 18:44 ` Tomi Valkeinen
2011-05-04 10:01 ` Janorkar, Mayuresh
2011-05-04 11:05 ` Tomi Valkeinen
2011-05-04 14:29 ` Janorkar, Mayuresh
2011-05-02 14:52 ` [PATCH v2 5/7] OMAP: DSS: Adding initialization routine to picodlp panel Mayuresh Janorkar
2011-05-03 18:58 ` Tomi Valkeinen
2011-05-04 14:31 ` Janorkar, Mayuresh
2011-05-05 9:35 ` Tomi Valkeinen
2011-05-02 14:52 ` [PATCH v2 6/7] OMAP4: DSS: Adding a picodlp in OMAP4430 SDP board file Mayuresh Janorkar
2011-05-02 14:52 ` [PATCH v2 7/7] OMAP4: DSS: Adding picodlp panel entry in Kconfig and Makefile Mayuresh Janorkar
2011-05-03 19:06 ` 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=1304449582.26126.41.camel@deskari \
--to=tomi.valkeinen@ti.com \
--cc=linux-omap@vger.kernel.org \
--cc=mayur@ti.com \
/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