This is actually setting DIG mode a second time, right? I don't think this is what sets GC_SEND.
Please mention that this is setting the DIG_MODE to the correct value after having been overridden by the call to transmitter control in your patch description. Also correct the HACK comment to mention that this second call is needed to reconfigure the DIG
as a workaround for the incorrect value being applied from transmitter control. Specifics help in source.
I don't think there is a case where we'd want HDMI changed to DVI so it's probably fine to leave this as is for now...
With those fixed, you can add my:
Reviewed-by: Nicholas Kazlauskas <nicholas.kazlauskas@xxxxxxx>
Thanks,
Nicholas Kazlauskas
_______________________________________________ amd-gfx mailing list amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/amd-gfx