linux-kernel - Re: [PATCH v4 2/2] regulator: pf530x: dt-bindings: nxp,pf530x-regulator

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: <45ab7df9-f547-47c8-817d-c3f7be600300@carnegierobotics.com>
Date: Thu, 4 Sep 2025 12:58:05 +0000
From: Woody Douglass <wdouglass@...negierobotics.com>
To: Krzysztof Kozlowski <krzk@...nel.org>, Liam Girdwood
	<lgirdwood@...il.com>, Mark Brown <broonie@...nel.org>, Rob Herring
	<robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
	<conor+dt@...nel.org>
CC: "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>
Subject: Re: [PATCH v4 2/2] regulator: pf530x: dt-bindings:
 nxp,pf530x-regulator

On 9/4/25 01:44, Krzysztof Kozlowski wrote:
> On 03/09/2025 22:03, Woodrow Douglass wrote:
>> Bindings for the pf530x series of voltage regulators
>>
>> Signed-off-by: Woodrow Douglass <wdouglass@...negierobotics.com>
>> ---
>>  .../devicetree/bindings/regulator/nxp,pf5300.yaml  | 52 ++++++++++++++++++++++
>>  1 file changed, 52 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/regulator/nxp,pf5300.yaml b/Documentation/devicetree/bindings/regulator/nxp,pf5300.yaml
>> new file mode 100644
>> index 000000000000..26cba1f1af62
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/regulator/nxp,pf5300.yaml
>> @@ -0,0 +1,52 @@
>> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>> +%YAML 1.2
>> +---
>> +$id: http://devicetree.org/schemas/regulator/nxp,pf530x-regulator.yaml#
>> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>> +
>> +title: NXP PF5300/PF5301/PF5302 PMIC regulators
>> +
>> +maintainers:
>> +  - Woodrow Douglass <wdouglass@...negierobotics.com>
>> +
>> +description: |
>> +  The PF5300, PF5301, and PF5302 integrate high-performance buck converters,
>> +  12 A, 8 A, and 15 A, respectively, to power high-end automotive and industrial
>> +  processors. With adaptive voltage positioning and a high-bandwidth loop, they
>> +  offer transient regulation to minimize capacitor requirements.
>> +
>> +properties:
>> +  compatible:
>> +    enum:
>> +      - nxp,pf5300
>> +      - nxp,pf5301
>> +      - nxp,pf5302
> 
> Still compatibility not expressed.
> 

I'm sorry for the misunderstanding here -- I did reply (as Mark Brown mentioned in his
reply to this email) in the cover letter for v3 of this patch (this message 
https://lore.kernel.org/lkml/20250902-pf530x-v3-0-4242e7687761@carnegierobotics.com/).
Mark is right, I don't really understand what you're asking for, and other bindings for 
regulators seem to list off each compatible model in the way that I have. I have prepared
a v5 of this patchset that incorporates the various comments, but I'll wait for some
clarification here before submitting it.

> Please respond to comments.
> 
> Best regards,
> Krzysztof
> 

Thanks,
Woodrow Douglass