Re: [PATCH 2/2] OMAP:DSS: RFC Patch to add HDMI DSS driver support

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

 



K, Mythri P <mythripk@xxxxxx> [2010-09-08 10:16:29]:

Hi,

[snip]

> Yes this is just a RFC patch to introduce the HDMI driver and panel as such,
> if you have no other comments on these 2 patch set I shall incorporate these
> comments and send out the complete patch series with all the relevant
> changes in display.h and some overlay.c and manager.c changes.

there's EDID parser[1] already in kernel, so you might better reuse it and/or
extend it for your needs.  The next thing is, that there's and there will be
the demand for EDID on OMAP3 also, so it might be wise to rethink it from the
beginning and split or write the code in a way, that it can be reused on OMAP3
also. I think, that at least the code for calculation of ideal video
resolution and timings could be made generic and could be used on OMAP3 also.
Or am I mistaken?

I'm adding Ricardo Salveti to the CC, he showed me his first draft of EDID
parsing[2] for OMAP3 yesterday. Yes, it's just the draft, but it's at least
reusing the code already available in kernel. I think, that the same could or
should apply for the OMAP4 code.

What I'm trying to say is, that I'm not uber kernel hacker, but I would like
to have EDID working on my Beagle also and I'll definitely help with this.

Thanks.

1. drivers/video/fbmon.c and drivers/video/edid.h
2. http://kernel.ubuntu.com/git?p=rsalveti/ubuntu-maverick.git;a=commitdiff;h=a0ea712b64988ccd7984069b99864a1b4ac43881

-- ynezz
--
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