Re: [PATCH v2] dt-bindings: omap: Convert omap.txt to yaml

From: Tony Lindgren
Date: Fri Mar 31 2023 - 07:37:21 EST


* Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> [230331 09:43]:
> On 30/03/2023 18:15, Andreas Kemnade wrote:
> > On Thu, 30 Mar 2023 14:39:18 +0300
> > Tony Lindgren <tony@xxxxxxxxxxx> wrote:
> >
> >> * Andreas Kemnade <andreas@xxxxxxxxxxxx> [230329 22:22]:
> >>> From: Andrew Davis <afd@xxxxxx>
> >>>
> >>> 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