[Intel-gfx] [PATCH v2 06/25] drm/edid: propagate drm_edid to drm_edid_to_eld()

[Nautiyal, Ankit K]

LGTM.

Reviewed-by: Ankit Nautiyal <ankit.k.nautiyal at intel.com>

Regards,

Ankit

On 5/9/2022 5:33 PM, Jani Nikula wrote:
> We'll need to propagate drm_edid everywhere.
>
> Signed-off-by: Jani Nikula <jani.nikula at intel.com>
> ---
>   drivers/gpu/drm/drm_edid.c | 20 ++++++++++----------
>   1 file changed, 10 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
> index 96e3f9327044..e4fdf742645b 100644
> --- a/drivers/gpu/drm/drm_edid.c
> +++ b/drivers/gpu/drm/drm_edid.c
> @@ -4883,13 +4883,13 @@ static void clear_eld(struct drm_connector *connector)
>   /*
>    * drm_edid_to_eld - build ELD from EDID
>    * @connector: connector corresponding to the HDMI/DP sink
> - * @edid: EDID to parse
> + * @drm_edid: EDID to parse
>    *
>    * Fill the ELD (EDID-Like Data) buffer for passing to the audio driver. The
>    * HDCP and Port_ID ELD fields are left for the graphics driver to fill in.
>    */
>   static void drm_edid_to_eld(struct drm_connector *connector,
> -			    const struct edid *edid)
> +			    const struct drm_edid *drm_edid)
>   {
>   	const struct drm_display_info *info = &connector->display_info;
>   	const struct cea_db *db;
> @@ -4900,10 +4900,10 @@ static void drm_edid_to_eld(struct drm_connector *connector,
>   
>   	clear_eld(connector);
>   
> -	if (!edid)
> +	if (!drm_edid)
>   		return;
>   
> -	mnl = get_monitor_name(edid, &eld[DRM_ELD_MONITOR_NAME_STRING]);
> +	mnl = get_monitor_name(drm_edid->edid, &eld[DRM_ELD_MONITOR_NAME_STRING]);
>   	DRM_DEBUG_KMS("ELD monitor %s\n", &eld[DRM_ELD_MONITOR_NAME_STRING]);
>   
>   	eld[DRM_ELD_CEA_EDID_VER_MNL] = info->cea_rev << DRM_ELD_CEA_EDID_VER_SHIFT;
> @@ -4911,12 +4911,12 @@ static void drm_edid_to_eld(struct drm_connector *connector,
>   
>   	eld[DRM_ELD_VER] = DRM_ELD_VER_CEA861D;
>   
> -	eld[DRM_ELD_MANUFACTURER_NAME0] = edid->mfg_id[0];
> -	eld[DRM_ELD_MANUFACTURER_NAME1] = edid->mfg_id[1];
> -	eld[DRM_ELD_PRODUCT_CODE0] = edid->prod_code[0];
> -	eld[DRM_ELD_PRODUCT_CODE1] = edid->prod_code[1];
> +	eld[DRM_ELD_MANUFACTURER_NAME0] = drm_edid->edid->mfg_id[0];
> +	eld[DRM_ELD_MANUFACTURER_NAME1] = drm_edid->edid->mfg_id[1];
> +	eld[DRM_ELD_PRODUCT_CODE0] = drm_edid->edid->prod_code[0];
> +	eld[DRM_ELD_PRODUCT_CODE1] = drm_edid->edid->prod_code[1];
>   
> -	cea_db_iter_edid_begin(edid, &iter);
> +	cea_db_iter_edid_begin(drm_edid->edid, &iter);
>   	cea_db_iter_for_each(db, &iter) {
>   		const u8 *data = cea_db_data(db);
>   		int len = cea_db_payload_len(db);
> @@ -5864,7 +5864,7 @@ static int drm_edid_connector_update(struct drm_connector *connector,
>   	quirks = update_display_info(connector, drm_edid);
>   
>   	/* Depends on info->cea_rev set by drm_add_display_info() above */
> -	drm_edid_to_eld(connector, edid);
> +	drm_edid_to_eld(connector, drm_edid);
>   
>   	/*
>   	 * EDID spec says modes should be preferred in this order: