Re: [PATCH v3 2/3] dt-bindings: power: supply: add cw2015_battery bindings

From: Daniel Thompson
Date: Thu Mar 12 2020 - 06:17:37 EST


On Thu, Mar 12, 2020 at 12:17:55AM +0100, Tobias Schramm wrote:
> Hi Daniel,
>
> thanks for reviewing. The typo will be fixed for v4.
>
> >> + power-supplies:
> >> + description:
> >> + Specifies supplies used for charging the battery connected to this gauge
> >> + allOf:
> >> + - $ref: /schemas/types.yaml#/definitions/phandle-array
> >> + - minItems: 1
> >> + maxItems: 8 # Should be enough
> >
> > Is it necessary to set a maximum? power_supply.txt is still a text file
> > but there is no mention of a maximum there.
> >
> I think so? Removing maxItems and running dtbs_check on a dts with more
> than one supply phandle in the power-supplies property results in an error:
> linux/arch/arm64/boot/dts/rockchip/rk3399-pinebook-pro.dt.yaml:
> cw2015@62: power-supplies: [[142], [50]] is too long

Interesting. I saw the "Should be enough" comment replicated in several
YAML bindings (with varying degress of paranoia about how much "enough" is).
There are also several that simply set minItems without setting
maxItems, perhaps they have just never been any DTs that test those
bindings with more than one item.


Daniel.