Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
In-Reply-To: <20220324071019.59483-1-jakobkoschel@gmail.com>
References: <20220324071019.59483-1-jakobkoschel@gmail.com>
Subject: Re: [PATCH] clk: ti: clkctrl: replace usage of found with dedicated list iterator variable
From:   Stephen Boyd <sboyd@kernel.org>
Cc:     Michael Turquette <mturquette@baylibre.com>,
        linux-omap@vger.kernel.org, linux-clk@vger.kernel.org,
        linux-kernel@vger.kernel.org, Mike Rapoport <rppt@kernel.org>,
        Brian Johannesmeyer <bjohannesmeyer@gmail.com>,
        Cristiano Giuffrida <c.giuffrida@vu.nl>,
        "Bos, H.J." <h.j.bos@vu.nl>, Jakob Koschel <jakobkoschel@gmail.com>
To:     Jakob Koschel <jakobkoschel@gmail.com>,
        Tero Kristo <kristo@kernel.org>
Date:   Fri, 20 May 2022 20:41:13 -0700
User-Agent: alot/0.10
Message-Id: <20220521034114.E5C49C385A5@smtp.kernel.org>
Precedence: bulk

Quoting Jakob Koschel (2022-03-24 00:10:19)
> To move the list iterator variable into the list_for_each_entry_*()
> macro in the future it should be avoided to use the list iterator
> variable after the loop body.
>=20
> To *never* use the list iterator variable after the loop it was
> concluded to use a separate iterator variable instead of a
> found boolean [1].
>=20
> This removes the need to use a found variable and simply checking if
> the variable was set, can determine if the break/goto was hit.
>=20
> Link: https://lore.kernel.org/all/CAHk-=3DwgRr_D8CB-D9Kg-c=3DEHreAsk5SqXP=
wr9Y7k9sA6cWXJ6w@mail.gmail.com/
> Signed-off-by: Jakob Koschel <jakobkoschel@gmail.com>
> ---

Applied to clk-next