Re: [PATCH v4 2/5] regulator: dt-bindings: Add TI TPS65214 PMIC bindings

From: Krzysztof Kozlowski
Date: Sat Feb 01 2025 - 06:43:03 EST


On 31/01/2025 23:11, Shree Ramamoorthy wrote:
>
> @@ -98,6 +102,16 @@ required:
> additionalProperties: false
>
> allOf:
> + - if:
> + properties:
> + compatible:
> + contains:
> + const: ti,tps65214
> + then:
> + properties:
> + regulators:
> + patternProperties:
> + "^ldo[3-4]$": false

So now, when you combined these patches, you can easily see that you are
duplicating your other "then:" from patch #1. Just combine these into
enum. Don't duplicate.

Best regards,
Krzysztof