Re: [PATCH net-next v9 08/14] dt-bindings: net: pse-pd: Add another way of describing several PSE PIs

[Kory Maincent]

On Thu, 18 Apr 2024 12:07:11 +0200
Oleksij Rempel <o.rempel@xxxxxxxxxxxxxx> wrote:

> Hi Rob,
> 
> On Wed, Apr 17, 2024 at 10:34:26AM -0500, Rob Herring wrote:
> > On Wed, 17 Apr 2024 16:39:56 +0200, Kory Maincent wrote:  
> ....
> > > ---
> > >  .../bindings/net/pse-pd/pse-controller.yaml        | 101
> > > ++++++++++++++++++++- 1 file changed, 98 insertions(+), 3 deletions(-)
> > >   
> > 
> > My bot found errors running 'make dt_binding_check' on your patch:
> > 
> > yamllint warnings/errors:
> > 
> > dtschema/dtc warnings/errors:
> > 
> > 
> > doc reference errors (make refcheckdocs):
> > Warning: Documentation/devicetree/bindings/net/pse-pd/pse-controller.yaml
> > references a file that doesn't exist:
> > Documentation/networking/pse-pd/pse-pi.rst
> > Documentation/devicetree/bindings/net/pse-pd/pse-controller.yaml:
> > Documentation/networking/pse-pd/pse-pi.rst  
> 
> Hm... this documentation was added by patch 7. I assume refcheckdocs
> is searching in the wrong place or should the path be relative?
> 
> Is this warning a blocker for this patch set?

Yeah it is a false positive.
I don't think so. I assume maintainers time is the blockers for the patch
series to be accepted.

Regards,
-- 
Köry Maincent, Bootlin
Embedded Linux and kernel engineering
https://bootlin.com