[PATCH 11/20] drm/amd/display: Implement DCE analog link encoders

Timur Kristóf timur.kristof at gmail.com
Sun Aug 3 20:08:04 UTC 2025


On Sun, 2025-08-03 at 12:26 -0400, Alexandre Demers wrote:
> On Fri, Aug 1, 2025 at 6:02 PM Timur Kristóf
> <timur.kristof at gmail.com> wrote:
> > 
> > Hi Alexandre,
> > 
> > With regards to the coding style. I already replied to your other
> > thread about it, let's have that conversation there.
> > 
> > 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.
> > 
> > What do you think about that?
> > 
> > Thanks,
> > Timur
> > 
> > 
> > 
> 
> 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.
> 
> Alexandre

Well, that's the issue. The distinction isn't really there:

- dce110_link_encoder does both analog and digital for DVI-I
- dce110_analog_link_encoder can only do analog for VGA

At this point I feel that adding the dce110_analog_link_encoder just
creates unnecessary confusion.

Since dce110_link_encoder does both analog and digital already, it will
make more sense to have just dce110_link_encoder. I'll make that change
in v2 of this series.

Timur


More information about the amd-gfx mailing list