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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <191c96d2-e51a-550e-16d9-0403e4528885@linaro.org>
Date:   Fri, 31 Mar 2023 11:43:41 +0200
From:   Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To:     Andreas Kemnade <andreas@...nade.info>,
        Tony Lindgren <tony@...mide.com>
Cc:     robh+dt@...nel.org, krzysztof.kozlowski+dt@...aro.org,
        devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
        linux-omap@...r.kernel.org, Andrew Davis <afd@...com>
Subject: Re: [PATCH v2] dt-bindings: omap: Convert omap.txt to yaml

On 30/03/2023 18:15, Andreas Kemnade wrote:
> On Thu, 30 Mar 2023 14:39:18 +0300
> Tony Lindgren <tony@...mide.com> wrote:
> 
>> * Andreas Kemnade <andreas@...nade.info> [230329 22:22]:
>>> From: Andrew Davis <afd@...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.

Best regards,
Krzysztof