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

Woody Douglass
2025-09-04T12:58:23Z

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@carnegierobotics.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@carnegierobotics.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