linux-kernel - Re: [PATCH v2 2/3] docs: dt: writing-bindings: Express better expectations of "specific"

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: <175197733309.276747.1347706075364482943.robh@kernel.org>
Date: Tue, 8 Jul 2025 07:22:14 -0500
From: "Rob Herring (Arm)" <robh@...nel.org>
To: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
Cc: devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
	Krzysztof Kozlowski <krzk+dt@...nel.org>,
	Conor Dooley <conor+dt@...nel.org>
Subject: Re: [PATCH v2 2/3] docs: dt: writing-bindings: Express better
 expectations of "specific"


On Mon, 07 Jul 2025 11:50:21 +0200, Krzysztof Kozlowski wrote:
> Devicetree bindings are supposed to be specific in terms of compatibles
> and other properties.  Short "specific" has many implications, so extend
> the description to cover them:
> 
> 1. Mention no family names and avoid generic SoC fallbacks in
>    compatible.  The list grew, mixing DO's and DON'T's, so split it into
>    multiple items.
> 
> 2. No properties implied by the compatible.
> 
> 3. Document desired lack of ABI impact and acceptable solution if such
>    needs arises: clearly marking it in commit msg.
> 
> All above follows established Devicetree bindings maintainers review
> practice, so no new rules in practice are introduced here.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
> 
> ---
> 
> Changes in v2:
> Implement entire Conor's feedback:
> 1. Grammar
> 2. Split DO/DON'T in compatible section
> 3. Rephrase the ABI break
> ---
>  .../devicetree/bindings/writing-bindings.rst  | 27 +++++++++++++++----
>  1 file changed, 22 insertions(+), 5 deletions(-)
> 

Applied, thanks!