All the mail mirrored from lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH net-next v2 0/2] net: xilinx_gmii2rgmii: Add clock support
@ 2024-05-17  5:47 ` Vineeth Karumanchi
  0 siblings, 0 replies; 10+ messages in thread
From: Vineeth Karumanchi @ 2024-05-17  5:47 UTC (permalink / raw
  To: git, davem, edumazet, kuba, pabeni, robh, krzk+dt, conor+dt,
	harini.katakam, andrew, hkallweit1, linux, michal.simek
  Cc: vineeth.karumanchi, netdev, devicetree, linux-kernel,
	linux-arm-kernel

Add input clock support to gmii_to_rgmii IP.
Add "clocks" bindings for the input clock.

Changes in v2:
- remove "clkin" clock name property.

v1 link : https://lore.kernel.org/netdev/20240515094645.3691877-1-vineeth.karumanchi@amd.com/ 

Vineeth Karumanchi (2):
  dt-bindings: net: xilinx_gmii2rgmii: Add clock support
  net: phy: xilinx-gmii2rgmii: Adopt clock support

 .../devicetree/bindings/net/xlnx,gmii-to-rgmii.yaml        | 4 ++++
 drivers/net/phy/xilinx_gmii2rgmii.c                        | 7 +++++++
 2 files changed, 11 insertions(+)

-- 
2.34.1


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

* [PATCH net-next v2 0/2] net: xilinx_gmii2rgmii: Add clock support
@ 2024-05-17  5:47 ` Vineeth Karumanchi
  0 siblings, 0 replies; 10+ messages in thread
From: Vineeth Karumanchi @ 2024-05-17  5:47 UTC (permalink / raw
  To: git, davem, edumazet, kuba, pabeni, robh, krzk+dt, conor+dt,
	harini.katakam, andrew, hkallweit1, linux, michal.simek
  Cc: vineeth.karumanchi, netdev, devicetree, linux-kernel,
	linux-arm-kernel

Add input clock support to gmii_to_rgmii IP.
Add "clocks" bindings for the input clock.

Changes in v2:
- remove "clkin" clock name property.

v1 link : https://lore.kernel.org/netdev/20240515094645.3691877-1-vineeth.karumanchi@amd.com/ 

Vineeth Karumanchi (2):
  dt-bindings: net: xilinx_gmii2rgmii: Add clock support
  net: phy: xilinx-gmii2rgmii: Adopt clock support

 .../devicetree/bindings/net/xlnx,gmii-to-rgmii.yaml        | 4 ++++
 drivers/net/phy/xilinx_gmii2rgmii.c                        | 7 +++++++
 2 files changed, 11 insertions(+)

-- 
2.34.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* [PATCH net-next v2 1/2] dt-bindings: net: xilinx_gmii2rgmii: Add clock support
  2024-05-17  5:47 ` Vineeth Karumanchi
@ 2024-05-17  5:47   ` Vineeth Karumanchi
  -1 siblings, 0 replies; 10+ messages in thread
From: Vineeth Karumanchi @ 2024-05-17  5:47 UTC (permalink / raw
  To: git, davem, edumazet, kuba, pabeni, robh, krzk+dt, conor+dt,
	harini.katakam, andrew, hkallweit1, linux, michal.simek
  Cc: vineeth.karumanchi, netdev, devicetree, linux-kernel,
	linux-arm-kernel

Add input clock support to gmii_to_rgmii IP.
Add "clocks" bindings for the input clock.

Signed-off-by: Vineeth Karumanchi <vineeth.karumanchi@amd.com>
---
 Documentation/devicetree/bindings/net/xlnx,gmii-to-rgmii.yaml | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/Documentation/devicetree/bindings/net/xlnx,gmii-to-rgmii.yaml b/Documentation/devicetree/bindings/net/xlnx,gmii-to-rgmii.yaml
index 0f781dac6717..672bff6dae28 100644
--- a/Documentation/devicetree/bindings/net/xlnx,gmii-to-rgmii.yaml
+++ b/Documentation/devicetree/bindings/net/xlnx,gmii-to-rgmii.yaml
@@ -31,6 +31,9 @@ properties:
   phy-handle:
     $ref: ethernet-controller.yaml#/properties/phy-handle
 
+  clocks:
+    description: 200/375 MHz free-running clock is used as input clock.
+
 required:
   - compatible
   - reg
@@ -51,5 +54,6 @@ examples:
             compatible = "xlnx,gmii-to-rgmii-1.0";
             reg = <8>;
             phy-handle = <&phy>;
+            clocks = <&dummy>;
         };
     };
-- 
2.34.1


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

* [PATCH net-next v2 1/2] dt-bindings: net: xilinx_gmii2rgmii: Add clock support
@ 2024-05-17  5:47   ` Vineeth Karumanchi
  0 siblings, 0 replies; 10+ messages in thread
From: Vineeth Karumanchi @ 2024-05-17  5:47 UTC (permalink / raw
  To: git, davem, edumazet, kuba, pabeni, robh, krzk+dt, conor+dt,
	harini.katakam, andrew, hkallweit1, linux, michal.simek
  Cc: vineeth.karumanchi, netdev, devicetree, linux-kernel,
	linux-arm-kernel

Add input clock support to gmii_to_rgmii IP.
Add "clocks" bindings for the input clock.

Signed-off-by: Vineeth Karumanchi <vineeth.karumanchi@amd.com>
---
 Documentation/devicetree/bindings/net/xlnx,gmii-to-rgmii.yaml | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/Documentation/devicetree/bindings/net/xlnx,gmii-to-rgmii.yaml b/Documentation/devicetree/bindings/net/xlnx,gmii-to-rgmii.yaml
index 0f781dac6717..672bff6dae28 100644
--- a/Documentation/devicetree/bindings/net/xlnx,gmii-to-rgmii.yaml
+++ b/Documentation/devicetree/bindings/net/xlnx,gmii-to-rgmii.yaml
@@ -31,6 +31,9 @@ properties:
   phy-handle:
     $ref: ethernet-controller.yaml#/properties/phy-handle
 
+  clocks:
+    description: 200/375 MHz free-running clock is used as input clock.
+
 required:
   - compatible
   - reg
@@ -51,5 +54,6 @@ examples:
             compatible = "xlnx,gmii-to-rgmii-1.0";
             reg = <8>;
             phy-handle = <&phy>;
+            clocks = <&dummy>;
         };
     };
-- 
2.34.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* [PATCH net-next v2 2/2] net: phy: xilinx-gmii2rgmii: Adopt clock support
  2024-05-17  5:47 ` Vineeth Karumanchi
@ 2024-05-17  5:47   ` Vineeth Karumanchi
  -1 siblings, 0 replies; 10+ messages in thread
From: Vineeth Karumanchi @ 2024-05-17  5:47 UTC (permalink / raw
  To: git, davem, edumazet, kuba, pabeni, robh, krzk+dt, conor+dt,
	harini.katakam, andrew, hkallweit1, linux, michal.simek
  Cc: vineeth.karumanchi, netdev, devicetree, linux-kernel,
	linux-arm-kernel

Add clock support to the gmii_to_rgmii IP.
Make clk optional to keep DTB backward compatibility.

Signed-off-by: Vineeth Karumanchi <vineeth.karumanchi@amd.com>
---
 drivers/net/phy/xilinx_gmii2rgmii.c | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/drivers/net/phy/xilinx_gmii2rgmii.c b/drivers/net/phy/xilinx_gmii2rgmii.c
index 7b1bc5fcef9b..7c51daecf18e 100644
--- a/drivers/net/phy/xilinx_gmii2rgmii.c
+++ b/drivers/net/phy/xilinx_gmii2rgmii.c
@@ -15,6 +15,7 @@
 #include <linux/mii.h>
 #include <linux/mdio.h>
 #include <linux/phy.h>
+#include <linux/clk.h>
 #include <linux/of_mdio.h>
 
 #define XILINX_GMII2RGMII_REG		0x10
@@ -85,11 +86,17 @@ static int xgmiitorgmii_probe(struct mdio_device *mdiodev)
 	struct device *dev = &mdiodev->dev;
 	struct device_node *np = dev->of_node, *phy_node;
 	struct gmii2rgmii *priv;
+	struct clk *clkin;
 
 	priv = devm_kzalloc(dev, sizeof(*priv), GFP_KERNEL);
 	if (!priv)
 		return -ENOMEM;
 
+	clkin = devm_clk_get_optional_enabled(dev, NULL);
+	if (IS_ERR(clkin))
+		return dev_err_probe(dev, PTR_ERR(clkin),
+					"Failed to get and enable clock from Device Tree\n");
+
 	phy_node = of_parse_phandle(np, "phy-handle", 0);
 	if (!phy_node) {
 		dev_err(dev, "Couldn't parse phy-handle\n");
-- 
2.34.1


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

* [PATCH net-next v2 2/2] net: phy: xilinx-gmii2rgmii: Adopt clock support
@ 2024-05-17  5:47   ` Vineeth Karumanchi
  0 siblings, 0 replies; 10+ messages in thread
From: Vineeth Karumanchi @ 2024-05-17  5:47 UTC (permalink / raw
  To: git, davem, edumazet, kuba, pabeni, robh, krzk+dt, conor+dt,
	harini.katakam, andrew, hkallweit1, linux, michal.simek
  Cc: vineeth.karumanchi, netdev, devicetree, linux-kernel,
	linux-arm-kernel

Add clock support to the gmii_to_rgmii IP.
Make clk optional to keep DTB backward compatibility.

Signed-off-by: Vineeth Karumanchi <vineeth.karumanchi@amd.com>
---
 drivers/net/phy/xilinx_gmii2rgmii.c | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/drivers/net/phy/xilinx_gmii2rgmii.c b/drivers/net/phy/xilinx_gmii2rgmii.c
index 7b1bc5fcef9b..7c51daecf18e 100644
--- a/drivers/net/phy/xilinx_gmii2rgmii.c
+++ b/drivers/net/phy/xilinx_gmii2rgmii.c
@@ -15,6 +15,7 @@
 #include <linux/mii.h>
 #include <linux/mdio.h>
 #include <linux/phy.h>
+#include <linux/clk.h>
 #include <linux/of_mdio.h>
 
 #define XILINX_GMII2RGMII_REG		0x10
@@ -85,11 +86,17 @@ static int xgmiitorgmii_probe(struct mdio_device *mdiodev)
 	struct device *dev = &mdiodev->dev;
 	struct device_node *np = dev->of_node, *phy_node;
 	struct gmii2rgmii *priv;
+	struct clk *clkin;
 
 	priv = devm_kzalloc(dev, sizeof(*priv), GFP_KERNEL);
 	if (!priv)
 		return -ENOMEM;
 
+	clkin = devm_clk_get_optional_enabled(dev, NULL);
+	if (IS_ERR(clkin))
+		return dev_err_probe(dev, PTR_ERR(clkin),
+					"Failed to get and enable clock from Device Tree\n");
+
 	phy_node = of_parse_phandle(np, "phy-handle", 0);
 	if (!phy_node) {
 		dev_err(dev, "Couldn't parse phy-handle\n");
-- 
2.34.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH net-next v2 0/2] net: xilinx_gmii2rgmii: Add clock support
  2024-05-17  5:47 ` Vineeth Karumanchi
@ 2024-05-17 19:25   ` Jakub Kicinski
  -1 siblings, 0 replies; 10+ messages in thread
From: Jakub Kicinski @ 2024-05-17 19:25 UTC (permalink / raw
  To: Vineeth Karumanchi
  Cc: git, davem, edumazet, pabeni, robh, krzk+dt, conor+dt,
	harini.katakam, andrew, hkallweit1, linux, michal.simek, netdev,
	devicetree, linux-kernel, linux-arm-kernel

On Fri, 17 May 2024 11:17:43 +0530 Vineeth Karumanchi wrote:
> Add input clock support to gmii_to_rgmii IP.
> Add "clocks" bindings for the input clock.

## Form letter - net-next-closed

The merge window for v6.10 has begun and we have already posted our pull
request. Therefore net-next is closed for new drivers, features, code
refactoring and optimizations. We are currently accepting bug fixes only.

Please repost when net-next reopens after May 26th.

RFC patches sent for review only are obviously welcome at any time.

See: https://www.kernel.org/doc/html/next/process/maintainer-netdev.html#development-cycle
-- 
pw-bot: defer


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

* Re: [PATCH net-next v2 0/2] net: xilinx_gmii2rgmii: Add clock support
@ 2024-05-17 19:25   ` Jakub Kicinski
  0 siblings, 0 replies; 10+ messages in thread
From: Jakub Kicinski @ 2024-05-17 19:25 UTC (permalink / raw
  To: Vineeth Karumanchi
  Cc: git, davem, edumazet, pabeni, robh, krzk+dt, conor+dt,
	harini.katakam, andrew, hkallweit1, linux, michal.simek, netdev,
	devicetree, linux-kernel, linux-arm-kernel

On Fri, 17 May 2024 11:17:43 +0530 Vineeth Karumanchi wrote:
> Add input clock support to gmii_to_rgmii IP.
> Add "clocks" bindings for the input clock.

## Form letter - net-next-closed

The merge window for v6.10 has begun and we have already posted our pull
request. Therefore net-next is closed for new drivers, features, code
refactoring and optimizations. We are currently accepting bug fixes only.

Please repost when net-next reopens after May 26th.

RFC patches sent for review only are obviously welcome at any time.

See: https://www.kernel.org/doc/html/next/process/maintainer-netdev.html#development-cycle
-- 
pw-bot: defer


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH net-next v2 1/2] dt-bindings: net: xilinx_gmii2rgmii: Add clock support
  2024-05-17  5:47   ` Vineeth Karumanchi
@ 2024-05-19 17:33     ` Krzysztof Kozlowski
  -1 siblings, 0 replies; 10+ messages in thread
From: Krzysztof Kozlowski @ 2024-05-19 17:33 UTC (permalink / raw
  To: Vineeth Karumanchi, git, davem, edumazet, kuba, pabeni, robh,
	krzk+dt, conor+dt, harini.katakam, andrew, hkallweit1, linux,
	michal.simek
  Cc: netdev, devicetree, linux-kernel, linux-arm-kernel

On 17/05/2024 07:47, Vineeth Karumanchi wrote:
> Add input clock support to gmii_to_rgmii IP.
> Add "clocks" bindings for the input clock.

Both sentences look like saying the same... confused.

> 
> Signed-off-by: Vineeth Karumanchi <vineeth.karumanchi@amd.com>
> ---
>  Documentation/devicetree/bindings/net/xlnx,gmii-to-rgmii.yaml | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/net/xlnx,gmii-to-rgmii.yaml b/Documentation/devicetree/bindings/net/xlnx,gmii-to-rgmii.yaml
> index 0f781dac6717..672bff6dae28 100644
> --- a/Documentation/devicetree/bindings/net/xlnx,gmii-to-rgmii.yaml
> +++ b/Documentation/devicetree/bindings/net/xlnx,gmii-to-rgmii.yaml
> @@ -31,6 +31,9 @@ properties:
>    phy-handle:
>      $ref: ethernet-controller.yaml#/properties/phy-handle
>  
> +  clocks:
> +    description: 200/375 MHz free-running clock is used as input clock.

Missing constraints. Probably you wanted items above description.


Best regards,
Krzysztof


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

* Re: [PATCH net-next v2 1/2] dt-bindings: net: xilinx_gmii2rgmii: Add clock support
@ 2024-05-19 17:33     ` Krzysztof Kozlowski
  0 siblings, 0 replies; 10+ messages in thread
From: Krzysztof Kozlowski @ 2024-05-19 17:33 UTC (permalink / raw
  To: Vineeth Karumanchi, git, davem, edumazet, kuba, pabeni, robh,
	krzk+dt, conor+dt, harini.katakam, andrew, hkallweit1, linux,
	michal.simek
  Cc: netdev, devicetree, linux-kernel, linux-arm-kernel

On 17/05/2024 07:47, Vineeth Karumanchi wrote:
> Add input clock support to gmii_to_rgmii IP.
> Add "clocks" bindings for the input clock.

Both sentences look like saying the same... confused.

> 
> Signed-off-by: Vineeth Karumanchi <vineeth.karumanchi@amd.com>
> ---
>  Documentation/devicetree/bindings/net/xlnx,gmii-to-rgmii.yaml | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/net/xlnx,gmii-to-rgmii.yaml b/Documentation/devicetree/bindings/net/xlnx,gmii-to-rgmii.yaml
> index 0f781dac6717..672bff6dae28 100644
> --- a/Documentation/devicetree/bindings/net/xlnx,gmii-to-rgmii.yaml
> +++ b/Documentation/devicetree/bindings/net/xlnx,gmii-to-rgmii.yaml
> @@ -31,6 +31,9 @@ properties:
>    phy-handle:
>      $ref: ethernet-controller.yaml#/properties/phy-handle
>  
> +  clocks:
> +    description: 200/375 MHz free-running clock is used as input clock.

Missing constraints. Probably you wanted items above description.


Best regards,
Krzysztof


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

end of thread, other threads:[~2024-05-19 17:33 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-05-17  5:47 [PATCH net-next v2 0/2] net: xilinx_gmii2rgmii: Add clock support Vineeth Karumanchi
2024-05-17  5:47 ` Vineeth Karumanchi
2024-05-17  5:47 ` [PATCH net-next v2 1/2] dt-bindings: " Vineeth Karumanchi
2024-05-17  5:47   ` Vineeth Karumanchi
2024-05-19 17:33   ` Krzysztof Kozlowski
2024-05-19 17:33     ` Krzysztof Kozlowski
2024-05-17  5:47 ` [PATCH net-next v2 2/2] net: phy: xilinx-gmii2rgmii: Adopt " Vineeth Karumanchi
2024-05-17  5:47   ` Vineeth Karumanchi
2024-05-17 19:25 ` [PATCH net-next v2 0/2] net: xilinx_gmii2rgmii: Add " Jakub Kicinski
2024-05-17 19:25   ` Jakub Kicinski

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.