On Tue, Mar 01, 2022 at 01:01:41PM +0800, Lee Shawn C wrote:
> VBT 249 update to support more TMDS clock rate 3.00G, 3.40G
> and 5.94G. Refer to this new definition to configure max
> TMDS clock rate for HDMI driver.
The patch itself looks fine. The patch subject is pretty much
incomprehensible to me.
>
> BSpec: 20124
>
> Cc: Jani Nikula <jani.nikula@xxxxxxxxxxxxxxx>
> Cc: Ville Syrjala <ville.syrjala@xxxxxxxxxxxxxxx>
> Cc: Ankit Nautiyal <ankit.k.nautiyal@xxxxxxxxx>
> Signed-off-by: Lee Shawn C <shawn.c.lee@xxxxxxxxx>
> ---
> drivers/gpu/drm/i915/display/intel_bios.c | 6 ++++++
> drivers/gpu/drm/i915/display/intel_vbt_defs.h | 3 +++
> 2 files changed, 9 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_bios.c b/drivers/gpu/drm/i915/display/intel_bios.c
> index 40b5e7ed12c2..a559a1914588 100644
> --- a/drivers/gpu/drm/i915/display/intel_bios.c
> +++ b/drivers/gpu/drm/i915/display/intel_bios.c
> @@ -1955,6 +1955,12 @@ static int _intel_bios_max_tmds_clock(const struct intel_bios_encoder_data *devd
> fallthrough;
> case HDMI_MAX_DATA_RATE_PLATFORM:
> return 0;
> + case HDMI_MAX_DATA_RATE_594:
> + return 594000;
> + case HDMI_MAX_DATA_RATE_340:
> + return 340000;
> + case HDMI_MAX_DATA_RATE_300:
> + return 300000;
> case HDMI_MAX_DATA_RATE_297:
> return 297000;
> case HDMI_MAX_DATA_RATE_165:
> diff --git a/drivers/gpu/drm/i915/display/intel_vbt_defs.h b/drivers/gpu/drm/i915/display/intel_vbt_defs.h
> index b9397d9363c5..e0508990df48 100644
> --- a/drivers/gpu/drm/i915/display/intel_vbt_defs.h
> +++ b/drivers/gpu/drm/i915/display/intel_vbt_defs.h
> @@ -289,6 +289,9 @@ struct bdb_general_features {
> #define HDMI_MAX_DATA_RATE_PLATFORM 0 /* 204 */
> #define HDMI_MAX_DATA_RATE_297 1 /* 204 */
> #define HDMI_MAX_DATA_RATE_165 2 /* 204 */
> +#define HDMI_MAX_DATA_RATE_594 3 /* 249 */
> +#define HDMI_MAX_DATA_RATE_340 4 /* 249 */
> +#define HDMI_MAX_DATA_RATE_300 5 /* 249 */
>
> #define LEGACY_CHILD_DEVICE_CONFIG_SIZE 33
>
> --
> 2.17.1
--
Ville Syrjälä
Intel
