On Fri, 03 May 2024, Ville Syrjala <ville.syrjala@xxxxxxxxxxxxxxx> wrote: > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > Define the contents of VBT block 10 (Mode Removal Table). > > There seem to be two variants: > - 8 byte entries for desktop systems > - 10 byte entries for mobile systems, with the extra > panel_flags being a bitmask of LFPs > > It seems starting from HSW only the mobile variant is > used anymore. > > Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> Awful stuff. Reviewed-by: Jani Nikula <jani.nikula@xxxxxxxxx> > --- > drivers/gpu/drm/i915/display/intel_vbt_defs.h | 23 +++++++++++++++++++ > 1 file changed, 23 insertions(+) > > diff --git a/drivers/gpu/drm/i915/display/intel_vbt_defs.h b/drivers/gpu/drm/i915/display/intel_vbt_defs.h > index 4f0b8be3034c..d78523cd4214 100644 > --- a/drivers/gpu/drm/i915/display/intel_vbt_defs.h > +++ b/drivers/gpu/drm/i915/display/intel_vbt_defs.h > @@ -677,6 +677,29 @@ struct bdb_psr { > u32 psr2_tp2_tp3_wakeup_time; /* 226+ */ > } __packed; > > +/* > + * Block 10 - Mode Removal Table > + */ > + > +struct mode_removal_table { > + u16 x_res; > + u16 y_res; > + u8 bpp; > + u16 refresh_rate; > + u8 removal_flags; > + u16 panel_flags; > +} __packed; > + > +struct bdb_mode_removal { > + u8 row_size; /* 8 or 10 bytes */ > + /* > + * VBT spec says this is always 20 entries, > + * but ALM seems to have only 15 entries. > + */ > + struct mode_removal_table modes[]; > + /* u16 terminator; 0x0000 */ > +} __packed; > + > /* > * Block 12 - Driver Features Data Block > */ -- Jani Nikula, Intel