Hi Ulrich, On 05/13/2016 10:36 AM, Hans Verkuil wrote: > On 05/11/2016 04:02 PM, Ulrich Hecht wrote: >> Initializes the decoder subdevice with a fixed EDID blob. >> >> Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@xxxxxxxxx> > > Nacked-by: Hans Verkuil <hans.verkuil@xxxxxxxxx> > > Instead implement the g/s_edid ioctls. > > You truly cannot default to an EDID. When an EDID is set the HPD will go high. > But you don't know the EDID here, the contents of the EDID is something that > only userspace will know depending on the type of device you're building. > > In practice userspace will overwrite the EDID with the real one and so the HPD > will go down and up again. And while transmitters are supposed to handle that > cleanly, in reality this is a different story. > > Just add the g/s_edid ioctls and you can use 'v4l2-ctl --set-edid=edid=hdmi' to > fill in a default EDID. > > I won't accept this patch since I know from my own experience that this doesn't > work. I haven't seen a follow-up on this. Can you do a v5? It's likely that will be the last version and I can commit this. Thanks! Hans
