Message ID | 0997057511416dd83482468c6da9708f3ae157c8.1651830938.git.jani.nikula@intel.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | drm/edid: introduce struct drm_edid | expand |
On Fri, 06 May 2022, Jani Nikula <jani.nikula@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.nikula@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);
On 5/6/2022 3:46 PM, Jani Nikula wrote: > On Fri, 06 May 2022, Jani Nikula <jani.nikula@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.nikula@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 */ Perhaps the comment above needs to be updated with update_display_info() Patch looks good to me. Reviewed-by: Ankit Nautiyal <ankit.k.nautiyal@intel.com> >> drm_edid_to_eld(connector, edid);
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);
We'll need to propagate drm_edid everywhere. Signed-off-by: Jani Nikula <jani.nikula@intel.com> --- drivers/gpu/drm/drm_edid.c | 50 +++++++++++++++++++++++--------------- 1 file changed, 31 insertions(+), 19 deletions(-)