From: Tomi Valkeinen <tomi.valkeinen@ti.com>
To: "K, Mythri P" <mythripk@ti.com>
Cc: linux-omap@vger.kernel.org
Subject: Re: [PATCH v3 00/10]HDMI: Split hdmi.c in DSS to seperate OMAP dependent
Date: Mon, 05 Sep 2011 13:41:15 +0300 [thread overview]
Message-ID: <1315219275.3214.21.camel@deskari> (raw)
In-Reply-To: <CAP5A+B8+f8mnFPAOJ8E5HX-5AbSRPwicHSEaDke8s9tQ0Fj0KA@mail.gmail.com>
On Mon, 2011-09-05 at 16:05 +0530, K, Mythri P wrote:
> Hi,
>
> On Mon, Sep 5, 2011 at 1:10 PM, Tomi Valkeinen <tomi.valkeinen@ti.com> wrote:
> > On Fri, 2011-09-02 at 16:17 +0530, mythripk@ti.com wrote:
> >> From: Mythri P K <mythripk@ti.com>
> >>
> >> HDMI IP block is common between TI OMAP4 Procerssor and Netra processor although
> >> the Display subsytem is different.Also the IP block in future OMAP may differ
> >> from the one existing in OMAP4. Thus to reuse the code between these two
> >> processors , and maintain the multi omap build functionality in DSS.
> >> HDMI IP dependant code is seperated out from hdmi.c and moved to
> >> new library file hdmi_ti_4xxx_ip.c.From the DSS dependent HDMI code only the
> >> function pointer to functionality/features offered by HDMI is called.
> >>
> >> This patch series does the split and also renames hdmi_omap4_panel.c to
> >> hdmi_panel.c as that file has nothing specific to OMAP4 and can be reused for
> >> other OMAP family of processors as well.
> >>
> >> This patch series is based on Tomi's LO-DSS2 master branch.
> >>
> >> changes since
> >> V2: Rename certain files/function to have standard format and handle scenario
> >> when hdmi is disabled.
> >> V1: Function pointer approach to call the HDMI IP functions from DSS HDMI
> >
> I guess i shall change the description , it is changes since , so v2
> would be actually changes since v2 to v3.
Ah, my mistake, I didn't read it carefully. I expected the change log to
be missing as there was only one change note, and there were other
things changed also in this version.
So I'll change my comment to:
It'd be nice to list most (if not all) of the changes made, so it's
easier for the reviewers to see that their comments were applied into
this version.
Tomi
next prev parent reply other threads:[~2011-09-05 10:41 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-09-02 10:47 [PATCH v3 00/10]HDMI: Split hdmi.c in DSS to seperate OMAP dependent mythripk
2011-09-02 10:47 ` [PATCH v3 01/10] OMAP4: DSS: HDMI: HDMI clean up to pass base_address mythripk
2011-09-02 10:47 ` [PATCH v3 02/10] OMAP4: DSS: HDMI: Move pll and video configuration mythripk
2011-09-02 10:47 ` [PATCH v3 03/10] OMAP4: DSS: HDMI: Use specific HDMI timings structure mythripk
2011-09-02 10:47 ` [PATCH v3 04/10] OMAP4: DSS: HDMI: Move IP independent common header mythripk
2011-09-02 10:47 ` [PATCH v3 05/10] OMAP4 : DSS : HDMI : Move the EDID portion from HDMI mythripk
2011-09-02 10:47 ` [PATCH v3 06/10] OMAP4: DSS2: HDMI: Split the HDMI driver to DSS and IP mythripk
2011-09-02 10:47 ` [PATCH v3 07/10] OMAP4: DSS: HDMI: Rename the functions in HDMI IP mythripk
2011-09-02 10:47 ` [PATCH v3 08/10] OMAP4: DSS: HDMI: Move the common HDMI header file mythripk
2011-09-02 10:47 ` [PATCH v3 09/10] OMAP4: DSS2: HDMI: Function pointer approach to call mythripk
2011-09-02 10:47 ` [PATCH v3 10/10] OMAP4: DSS: Rename hdmi_omap4_panel.c to hdmi_panel.c mythripk
2011-09-05 5:40 ` [PATCH v3 09/10] OMAP4: DSS2: HDMI: Function pointer approach to call Semwal, Sumit
2011-09-05 11:01 ` Tomi Valkeinen
2011-09-05 17:33 ` K, Mythri P
2011-09-06 6:08 ` K, Mythri P
2011-09-06 10:17 ` Tomi Valkeinen
2011-09-06 11:25 ` K, Mythri P
2011-09-06 12:45 ` Tomi Valkeinen
2011-09-06 13:05 ` K, Mythri P
2011-09-05 7:40 ` [PATCH v3 00/10]HDMI: Split hdmi.c in DSS to seperate OMAP dependent Tomi Valkeinen
2011-09-05 10:35 ` K, Mythri P
2011-09-05 10:41 ` Tomi Valkeinen [this message]
2011-09-05 12:15 ` Tomi Valkeinen
2011-09-05 13:22 ` K, Mythri P
2011-09-06 9:39 ` K, Mythri P
2011-09-06 11:28 ` K, Mythri P
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=1315219275.3214.21.camel@deskari \
--to=tomi.valkeinen@ti.com \
--cc=linux-omap@vger.kernel.org \
--cc=mythripk@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