Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Thu, 30 Mar 2023 18:15:06 +0200
From:   Andreas Kemnade <andreas@kemnade.info>
To:     Tony Lindgren <tony@atomide.com>
Cc:     robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-omap@vger.kernel.org, Andrew Davis <afd@ti.com>
Subject: Re: [PATCH v2] dt-bindings: omap: Convert omap.txt to yaml
Message-ID: <20230330181506.4d5fcf51@aktux>
In-Reply-To: <20230330113918.GS7501@atomide.com>
References: <20230329222246.3292766-1-andreas@kemnade.info>
        <20230330113918.GS7501@atomide.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On Thu, 30 Mar 2023 14:39:18 +0300
Tony Lindgren <tony@atomide.com> wrote:

> * Andreas Kemnade <andreas@kemnade.info> [230329 22:22]:
> > From: Andrew Davis <afd@ti.com>
> >=20
> > Convert omap.txt to yaml.
> >  Documentation/devicetree/bindings/arm/ti.yaml | 157 ++++++++++++++++++
> >  1 file changed, 157 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/arm/ti.yaml =20
>=20
> Great, can we also drop the old txt file or is more changes needed before
> we can do that?
>
we have still=20
- ti,hwmods: list of hwmod names (ascii strings), that comes from the OMAP
  HW documentation, attached to a device. Must contain at least
  one hwmod.

Optional properties:
- ti,no_idle_on_suspend: When present, it prevents the PM to idle the module
  during suspend.
- ti,no-reset-on-init: When present, the module should not be reset at init
- ti,no-idle-on-init: When present, the module should not be idled at init
- ti,no-idle: When present, the module is never allowed to idle.

These optional properties are not in the root node but in subnodes.
=46rom my guts feeling this belongs in a separate file and should
be somehow dealed with in a second step.

So how to proceed?

BTW: I think this file then also belongs into
OMAP2+ SUPPORT section of MAINTAINERS

Regards,
Andreas