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. For the entire series: Reviewed-by: Maxime Ripard <mrip...@kernel.org> Maxime
signature.asc
Description: PGP signature
