On Mon, 2025-09-01 at 13:21 +0300, Jani Nikula wrote: > On Mon, 01 Sep 2025, Jouni Högander <jouni.hogan...@intel.com> wrote: > > PSR idle frames in VBT binary is a 4 bits wide bitfield. Checking > > if it's > > below 0 or over 15 doesn't make sense. Remove these checks. > > > > Signed-off-by: Jouni Högander <jouni.hogan...@intel.com> > > Reviewed-by: Jani Nikula <jani.nik...@intel.com>
Thank you for checking my patch. This is now pushed to drm-intel-next. BR, Jouni Högander > > > --- > > drivers/gpu/drm/i915/display/intel_bios.c | 5 +---- > > 1 file changed, 1 insertion(+), 4 deletions(-) > > > > diff --git a/drivers/gpu/drm/i915/display/intel_bios.c > > b/drivers/gpu/drm/i915/display/intel_bios.c > > index 7d7193a05d88..3596dce84c28 100644 > > --- a/drivers/gpu/drm/i915/display/intel_bios.c > > +++ b/drivers/gpu/drm/i915/display/intel_bios.c > > @@ -1567,10 +1567,7 @@ parse_psr(struct intel_display *display, > > > > panel->vbt.psr.full_link = psr_table->full_link; > > panel->vbt.psr.require_aux_wakeup = psr_table- > > >require_aux_to_wakeup; > > - > > - /* Allowed VBT values goes from 0 to 15 */ > > - panel->vbt.psr.idle_frames = psr_table->idle_frames < 0 ? > > 0 : > > - psr_table->idle_frames > 15 ? 15 : psr_table- > > >idle_frames; > > + panel->vbt.psr.idle_frames = psr_table->idle_frames; > > > > /* > > * New psr options 0=500us, 1=100us, 2=2500us, 3=0us >
