On Mon, 15 Sep 2025 14:22:24 +0200
Maxime Ripard <mrip...@kernel.org> wrote:
> Hi,
>
> On Fri, Aug 08, 2025 at 04:49:14PM +0200, Luca Ceresoli wrote:
> > All users have been replaced by drm_for_each_bridge_in_chain_scoped().
> >
> > Signed-off-by: Luca Ceresoli <luca.ceres...@bootlin.com>
> > ---
> > .clang-format | 1 -
> > include/drm/drm_bridge.h | 14 --------------
> > 2 files changed, 15 deletions(-)
> >
> > diff --git a/.clang-format b/.clang-format
> > index
> > 1cac7d4976644c8f083f801e98f619782c2e23cc..d5c05db1a0d96476b711b95912d2b82b2e780397
> > 100644
> > --- a/.clang-format
> > +++ b/.clang-format
> > @@ -167,7 +167,6 @@ ForEachMacros:
> > - 'drm_connector_for_each_possible_encoder'
> > - 'drm_exec_for_each_locked_object'
> > - 'drm_exec_for_each_locked_object_reverse'
> > - - 'drm_for_each_bridge_in_chain'
> > - 'drm_for_each_bridge_in_chain_scoped'
> > - 'drm_for_each_connector_iter'
> > - 'drm_for_each_crtc'
> > diff --git a/include/drm/drm_bridge.h b/include/drm/drm_bridge.h
> > index
> > a8e2f599aea764c705da3582df0ca428bb32f19c..6adf9221c2d462ec8e0e4e281c97b39081b3da24
> > 100644
> > --- a/include/drm/drm_bridge.h
> > +++ b/include/drm/drm_bridge.h
> > @@ -1358,20 +1358,6 @@ drm_bridge_chain_get_first_bridge(struct drm_encoder
> > *encoder)
> > struct drm_bridge,
> > chain_node));
> > }
> >
> > -/**
> > - * drm_for_each_bridge_in_chain() - Iterate over all bridges present in a
> > chain
> > - * @encoder: the encoder to iterate bridges on
> > - * @bridge: a bridge pointer updated to point to the current bridge at each
> > - * iteration
> > - *
> > - * Iterate over all bridges present in the bridge chain attached to
> > @encoder.
> > - *
> > - * This is deprecated, do not use!
> > - * New drivers shall use drm_for_each_bridge_in_chain_scoped().
> > - */
> > -#define drm_for_each_bridge_in_chain(encoder, bridge)
> > \
> > - list_for_each_entry(bridge, &(encoder)->bridge_chain, chain_node)
> > -
>
> I think I'd go a step further and rename
> drm_for_each_bridge_in_chain_scoped to drm_for_each_bridge_in_chain,
> there's no need to have a "scoped" variant if it's our only variant.
>
> It can be done in a subsequent patch though.
Sure, that's the plan. There's a note in patch 3:
Note 1: drm_for_each_bridge_in_chain_scoped() could be renamed removing the
_scoped suffix after removing all the users of the current macro
and eventually the current macro itself. Even though this series is
converting all users, I'd at least wait one kernel release before
renaming, to minimize issues with existing patches which would fail
building.
> For the entire series:
> Reviewed-by: Maxime Ripard <mrip...@kernel.org>
Great, thanks!
I'll wait a few more days before applying.
Luca
--
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
