Re: [PATCH net-next v4 6/7] dt-bindings: net: pse-dt: add bindings for generic PSE controller
flat view
From: Rob Herring <robh@kernel.org>
Date: 2022-08-30 20:11:14
Also in:
linux-devicetree, linux-doc, lkml
On Sun, Aug 28, 2022 at 08:30:20AM +0200, Oleksij Rempel wrote:
quoted hunk ↗ jump to hunk
Add binding for generic Ethernet PSE controller. Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de> --- changes v4: - rename to PSE regulator - drop currently unused properties - use own compatible for PoDL PSE changes v2: - rename compatible to more generic "ieee802.3-pse" - add class and type properties for PoDL and PoE variants - add pairs property --- .../bindings/net/pse-pd/pse-regulator.yaml | 40 +++++++++++++++++++ 1 file changed, 40 insertions(+) create mode 100644 Documentation/devicetree/bindings/net/pse-pd/pse-regulator.yamldiff --git a/Documentation/devicetree/bindings/net/pse-pd/pse-regulator.yaml b/Documentation/devicetree/bindings/net/pse-pd/pse-regulator.yaml new file mode 100644 index 0000000000000..1a906d2135a7a --- /dev/null +++ b/Documentation/devicetree/bindings/net/pse-pd/pse-regulator.yaml@@ -0,0 +1,40 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/net/pse-pd/pse-regulator.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Regulator based Power Sourcing Equipment + +maintainers: + - Oleksij Rempel <o.rempel@pengutronix.de> + +description: Regulator based PSE controller. The device must be referenced by + the PHY node to control power injection to the Ethernet cable. + +properties: + compatible: + description: Regulator based PoDL PSE controller for a single twisted-pair + link. + const: podl-pse-regulator + + '#pse-cells': + const: 0
Do you have intentions that this would be non-zero? This series is defining a new common binding, but only creating a specific schema. You need to define in a common schema what the purpose of the cells may be and if there's common constraints defining those in the common schema. There's several examples where only 0 or 1 is allowed for example. There's a standard pattern of '#foo-cells' for the provider and 'foos' for the consumer. Ultimately, anything that doesn't follow that pattern needs explicit support in dtschema to decode the consumers. IOW, try to follow the pattern. Either '#ieee802.3-pse-cells'/'ieee802.3-pses' or '#pse-cells'/'pses'. Neither seems great to me. The former is a bit too specific and the latter a bit too short. Shrug. Rob