On Fri, 06 May 2022, Jani Nikula <jani.nik...@intel.com> wrote:
> We'll need to propagate       drm_edid everywhere.

I seem to have copy-pasted a TAB in some of the commit messages, in a
way that does not show up in git log.

>
> Signed-off-by: Jani Nikula <jani.nik...@intel.com>
> ---
>  drivers/gpu/drm/drm_edid.c | 50 +++++++++++++++++++++++---------------
>  1 file changed, 31 insertions(+), 19 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
> index 946296632b2e..c9d48fbd0a76 100644
> --- a/drivers/gpu/drm/drm_edid.c
> +++ b/drivers/gpu/drm/drm_edid.c
> @@ -2419,13 +2419,13 @@ EXPORT_SYMBOL(drm_edid_duplicate);
>  
>  /**
>   * edid_get_quirks - return quirk flags for a given EDID
> - * @edid: EDID to process
> + * @drm_edid: EDID to process
>   *
>   * This tells subsequent routines what fixes they need to apply.
>   */
> -static u32 edid_get_quirks(const struct edid *edid)
> +static u32 edid_get_quirks(const struct drm_edid *drm_edid)
>  {
> -     u32 panel_id = edid_extract_panel_id(edid);
> +     u32 panel_id = edid_extract_panel_id(drm_edid->edid);
>       const struct edid_quirk *quirk;
>       int i;
>  
> @@ -5448,7 +5448,7 @@ static void drm_parse_microsoft_vsdb(struct 
> drm_connector *connector,
>  }
>  
>  static void drm_parse_cea_ext(struct drm_connector *connector,
> -                           const struct edid *edid)
> +                           const struct drm_edid *drm_edid)
>  {
>       struct drm_display_info *info = &connector->display_info;
>       struct drm_edid_iter edid_iter;
> @@ -5456,7 +5456,7 @@ static void drm_parse_cea_ext(struct drm_connector 
> *connector,
>       struct cea_db_iter iter;
>       const u8 *edid_ext;
>  
> -     drm_edid_iter_begin(edid, &edid_iter);
> +     drm_edid_iter_begin(drm_edid->edid, &edid_iter);
>       drm_edid_iter_for_each(edid_ext, &edid_iter) {
>               if (edid_ext[0] != CEA_EXT)
>                       continue;
> @@ -5477,7 +5477,7 @@ static void drm_parse_cea_ext(struct drm_connector 
> *connector,
>       }
>       drm_edid_iter_end(&edid_iter);
>  
> -     cea_db_iter_edid_begin(edid, &iter);
> +     cea_db_iter_edid_begin(drm_edid->edid, &iter);
>       cea_db_iter_for_each(db, &iter) {
>               /* FIXME: convert parsers to use struct cea_db */
>               const u8 *data = (const u8 *)db;
> @@ -5523,16 +5523,15 @@ void get_monitor_range(const struct detailed_timing 
> *timing,
>       monitor_range->max_vfreq = range->max_vfreq;
>  }
>  
> -static
> -void drm_get_monitor_range(struct drm_connector *connector,
> -                        const struct edid *edid)
> +static void drm_get_monitor_range(struct drm_connector *connector,
> +                               const struct drm_edid *drm_edid)
>  {
>       struct drm_display_info *info = &connector->display_info;
>  
> -     if (!version_greater(edid, 1, 1))
> +     if (!version_greater(drm_edid->edid, 1, 1))
>               return;
>  
> -     drm_for_each_detailed_block(edid, get_monitor_range,
> +     drm_for_each_detailed_block(drm_edid->edid, get_monitor_range,
>                                   &info->monitor_range);
>  
>       DRM_DEBUG_KMS("Supported Monitor Refresh rate range is %d Hz - %d Hz\n",
> @@ -5592,12 +5591,13 @@ static void drm_parse_vesa_mso_data(struct 
> drm_connector *connector,
>                   info->mso_stream_count, info->mso_pixel_overlap);
>  }
>  
> -static void drm_update_mso(struct drm_connector *connector, const struct 
> edid *edid)
> +static void drm_update_mso(struct drm_connector *connector,
> +                        const struct drm_edid *drm_edid)
>  {
>       const struct displayid_block *block;
>       struct displayid_iter iter;
>  
> -     displayid_iter_edid_begin(edid, &iter);
> +     displayid_iter_edid_begin(drm_edid->edid, &iter);
>       displayid_iter_for_each(block, &iter) {
>               if (block->tag == DATA_BLOCK_2_VENDOR_SPECIFIC)
>                       drm_parse_vesa_mso_data(connector, block);
> @@ -5636,18 +5636,20 @@ drm_reset_display_info(struct drm_connector 
> *connector)
>       info->mso_pixel_overlap = 0;
>  }
>  
> -u32 drm_add_display_info(struct drm_connector *connector, const struct edid 
> *edid)
> +static u32 update_display_info(struct drm_connector *connector,
> +                            const struct drm_edid *drm_edid)
>  {
>       struct drm_display_info *info = &connector->display_info;
> +     const struct edid *edid = drm_edid->edid;
>  
> -     u32 quirks = edid_get_quirks(edid);
> +     u32 quirks = edid_get_quirks(drm_edid);
>  
>       drm_reset_display_info(connector);
>  
>       info->width_mm = edid->width_cm * 10;
>       info->height_mm = edid->height_cm * 10;
>  
> -     drm_get_monitor_range(connector, edid);
> +     drm_get_monitor_range(connector, drm_edid);
>  
>       if (edid->revision < 3)
>               goto out;
> @@ -5656,7 +5658,7 @@ u32 drm_add_display_info(struct drm_connector 
> *connector, const struct edid *edi
>               goto out;
>  
>       info->color_formats |= DRM_COLOR_FORMAT_RGB444;
> -     drm_parse_cea_ext(connector, edid);
> +     drm_parse_cea_ext(connector, drm_edid);
>  
>       /*
>        * Digital sink with "DFP 1.x compliant TMDS" according to EDID 1.3?
> @@ -5709,7 +5711,7 @@ u32 drm_add_display_info(struct drm_connector 
> *connector, const struct edid *edi
>       if (edid->features & DRM_EDID_FEATURE_RGB_YCRCB422)
>               info->color_formats |= DRM_COLOR_FORMAT_YCBCR422;
>  
> -     drm_update_mso(connector, edid);
> +     drm_update_mso(connector, drm_edid);
>  
>  out:
>       if (quirks & EDID_QUIRK_NON_DESKTOP) {
> @@ -5721,6 +5723,16 @@ u32 drm_add_display_info(struct drm_connector 
> *connector, const struct edid *edi
>       return quirks;
>  }
>  
> +u32 drm_add_display_info(struct drm_connector *connector, const struct edid 
> *edid)
> +{
> +     struct drm_edid drm_edid = {
> +             .edid = edid,
> +             .size = edid_size(edid),
> +     };
> +
> +     return update_display_info(connector, &drm_edid);
> +}
> +
>  static struct drm_display_mode *drm_mode_displayid_detailed(struct 
> drm_device *dev,
>                                                           struct 
> displayid_detailed_timings_1 *timings,
>                                                           bool type_7)
> @@ -5833,7 +5845,7 @@ static int drm_edid_connector_update(struct 
> drm_connector *connector,
>        * To avoid multiple parsing of same block, lets parse that map
>        * from sink info, before parsing CEA modes.
>        */
> -     quirks = drm_add_display_info(connector, edid);
> +     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);

-- 
Jani Nikula, Intel Open Source Graphics Center

Reply via email to