[PATCH net-next v2] dt-bindings: net: Document Marvell PHY "marvell,reg-init"

[PATCH net-next v2] dt-bindings: net: Document Marvell PHY "marvell,reg-init"

[Rob Herring (Arm)]

The Marvell PHY driver consumes marvell,reg-init as one or more
<page register mask value> tuples. Define the property so dtbs_check
validates the existing board descriptions.

Existing users have no compatible or just "ethernet-phy-ieee802.3-c22"
so this has to go into the generic ethernet-phy.yaml schema. Marking it
deprecated both for that reason and because opaque register
initialization values are discouraged in DT.

Assisted-by: LLM
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
v2:
 - Add constraints on item values
 - Correct mask description
---
 .../devicetree/bindings/net/ethernet-phy.yaml    | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/Documentation/devicetree/bindings/net/ethernet-phy.yaml b/Documentation/devicetree/bindings/net/ethernet-phy.yaml
index c3ebb3af8b52..df50c4c33d76 100644
--- a/Documentation/devicetree/bindings/net/ethernet-phy.yaml
+++ b/Documentation/devicetree/bindings/net/ethernet-phy.yaml
@@ -269,6 +269,22 @@ properties:
       have to be placed on board. When omitted, the PHYs default will be left as
       is.
 
+  marvell,reg-init:
+    $ref: /schemas/types.yaml#/definitions/uint32-matrix
+    deprecated: true
+    description:
+      Register initialization tuples for Marvell PHYs.
+    items:
+      items:
+        - description: register page
+          maximum: 0xffff
+        - description: register number
+          maximum: 31
+        - description: mask of bits to preserve
+          maximum: 0xffff
+        - description: value to apply
+          maximum: 0xffff
+
   leds:
     type: object
 
-- 
2.53.0

Re: [PATCH net-next v2] dt-bindings: net: Document Marvell PHY "marvell,reg-init"

[Andrew Lunn]

> +  marvell,reg-init:
> +    $ref: /schemas/types.yaml#/definitions/uint32-matrix
> +    deprecated: true
> +    description:
> +      Register initialization tuples for Marvell PHYs.
> +    items:
> +      items:
> +        - description: register page
> +          maximum: 0xffff
> +        - description: register number
> +          maximum: 31
> +        - description: mask of bits to preserve
> +          maximum: 0xffff
> +        - description: value to apply
> +          maximum: 0xffff

This fits with my understanding of the hardware.

Reviewed-by: Andrew Lunn <andrew@lunn.ch>

    Andrew