On 05/12/2024 08:48, Krzysztof Kozlowski wrote: >> + patternProperties: >> + "^led@[0-3]$": >> + type: object >> + $ref: /schemas/leds/common.yaml# >> + unevaluatedProperties: false >> + >> + properties: >> + reg: >> + maxItems: 1 >> + >> + required: >> + - reg >> + >> + required: >> + - compatible >> + >> +patternProperties: >> + "^fuel-gauge@[0-9a-f]+$": > > > How unit address appeared here? It was absolutely never reviewed. This > is significant change, so drop the review tag. > > It's also a no, drop unit address. To explain more: In the few Maxim PMIC/MUIC charger designs known to me, the fuel gauge is a separate device. In all implementations it is even on separate bus, although does not have to. You claim that this is not a separate device and you even claim that this *MUST* be on the same bus. Both claims sound just wrong, IMO. This needs explanation. It seems you develop bindings just to match your drivers and that is not the correct process. Bindings must match the hardware, not your driver architecture. Best regards, Krzysztof
