On 24 May 2011 03:50, Ricardo Salveti <ricardo.salv...@linaro.org> wrote:
> On Mon, May 23, 2011 at 5:00 PM, Jaswinder Singh
> <jaswinder.si...@linaro.org> wrote:
>> On 24 May 2011 00:53, Ricardo Salveti <ricardo.salv...@linaro.org> wrote:
>>>>> Can you boot with omapdss.debug=1 drm.debug=7 and give me your kernel log?
>>>>
>>>> Please find the log attached.
>>>
>>> Interesting, the driver didn't find your monitor at all!
>>> [    5.738037] omapdss HDMI: Fail to detect the connector and force is
>>> enabled, trying at least one more time
>>> [    7.750610] [drm:drm_enable_connectors], connector 7 enabled? no
>>>
>>> Did you ever got your HDMI output to work with another kernel? Just to
>>> check if it's not a hardware issue.
>>
>> Yes. Maverick armel release worked and so does Natty's.  And Andy's TILT 
>> works.
>
> There is a new upload that will be available soon at
> https://launchpad.net/~linaro-maintainers/+archive/kernel that's a
> rebase from current npitre's tree, let's see if that will work better
> for you as Andy also got the same patches at his tree.

Ok I messed up the status. Sorry. Even Andy's TILT doesn't work as such.

My monitor seems to be of even extreme class of the 'LG monitors'
mentioned in the omapdss_hdmi_is_detected function.
IMHO the perfect fix should involve checking correct bits in correct
order (of which I know nothing atm), mine is more like a hack I admit.

Please have a look at the attached patch.

Cheers!
-j
From 5074ce2985eba858e4482a45d18b91800b0e85d6 Mon Sep 17 00:00:00 2001
From: Jassi Brar <jaswinder.si...@linaro.org>
Date: Tue, 31 May 2011 01:26:26 +0530
Subject: [PATCH] OMAP2: HDMI: Assume EDID read success as detection

Some class of hdmi displays never make the bit(2) of
HDMI_CORE_SYS_SYS_STAT set. EDID is read before calls to
is_detected callback, so if reading EDID succeeded that implies
we do have HDMI connected. In that case skip reading/waiting on
the second bit.

Signed-off-by: Jassi Brar <jaswinder.si...@linaro.org>
---
 drivers/video/omap2/dss/hdmi.c |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/drivers/video/omap2/dss/hdmi.c b/drivers/video/omap2/dss/hdmi.c
index a764662..dc809c4 100644
--- a/drivers/video/omap2/dss/hdmi.c
+++ b/drivers/video/omap2/dss/hdmi.c
@@ -1104,6 +1104,10 @@ bool omapdss_hdmi_is_detected(struct omap_dss_device *dssdev, bool force)
 {
 	u32 r;
 
+	/* If EDID has already been read, we have HDMI connected */
+	if (hdmi.edid_set)
+		return true;
+
 	r = hdmi_read_reg(HDMI_CORE_SYS_SYS_STAT);
 
 	/* Some annoying LG monitors will report that's disconnected
-- 
1.7.4.1

_______________________________________________
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev

Reply via email to