On Tue, 2012-03-20 at 18:45 +0530, mythripk@xxxxxx wrote: > From: Mythri P K <mythripk@xxxxxx> > > Add support for handling the DSS_HDMI interrupt in HDMI, A line that serves to > notify HDMI of status change in PHY, PLL and CORE based on the registration. > Also logic to support enabling of the PHY in TX_ON state only when a PHY_CONNECT > which would make sure that TMDS lines are high before putting it in TX_ON state. > > Mythri P K (3): > OMAPDSS: HDMI: support for interrupt enabling > OMAPDSS: HDMI: Add support for DSS_HDMI Interrupt > OMAPDSS: HDMI: wait for TMDS to be high before putting phy in TX_ON Sorry, but this patch set is quite bad. First of all, the subject on your third patch is again cut short. How do you even do that? The subject seems to be fine above. I've never seen other people do that, but you somehow manage to do it all the time. So, please, read the git manual, fix your environment, your way of working, or whatever is causing this so that we'll never see it again. Then, the intro text above is rather confusing. Are there words missing around the part with "PHY_CONNECT"? Please spend more time on the descriptions, and possibly ask someone to proof-read them so that at least there aren't words missing and they are somehow understandable. I'll comment on other things in later mails. Tomi
Attachment:
signature.asc
Description: This is a digitally signed message part
