On Fri, 17 Oct 2025, Gustavo Sousa <[email protected]> wrote:
> Quoting Jani Nikula (2025-10-15 12:29:00-03:00)
>>We probably want to add the info to print_ddi_port().
>
> Yep.  Good idea.
>
> I'm currently looking at print_ddi_port() and the one-liner is already
> quite long and I don't know we would be able to come up with a good
> abbreviation to put there.

Yeah, it is...

>
> Probably just print on its own line?
> E.g.:
>
>       dedicated_external = intel_bios_encoder_is_dedicated_external(devdata);
>       if (dedicated_external)
>               drm_dbg_kms(display->drm,
>                           "Port %c is dedicated external\n");

Ack.

> Do you think printing for dyn_port_over_tc is also useful?

We'll only know when we're debugging some issue! ;)

BR,
Jani.


-- 
Jani Nikula, Intel

Reply via email to