[RESEND,net,v2] dt-bindings: net: sun8i-emac: Add phy-supply property

Message ID 20221231220546.1188-1-samuel@sholland.org
State New
Headers
Series [RESEND,net,v2] dt-bindings: net: sun8i-emac: Add phy-supply property |

Commit Message

Samuel Holland Dec. 31, 2022, 10:05 p.m. UTC
  This property has always been supported by the Linux driver; see
commit 9f93ac8d4085 ("net-next: stmmac: Add dwmac-sun8i"). In fact, the
original driver submission includes the phy-supply code but no mention
of it in the binding, so the omission appears to be accidental. In
addition, the property is documented in the binding for the previous
hardware generation, allwinner,sun7i-a20-gmac.

Document phy-supply in the binding to fix devicetree validation for the
25+ boards that already use this property.

Fixes: 0441bde003be ("dt-bindings: net-next: Add DT bindings documentation for Allwinner dwmac-sun8i")
Acked-by: Rob Herring <robh@kernel.org>
Reviewed-by: Andre Przywara <andre.przywara@arm.com>
Signed-off-by: Samuel Holland <samuel@sholland.org>
---
Resending with "net" explicitly in the subject.

Changes in v2:
 - Drop the rest of the series, which was obsoleted by the dt-schema fix
 - Add Acked-by/Reviewed-by tags

 .../devicetree/bindings/net/allwinner,sun8i-a83t-emac.yaml     | 3 +++
 1 file changed, 3 insertions(+)
  

Comments

patchwork-bot+netdevbpf@kernel.org Jan. 1, 2023, 3 p.m. UTC | #1
Hello:

This patch was applied to netdev/net.git (master)
by David S. Miller <davem@davemloft.net>:

On Sat, 31 Dec 2022 16:05:46 -0600 you wrote:
> This property has always been supported by the Linux driver; see
> commit 9f93ac8d4085 ("net-next: stmmac: Add dwmac-sun8i"). In fact, the
> original driver submission includes the phy-supply code but no mention
> of it in the binding, so the omission appears to be accidental. In
> addition, the property is documented in the binding for the previous
> hardware generation, allwinner,sun7i-a20-gmac.
> 
> [...]

Here is the summary with links:
  - [RESEND,net,v2] dt-bindings: net: sun8i-emac: Add phy-supply property
    https://git.kernel.org/netdev/net/c/a3542b0ccd58

You are awesome, thank you!
  

Patch

diff --git a/Documentation/devicetree/bindings/net/allwinner,sun8i-a83t-emac.yaml b/Documentation/devicetree/bindings/net/allwinner,sun8i-a83t-emac.yaml
index 1432fda3b603..47bc2057e629 100644
--- a/Documentation/devicetree/bindings/net/allwinner,sun8i-a83t-emac.yaml
+++ b/Documentation/devicetree/bindings/net/allwinner,sun8i-a83t-emac.yaml
@@ -40,6 +40,9 @@  properties:
   clock-names:
     const: stmmaceth
 
+  phy-supply:
+    description: PHY regulator
+
   syscon:
     $ref: /schemas/types.yaml#/definitions/phandle
     description: