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:   Fri, 31 Mar 2023 14:37:14 +0300
From:   Tony Lindgren <tony@atomide.com>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc:     Andreas Kemnade <andreas@kemnade.info>, 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: <20230331113714.GV7501@atomide.com>
References: <20230329222246.3292766-1-andreas@kemnade.info>
 <20230330113918.GS7501@atomide.com>
 <20230330181506.4d5fcf51@aktux>
 <191c96d2-e51a-550e-16d9-0403e4528885@linaro.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <191c96d2-e51a-550e-16d9-0403e4528885@linaro.org>
Precedence: bulk

* Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> [230331 09:43]:
> On 30/03/2023 18:15, Andreas Kemnade wrote:
> > 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>
> >>>
> >>> 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  
> >>
> >> Great, can we also drop the old txt file or is more changes needed before
> >> we can do that?
> >>
> > we have still 
> > - 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.
> > From my guts feeling this belongs in a separate file and should
> > be somehow dealed with in a second step.
> > 
> > So how to proceed?
> 
> Any compatible converted to DT Schema should be removed from TXT. It
> does not mean entire TXT has to be removed if it contains other pieces.

Sounds good to me.

Tony