Re: [PATCH] clang-format: add for_each macros from libfdt

From: Bill Wendling
Date: Fri Mar 21 2025 - 18:25:08 EST


On Fri, Mar 21, 2025 at 3:05 PM Changyuan Lyu <changyuanl@xxxxxxxxxx> wrote:
>
> Hi Bill,
>
> On Fri, Mar 21, 2025 at 12:28:44 -0700, Bill Wendling <morbo@xxxxxxxxxx> wrote:
> > On Thu, Mar 20, 2025 at 4:59=E2=80=AFPM Changyuan Lyu <changyuanl@xxxxxxxxx=
> > m> wrote:
> > >
> > > Add two for_each macros from scripts/dtc/libfdt/libfdt.h.
> > >
> > > Signed-off-by: Changyuan Lyu <changyuanl@xxxxxxxxxx>
> > > ---
> > > .clang-format | 2 ++
> > > 1 file changed, 2 insertions(+)
> > >
> > > diff --git a/.clang-format b/.clang-format
> > > index fe1aa1a30d40..4decda7474e1 100644
> > > --- a/.clang-format
> > > +++ b/.clang-format
> > > @@ -210,6 +210,8 @@ ForEachMacros:
> > > - 'evlist__for_each_entry_from'
> > > - 'evlist__for_each_entry_reverse'
> > > - 'evlist__for_each_entry_safe'
> > > + - 'fdt_for_each_property_offset'
> > > + - 'fdt_for_each_subnode'
> >
> > Could you alphabetize these please?
> >
> > > - 'flow_action_for_each'
> > > - 'for_each_acpi_consumer_dev'
> > > - 'for_each_acpi_dev_match'
> > > --
> > > 2.49.0.395.g12beb8f557-goog
> > >
>
> Thanks for taking a look, but I think the list `ForEachMacros`
> is already sorted alphabetically.
> Am I missing anything?
>
Oh! I'm sorry. I tought that the 'd' (dee) in 'fdt_...' was an 'o' (oh). My bad.

-bw