On Wed, 2026-09-09 at 14:19 +0300, Imre Deak wrote:
> On Tue, Sep 08, 2026 at 01:06:51PM +0300, Luca Coelho wrote:
> > The calc_allowed_config_filter() function passes the return value of
> > iter_pos_to_idx() directly to BIT(), but the helper can return -1 for
> > an invalid iterator.
> >
> > The iterator already rejects negative indices before doing a
> > configuration, so this should not matter in normal flows. In any
> > case, for robustness, check the index explicitly and warn if it is
> > negative, avoiding an undefined shift.
> >
> > Fixes: 39e30bdf2f92 ("drm/i915/dp_link_caps: Add link configuration
> > iterator")
> > Signed-off-by: Luca Coelho <[email protected]>
>
> Reviewed-by: Imre Deak <[email protected]>
Thanks! This is now merged to drm-intel-next.
--
Cheers,
Luca.
>
