Re: [PATCH v2] dt-bindings: mfd: sprd,sc2731: convert to YAML

Stanislav Jakubek
2024-10-30T08:14:29Z

On Wed, Oct 30, 2024 at 08:48:25AM +0100, Krzysztof Kozlowski wrote:
> On 30/10/2024 08:42, Stanislav Jakubek wrote:
> >>
> >>> +
> >>> + '#address-cells':
> >>> + const: 1
> >>> +
> >>> + '#interrupt-cells':
> >>> + const: 1
> >>> +
> >>> + '#size-cells':
> >>> + const: 0
> >>> +
> >>> + regulators:
> >>> + type: object
> >>> + $ref: /schemas/regulator/sprd,sc2731-regulator.yaml#
> >>> +
> >>> +patternProperties:
> >>> + "^adc@[0-9a-f]+$":
> >>> + type: object
> >>> + $ref: /schemas/iio/adc/sprd,sc2720-adc.yaml#
> >>> +
> >>> + "^charger@[0-9a-f]+$":
> >>> + type: object
> >>> + $ref: /schemas/power/supply/sc2731-charger.yaml#
> >>> +
> >>> + "^efuse@[0-9a-f]+$":
> >>> + type: object
> >>> + $ref: /schemas/nvmem/sprd,sc2731-efuse.yaml#
> >>
> >> I don't think this was merged. You still have dependency.
> >
> > This is in next-20241029, which this patch is based on.
>
> Try what I wrote below and see if this works...

I assume you meant the MFD maintainers' tree here.
Yes, that tree doesn't have the nvmem patch this depends on.

Would the approach with listing the compatibles and additionalProperties:
true be considered a temporary workaround?

If so, should I split this into 2 patches?
- 1st patch with the nvmem workaround above
- 2nd with adding the nvmem $ref back (which would get merged later)

Regards,
Stanislav

>
> Best regards,
> Krzysztof
>