Re: [PATCH v3 00/10]HDMI: Split hdmi.c in DSS to seperate OMAP dependent

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxx> wrote:
> > On Fri, 2011-09-02 at 16:17 +0530, mythripk@xxxxxx wrote:
> >> From: Mythri P K <mythripk@xxxxxx>
> >>
> >> 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


--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux