lkml.org 
[lkml]   [2022]   [Aug]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH net-next v4 6/7] dt-bindings: net: pse-dt: add bindings for generic PSE controller
On Sun, Aug 28, 2022 at 08:30:20AM +0200, Oleksij Rempel wrote:
> 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.yaml
>
> diff --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

\
 
 \ /
  Last update: 2022-08-30 22:11    [W:0.123 / U:0.196 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site