netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [net-next,PATCH v2] dt-bindings: net: realtek,rtl82xx: Document RTL8211F LED support
@ 2024-07-08 21:16 Marek Vasut
  2024-07-09 16:32 ` Rob Herring (Arm)
  2024-07-11  2:20 ` patchwork-bot+netdevbpf
  0 siblings, 2 replies; 3+ messages in thread
From: Marek Vasut @ 2024-07-08 21:16 UTC (permalink / raw)
  To: netdev
  Cc: kernel, Marek Vasut, David S. Miller, Andrew Lunn, Conor Dooley,
	Eric Dumazet, Florian Fainelli, Heiner Kallweit, Jakub Kicinski,
	Krzysztof Kozlowski, Paolo Abeni, Rob Herring, devicetree

The RTL8211F PHY does support LED configuration, document support
for LEDs in the binding document.

Signed-off-by: Marek Vasut <marex@denx.de>
---
Cc: "David S. Miller" <davem@davemloft.net>
Cc: Andrew Lunn <andrew@lunn.ch>
Cc: Conor Dooley <conor+dt@kernel.org>
Cc: Eric Dumazet <edumazet@google.com>
Cc: Florian Fainelli <f.fainelli@gmail.com>
Cc: Heiner Kallweit <hkallweit1@gmail.com>
Cc: Jakub Kicinski <kuba@kernel.org>
Cc: Krzysztof Kozlowski <krzk+dt@kernel.org>
Cc: Paolo Abeni <pabeni@redhat.com>
Cc: Rob Herring <robh@kernel.org>
Cc: devicetree@vger.kernel.org
Cc: netdev@vger.kernel.org
---
V2: Invert the conditional
---
 .../bindings/net/realtek,rtl82xx.yaml           | 17 ++++++++++++++---
 1 file changed, 14 insertions(+), 3 deletions(-)

diff --git a/Documentation/devicetree/bindings/net/realtek,rtl82xx.yaml b/Documentation/devicetree/bindings/net/realtek,rtl82xx.yaml
index 18ee72f5c74a8..d248a08a2136b 100644
--- a/Documentation/devicetree/bindings/net/realtek,rtl82xx.yaml
+++ b/Documentation/devicetree/bindings/net/realtek,rtl82xx.yaml
@@ -14,9 +14,6 @@ maintainers:
 description:
   Bindings for Realtek RTL82xx PHYs
 
-allOf:
-  - $ref: ethernet-phy.yaml#
-
 properties:
   compatible:
     enum:
@@ -41,6 +38,8 @@ properties:
       - ethernet-phy-id001c.cad0
       - ethernet-phy-id001c.cb00
 
+  leds: true
+
   realtek,clkout-disable:
     type: boolean
     description:
@@ -54,6 +53,18 @@ properties:
 
 unevaluatedProperties: false
 
+allOf:
+  - $ref: ethernet-phy.yaml#
+  - if:
+      not:
+        properties:
+          compatible:
+            contains:
+              const: ethernet-phy-id001c.c916
+    then:
+      properties:
+        leds: false
+
 examples:
   - |
     mdio {
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [net-next,PATCH v2] dt-bindings: net: realtek,rtl82xx: Document RTL8211F LED support
  2024-07-08 21:16 [net-next,PATCH v2] dt-bindings: net: realtek,rtl82xx: Document RTL8211F LED support Marek Vasut
@ 2024-07-09 16:32 ` Rob Herring (Arm)
  2024-07-11  2:20 ` patchwork-bot+netdevbpf
  1 sibling, 0 replies; 3+ messages in thread
From: Rob Herring (Arm) @ 2024-07-09 16:32 UTC (permalink / raw)
  To: Marek Vasut
  Cc: Eric Dumazet, Jakub Kicinski, devicetree, David S. Miller,
	Conor Dooley, kernel, Florian Fainelli, Paolo Abeni,
	Krzysztof Kozlowski, netdev, Heiner Kallweit, Andrew Lunn


On Mon, 08 Jul 2024 23:16:29 +0200, Marek Vasut wrote:
> The RTL8211F PHY does support LED configuration, document support
> for LEDs in the binding document.
> 
> Signed-off-by: Marek Vasut <marex@denx.de>
> ---
> Cc: "David S. Miller" <davem@davemloft.net>
> Cc: Andrew Lunn <andrew@lunn.ch>
> Cc: Conor Dooley <conor+dt@kernel.org>
> Cc: Eric Dumazet <edumazet@google.com>
> Cc: Florian Fainelli <f.fainelli@gmail.com>
> Cc: Heiner Kallweit <hkallweit1@gmail.com>
> Cc: Jakub Kicinski <kuba@kernel.org>
> Cc: Krzysztof Kozlowski <krzk+dt@kernel.org>
> Cc: Paolo Abeni <pabeni@redhat.com>
> Cc: Rob Herring <robh@kernel.org>
> Cc: devicetree@vger.kernel.org
> Cc: netdev@vger.kernel.org
> ---
> V2: Invert the conditional
> ---
>  .../bindings/net/realtek,rtl82xx.yaml           | 17 ++++++++++++++---
>  1 file changed, 14 insertions(+), 3 deletions(-)
> 

Reviewed-by: Rob Herring (Arm) <robh@kernel.org>


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [net-next,PATCH v2] dt-bindings: net: realtek,rtl82xx: Document RTL8211F LED support
  2024-07-08 21:16 [net-next,PATCH v2] dt-bindings: net: realtek,rtl82xx: Document RTL8211F LED support Marek Vasut
  2024-07-09 16:32 ` Rob Herring (Arm)
@ 2024-07-11  2:20 ` patchwork-bot+netdevbpf
  1 sibling, 0 replies; 3+ messages in thread
From: patchwork-bot+netdevbpf @ 2024-07-11  2:20 UTC (permalink / raw)
  To: Marek Vasut
  Cc: netdev, kernel, davem, andrew, conor+dt, edumazet, f.fainelli,
	hkallweit1, kuba, krzk+dt, pabeni, robh, devicetree

Hello:

This patch was applied to netdev/net-next.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Mon,  8 Jul 2024 23:16:29 +0200 you wrote:
> The RTL8211F PHY does support LED configuration, document support
> for LEDs in the binding document.
> 
> Signed-off-by: Marek Vasut <marex@denx.de>
> ---
> Cc: "David S. Miller" <davem@davemloft.net>
> Cc: Andrew Lunn <andrew@lunn.ch>
> Cc: Conor Dooley <conor+dt@kernel.org>
> Cc: Eric Dumazet <edumazet@google.com>
> Cc: Florian Fainelli <f.fainelli@gmail.com>
> Cc: Heiner Kallweit <hkallweit1@gmail.com>
> Cc: Jakub Kicinski <kuba@kernel.org>
> Cc: Krzysztof Kozlowski <krzk+dt@kernel.org>
> Cc: Paolo Abeni <pabeni@redhat.com>
> Cc: Rob Herring <robh@kernel.org>
> Cc: devicetree@vger.kernel.org
> Cc: netdev@vger.kernel.org
> 
> [...]

Here is the summary with links:
  - [net-next,v2] dt-bindings: net: realtek,rtl82xx: Document RTL8211F LED support
    https://git.kernel.org/netdev/net-next/c/ab896aa62e30

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2024-07-11  2:20 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-07-08 21:16 [net-next,PATCH v2] dt-bindings: net: realtek,rtl82xx: Document RTL8211F LED support Marek Vasut
2024-07-09 16:32 ` Rob Herring (Arm)
2024-07-11  2:20 ` patchwork-bot+netdevbpf

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).