On Tue, Mar 12, 2024 at 03:41:32PM +0100, Geert Uytterhoeven wrote:
> On Wed, Feb 21, 2024 at 3:06 PM Rob Herring <robh+dt@xxxxxxxxxx> wrote:
> static int __init regulator_init_complete(void)
> {
> /*
> * Since DT doesn't provide an idiomatic mechanism for
> * enabling full constraints and since it's much more natural
> * with DT to provide them just assume that a DT enabled
> * system has full constraints.
> */
> if (of_have_populated_dt())
> has_full_constraints = true;
>
> ...
> }
> late_initcall_sync(regulator_init_complete);
> (The latter looks a bit futile, as have_full_constraints() already
> contains a check for of_have_populated_dt()?)
Yes, it's duplicated and we should remove it.
Attachment:
signature.asc
Description: PGP signature
