<div dir="ltr"><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Fri, Aug 1, 2025 at 6:02 PM Timur Kristóf <<a href="mailto:timur.kristof@gmail.com">timur.kristof@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br><br>2025. augusztus 1., péntek dátummal Alexandre Demers <<a href="mailto:alexandre.f.demers@gmail.com" target="_blank">alexandre.f.demers@gmail.com</a>> ezt írta:<br>>> We support two kinds of analog connections:<br>>><br>>> 1. VGA, which only supports analog signals:<br>>> For VGA, we need to create a link encoder that only works with the<br>>> DAC without perturbing any digital transmitter functionality.<br>>> This is achieved by the new dce110_analog_link_encoder_construct.<br>>><br>>> 2. DVI-I, which allows both digital and analog signals:<br>>> The DC code base only allows 1 encoder per connector, and the<br>>> preferred engine type is still going to be digital. So, for DVI-I<br>>> to work, we need to make sure the pre-existing link encoder can<br>>> also work with analog signals.<br>>><br>>> Signed-off-by: Timur Kristóf <timur.kristof at <a href="http://gmail.com" target="_blank">gmail.com</a>><br>>> ---<br>>> .../drm/amd/display/dc/dce/dce_link_encoder.c | 100 ++++++++++++++++++<br>>> .../drm/amd/display/dc/dce/dce_link_encoder.h | 21 ++--<br>><br>> I have the same comment about the use of "dce110_" prefix under general/global DCE code that I left on the previous patch.<br>><br>> 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.<br>><br>> Alexandre<br><br>Hi Alexandre,<br><br>With regards to the coding style. I already replied to your other thread about it, let's have that conversation there.<br><br>With regards to the link encoders specifically. Due to DVI-I, we need dce110_link_encoder to handle analog signals in addition to digital, so the question about this part is, is there any need to have a separate dce110_analog_link_encoder? When I wrote the patch I felt yes, but now I feel maybe we should just let dce110_link_encoder handle VGA as well.<br><br>What do you think about that?<br><br>Thanks,<br>Timur<br><br><br></blockquote><div>Since a distinction is already made in the code between digital and analog encoders, I would be tempted to go with the dce110_analog_link_encoder so it may be just easier/quicker to distinguish what this part of code does.</div><div><br></div><div>Alexandre</div></div></div>