> > > > > Currently we just disable the GCP infoframe when turning off the port. > > > > > That means if the same transcoder is used on a DP port next, we might > > > > > end up pushing infoframes over DP, which isn't intended. Just disable > > > > > > > > Wonder how it is working. May be it is ok, or never hit using a > > > > previously > > > > used transcoder for HDMI port for DP. > > > > > > > > By the way, you mean end up pushing "other" infoframes over DP? > > > > > > We don't send infoframes over DP at all currently. Or I should say we > > > never intended to send them. After this patch that should even be true. > > > Well, unless the BIOS already enables them and we just fire up a DP port > > > using the transcoder in question. So I suppose we should really have the > > > DP code clear the infoframe settings explicitly, or we should clear them > > > during the sanitize phase. > > > > Agree that DP code path should clear the infoframe settings explicitly. > > As you are already touching this code, will you plan to handle it? > > I don't think I'll go there now. It would require quite a bit more work > to expose the infoframe stuff to the DP code. DP + infoframes in general > sounds like a good project for someone else to figure out. A quick > glance at the DP spec suggests that we should at least send the audio > infoframe.
OK, may be binned for future. _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx
