Messages in this thread |  | | From | Jani Nikula <> | | Subject | Re: [PATCH v6 5/7] drm/edid: for consistency, use mask everywhere for block rev parsing | | Date | Wed, 26 Nov 2025 16:20:14 +0200 |
| |
On Wed, 26 Nov 2025, Yaroslav Bolyukin <iam@lach.pw> wrote:
Commit message goes here.
> Signed-off-by: Yaroslav Bolyukin <iam@lach.pw> > --- > drivers/gpu/drm/drm_displayid_internal.h | 1 + > drivers/gpu/drm/drm_edid.c | 3 ++- > 2 files changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/drm_displayid_internal.h b/drivers/gpu/drm/drm_displayid_internal.h > index 724174b429f2..55f972d32847 100644 > --- a/drivers/gpu/drm/drm_displayid_internal.h > +++ b/drivers/gpu/drm/drm_displayid_internal.h > @@ -139,6 +139,7 @@ struct displayid_formula_timings_9 { > u8 vrefresh; > } __packed; > > +#define DISPLAYID_BLOCK_DESCRIPTOR_PAYLOAD_BYTES GENMASK(6, 4) > struct displayid_formula_timing_block { > struct displayid_block base; > struct displayid_formula_timings_9 timings[]; > diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c > index 72a94b1713e2..7bdc99d5084a 100644 > --- a/drivers/gpu/drm/drm_edid.c > +++ b/drivers/gpu/drm/drm_edid.c > @@ -6906,7 +6906,8 @@ static int add_displayid_formula_modes(struct drm_connector *connector, > struct drm_display_mode *newmode; > int num_modes = 0; > bool type_10 = block->tag == DATA_BLOCK_2_TYPE_10_FORMULA_TIMING; > - int timing_size = 6 + ((formula_block->base.rev & 0x70) >> 4); > + int timing_size = 6 + > + FIELD_GET(DISPLAYID_BLOCK_DESCRIPTOR_PAYLOAD_BYTES, formula_block->base.rev); > > /* extended blocks are not supported yet */ > if (timing_size != 6)
-- Jani Nikula, Intel
|  |