On Thu, 2012-06-28 at 16:28 +0530, Jassi Brar wrote: > > I think among the first things, while enabling HDMI, should be to see > > if there is really some display connected on the port i.e, HPD > > asserted. Only if ti_hdmi_4_detect() returned true, should we > > proceed otherwise wait for HPQ irq. > > > > Unconditionally invalidating edid really seems like a regression - we > > impose atleast 50ms (edid read) as extra cost on > > hdmi_check_hpd_state(), which kills half the purpose of this patch. > > > Sorry a correction. Reading detect() won't work. I suggest we keep HPD > IRQ enabled for the lifetime of the driver. By the way, when the device is in system suspend, we surely won't detect the HPD even if we kept the HPD always enabled. So there we'll miss the HPD interrupt anyway, and the EDID cache would be invalid. Tomi
Attachment:
signature.asc
Description: This is a digitally signed message part
