Re: [PATCH v2 1/2] dt-bindings: power: supply: ltc3350-charger: Add bindings

[Mike Looijmans]

On 09-04-2024 15:31, Rob Herring wrote:
> On Tue, 09 Apr 2024 14:35:54 +0200, Mike Looijmans wrote:
> > The LTC3350 is a backup power controller that can charge and monitor
> > a series stack of one to four supercapacitors.
> >
> > Signed-off-by: Mike Looijmans <mike.looijmans@xxxxxxxx>
> >
> > ---
> >
> > Changes in v2:
> > Rename to lltc,ltc3350.yaml
> > Fix spaces and indentation
> >
> >   .../bindings/power/supply/lltc,ltc3350.yaml   | 54 +++++++++++++++++++
> >   1 file changed, 54 insertions(+)
> >   create mode 100644 Documentation/devicetree/bindings/power/supply/lltc,ltc3350.yaml
> My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check'
> on your patch (DT_CHECKER_FLAGS is new in v5.13):
>
> yamllint warnings/errors:
>
> dtschema/dtc warnings/errors:
> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/power/supply/lltc,ltc3350.yaml: $id: Cannot determine base path from $id, relative path/filename doesn't match actual path or filename
>   	 $id: http://devicetree.org/schemas/power/supply/ltc3350.yaml
>   	file: /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/power/supply/lltc,ltc3350.yaml

Ah, should have run the test AFTER renaming...

Need a v3 then.

--

Mike Looijmans
System Expert

TOPIC Embedded Products B.V.
Materiaalweg 4, 5681 RJ Best
The Netherlands

T: +31 (0) 499 33 69 69
E: mike.looijmans@xxxxxxxx
W: www.topic.nl