Messages in this thread | | | Date | Tue, 28 Mar 2023 00:44:45 +0100 | From | Andre Przywara <> | Subject | Re: [PATCH v9 1/3] dt-bindings: mfd: x-powers,axp152: Document the AXP313a variant |
| |
On Sat, 25 Mar 2023 12:40:38 +0100 Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote:
Hi,
> On 24/03/2023 12:30, Andre Przywara wrote: > > From: Martin Botka <martin.botka@somainline.org> > > > > The X-Powers AXP313a is a PMIC used on some devices with the Allwinner > > H616 or H313 SoC. > > According to the datasheet, the DC/DC converter PWM frequency is fixed > > (to 3 MHz), so disallow the property that lets us set this frequency > > for the other PMICs. > > > > Signed-off-by: Martin Botka <martin.botka@somainline.org> > > Signed-off-by: Andre Przywara <andre.przywara@arm.com> > > --- > > .../devicetree/bindings/mfd/x-powers,axp152.yaml | 12 ++++++++++++ > > 1 file changed, 12 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/mfd/x-powers,axp152.yaml b/Documentation/devicetree/bindings/mfd/x-powers,axp152.yaml > > index b7a8747d5fa0e..4b4f42cb09ef7 100644 > > --- a/Documentation/devicetree/bindings/mfd/x-powers,axp152.yaml > > +++ b/Documentation/devicetree/bindings/mfd/x-powers,axp152.yaml > > @@ -79,6 +79,17 @@ allOf: > > required: > > - interrupts > > > > + - if: > > + properties: > > + compatible: > > + contains: > > + const: x-powers,axp313a > > + > > + then: > > + not: > > + required: > > + - x-powers,dcdc-freq > > Should be simpler x-powers,dcdc-freq: false
Ah, many thanks! I was a bit concerned about "not required" not sounding quite right, but I indeed copied from the other cases and verified that it worked as expected. But yours is indeed much better, thanks for the other patch!
Cheers, Andre
> I'll fix the other cases. > > Best regards, > Krzysztof >
| |