On Thu, 28 Mar 2024 13:32:09 +0100 Andrew Lunn <andrew@xxxxxxx> wrote: > On Tue, Mar 26, 2024 at 10:39:28AM -0500, Rob Herring wrote: > > > > On Tue, 26 Mar 2024 15:04:48 +0100, Kory Maincent wrote: > > > From: Kory Maincent (Dent Project) <kory.maincent@xxxxxxxxxxx> > > > > > > PSE PI setup may encompass multiple PSE controllers or auxiliary circuits > > > that collectively manage power delivery to one Ethernet port. > > > Such configurations might support a range of PoE standards and require > > > the capability to dynamically configure power delivery based on the > > > operational mode (e.g., PoE2 versus PoE4) or specific requirements of > > > connected devices. In these instances, a dedicated PSE PI node becomes > > > essential for accurately documenting the system architecture. This node > > > would serve to detail the interactions between different PSE controllers, > > > the support for various PoE modes, and any additional logic required to > > > coordinate power delivery across the network infrastructure. > > > > > > The old usage of "#pse-cells" is unsuficient as it carries only the PSE PI > > > index information. > > > > > > Signed-off-by: Kory Maincent <kory.maincent@xxxxxxxxxxx> > > > --- > > 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: > > > > > > 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 > > Is this a false positive? I suppose so as the file is added in the patch 10. Regards, -- Köry Maincent, Bootlin Embedded Linux and kernel engineering https://bootlin.com