<!DOCTYPE html>
<html>
<head>
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
</head>
<body>
<pre>> We support two kinds of analog connections:
>
> 1. VGA, which only supports analog signals:
> For VGA, we need to create a link encoder that only works with the
> DAC without perturbing any digital transmitter functionality.
> This is achieved by the new dce110_analog_link_encoder_construct.
>
> 2. DVI-I, which allows both digital and analog signals:
> The DC code base only allows 1 encoder per connector, and the
> preferred engine type is still going to be digital. So, for DVI-I
> to work, we need to make sure the pre-existing link encoder can
> also work with analog signals.
>
> Signed-off-by: Timur Kristóf <<a
href="https://lists.freedesktop.org/mailman/listinfo/amd-gfx"
data-bcup-haslogintext="no">timur.kristof at gmail.com</a>>
> ---
> .../drm/amd/display/dc/dce/dce_link_encoder.c | 100 ++++++++++++++++++
> .../drm/amd/display/dc/dce/dce_link_encoder.h | 21 ++--
I have the same comment about the use of "dce110_" prefix under general/global DCE code that I left on the previous patch.
For consistency with the current code, I understand why this prefix is used, but I'd gladly clean this up once the patches have landed in if there is a common agreement.
Alexandre</pre>
</body>
</html>