linux-kernel - Re: [PATCH v1] dt-bindings: trivial: Document TPS53685

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] [day] [month] [year] [list]

Message-ID: <9fa738c8-e993-4ca1-8bed-cd31029cadec@kernel.org>
Date: Sat, 12 Apr 2025 12:06:50 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Chiang Brian <chiang.brian@...entec.com>
Cc: conor+dt@...nel.org, devicetree@...r.kernel.org, krzk+dt@...nel.org,
 linux-kernel@...r.kernel.org, robh@...nel.org
Subject: Re: [PATCH v1] dt-bindings: trivial: Document TPS53685

On 11/04/2025 14:47, Chiang Brian wrote:
> On 14/03/2025 07:11, Chiang Brian wrote:
>>
>> On 14/03/2025 04:28, Chiang Brian wrote:
>>> Add undocumented tps53685 into compatible in dt-bindings
>>>
>>> Signed-off-by: Chiang Brian <chiang.brian@...entec.com>
>>> ---
>>>  Documentation/devicetree/bindings/trivial-devices.yaml | 2 ++
>>>  1 file changed, 2 insertions(+)
>>>
>>> diff --git a/Documentation/devicetree/bindings/trivial-devices.yaml b/Documentation/devicetree/bindings/trivial-devices.yaml
>>> index fadbd3c041c8..c98d69facb48 100644
>>> --- a/Documentation/devicetree/bindings/trivial-devices.yaml
>>> +++ b/Documentation/devicetree/bindings/trivial-devices.yaml
>>> @@ -380,6 +380,8 @@ properties:
>>>            - ti,tps53676
>>>              # TI Dual channel DCAP+ multiphase controller TPS53679
>>>            - ti,tps53679
>>> +            # TI Dual channel DCAP+ multiphase controller TPS53685 with AMD-SVI3
>>> +          - ti,tps53685
>>
>> There is no user of such compatible, so how can it be undocumented?
> 
> The following link is the patch which I would like to add support tps53685,
> and I think it is the user of the compatible:

So that patch introduces undocumented compatible? That patchset needs to
be fixed instead - this belongs there. The commit msg here is not
correct: there is no undocumented compatible.

Please also read submitting patches in DT or just simply look how every
other patchset is sent. Every.


Best regards,
Krzysztof