Re: [PATCH 3/7] drm/bridge: Extend struct drm_bus_cfg with clock field

[Maxime Ripard]

Hi,

On Sat, Feb 19, 2022 at 01:28:40AM +0100, Marek Vasut wrote:
> diff --git a/include/drm/drm_atomic.h b/include/drm/drm_atomic.h
> index 1701c2128a5cb..32455cf28f0bc 100644
> --- a/include/drm/drm_atomic.h
> +++ b/include/drm/drm_atomic.h
> @@ -1077,6 +1077,11 @@ struct drm_bus_cfg {
>        * @flags: DRM_BUS_* flags used on this bus
>        */
>       u32 flags;
> +
> +     /**
> +      * @clock: Clock frequency in kHz used on this bus
> +      */
> +     u32 clock;
>  };

This is fairly vague. You were mentioning DSI: is it the pixel clock?
The HS clock rate? With or without counting the lanes? What about the
burst mode: would it be the lane or pixel rate?

It would be just as confusing for HDMI: is it the the TMDS character
rate? The TMDS bit rate ? TMDS Clock rate?

Maxime

signature.asc
Description: PGP signature