* [PATCH v2] net: zynq: Add support for PHY configuration in SGMII mode
@ 2021-12-14 12:42 Michal Simek
2021-12-14 14:39 ` Ramon Fried
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Michal Simek @ 2021-12-14 12:42 UTC (permalink / raw)
To: u-boot, git; +Cc: Joe Hershberger, Michal Simek, Ramon Fried
SGMII configuration depends on proper GT setting that's why when node has
phys property call PSGTR driver to configure it properly.
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---
Changes in v2:
- Handle also cases where phy reference doesn't exist which means
that u-boot doesn't need to configure it (configured via psu_init for
example)
- Error out where reference exists but driver is not compiled in - reported
by Sean
drivers/net/zynq_gem.c | 19 +++++++++++++++++++
1 file changed, 19 insertions(+)
diff --git a/drivers/net/zynq_gem.c b/drivers/net/zynq_gem.c
index 5cbe8d28304b..ac658cd6b5e9 100644
--- a/drivers/net/zynq_gem.c
+++ b/drivers/net/zynq_gem.c
@@ -12,6 +12,7 @@
#include <common.h>
#include <cpu_func.h>
#include <dm.h>
+#include <generic-phy.h>
#include <log.h>
#include <net.h>
#include <netdev.h>
@@ -716,6 +717,24 @@ static int zynq_gem_probe(struct udevice *dev)
struct zynq_gem_priv *priv = dev_get_priv(dev);
int ret;
+ if (priv->interface == PHY_INTERFACE_MODE_SGMII) {
+ struct phy phy;
+
+ ret = generic_phy_get_by_index(dev, 0, &phy);
+ if (!ret) {
+ ret = generic_phy_init(&phy);
+ if (ret)
+ return ret;
+
+ ret = generic_phy_power_on(&phy);
+ if (ret)
+ return ret;
+ } else if (ret != -ENOENT) {
+ debug("could not get phy (err %d)\n", ret);
+ return ret;
+ }
+ }
+
ret = zynq_gem_reset_init(dev);
if (ret)
return ret;
--
2.34.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2] net: zynq: Add support for PHY configuration in SGMII mode
2021-12-14 12:42 [PATCH v2] net: zynq: Add support for PHY configuration in SGMII mode Michal Simek
@ 2021-12-14 14:39 ` Ramon Fried
2021-12-14 15:54 ` Sean Anderson
2021-12-15 9:54 ` Michal Simek
2 siblings, 0 replies; 4+ messages in thread
From: Ramon Fried @ 2021-12-14 14:39 UTC (permalink / raw)
To: Michal Simek; +Cc: U-Boot Mailing List, git, Joe Hershberger, Michal Simek
On Tue, Dec 14, 2021 at 2:42 PM Michal Simek <michal.simek@xilinx.com> wrote:
>
> SGMII configuration depends on proper GT setting that's why when node has
> phys property call PSGTR driver to configure it properly.
>
> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
> ---
>
> Changes in v2:
> - Handle also cases where phy reference doesn't exist which means
> that u-boot doesn't need to configure it (configured via psu_init for
> example)
> - Error out where reference exists but driver is not compiled in - reported
> by Sean
>
> drivers/net/zynq_gem.c | 19 +++++++++++++++++++
> 1 file changed, 19 insertions(+)
>
> diff --git a/drivers/net/zynq_gem.c b/drivers/net/zynq_gem.c
> index 5cbe8d28304b..ac658cd6b5e9 100644
> --- a/drivers/net/zynq_gem.c
> +++ b/drivers/net/zynq_gem.c
> @@ -12,6 +12,7 @@
> #include <common.h>
> #include <cpu_func.h>
> #include <dm.h>
> +#include <generic-phy.h>
> #include <log.h>
> #include <net.h>
> #include <netdev.h>
> @@ -716,6 +717,24 @@ static int zynq_gem_probe(struct udevice *dev)
> struct zynq_gem_priv *priv = dev_get_priv(dev);
> int ret;
>
> + if (priv->interface == PHY_INTERFACE_MODE_SGMII) {
> + struct phy phy;
> +
> + ret = generic_phy_get_by_index(dev, 0, &phy);
> + if (!ret) {
> + ret = generic_phy_init(&phy);
> + if (ret)
> + return ret;
> +
> + ret = generic_phy_power_on(&phy);
> + if (ret)
> + return ret;
> + } else if (ret != -ENOENT) {
> + debug("could not get phy (err %d)\n", ret);
> + return ret;
> + }
> + }
> +
> ret = zynq_gem_reset_init(dev);
> if (ret)
> return ret;
> --
> 2.34.1
>
Reviewed-by: Ramon Fried <rfried.dev@gmail.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] net: zynq: Add support for PHY configuration in SGMII mode
2021-12-14 12:42 [PATCH v2] net: zynq: Add support for PHY configuration in SGMII mode Michal Simek
2021-12-14 14:39 ` Ramon Fried
@ 2021-12-14 15:54 ` Sean Anderson
2021-12-15 9:54 ` Michal Simek
2 siblings, 0 replies; 4+ messages in thread
From: Sean Anderson @ 2021-12-14 15:54 UTC (permalink / raw)
To: Michal Simek, u-boot, git; +Cc: Joe Hershberger, Michal Simek, Ramon Fried
On 12/14/21 7:42 AM, Michal Simek wrote:
> SGMII configuration depends on proper GT setting that's why when node has
> phys property call PSGTR driver to configure it properly.
>
> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
> ---
>
> Changes in v2:
> - Handle also cases where phy reference doesn't exist which means
> that u-boot doesn't need to configure it (configured via psu_init for
> example)
> - Error out where reference exists but driver is not compiled in - reported
> by Sean
>
> drivers/net/zynq_gem.c | 19 +++++++++++++++++++
> 1 file changed, 19 insertions(+)
>
> diff --git a/drivers/net/zynq_gem.c b/drivers/net/zynq_gem.c
> index 5cbe8d28304b..ac658cd6b5e9 100644
> --- a/drivers/net/zynq_gem.c
> +++ b/drivers/net/zynq_gem.c
> @@ -12,6 +12,7 @@
> #include <common.h>
> #include <cpu_func.h>
> #include <dm.h>
> +#include <generic-phy.h>
> #include <log.h>
> #include <net.h>
> #include <netdev.h>
> @@ -716,6 +717,24 @@ static int zynq_gem_probe(struct udevice *dev)
> struct zynq_gem_priv *priv = dev_get_priv(dev);
> int ret;
>
> + if (priv->interface == PHY_INTERFACE_MODE_SGMII) {
> + struct phy phy;
> +
> + ret = generic_phy_get_by_index(dev, 0, &phy);
> + if (!ret) {
> + ret = generic_phy_init(&phy);
> + if (ret)
> + return ret;
> +
> + ret = generic_phy_power_on(&phy);
> + if (ret)
> + return ret;
> + } else if (ret != -ENOENT) {
> + debug("could not get phy (err %d)\n", ret);
> + return ret;
> + }
> + }
> +
> ret = zynq_gem_reset_init(dev);
> if (ret)
> return ret;
>
Reviewed-by: Sean Anderson <sean.anderson@seco.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] net: zynq: Add support for PHY configuration in SGMII mode
2021-12-14 12:42 [PATCH v2] net: zynq: Add support for PHY configuration in SGMII mode Michal Simek
2021-12-14 14:39 ` Ramon Fried
2021-12-14 15:54 ` Sean Anderson
@ 2021-12-15 9:54 ` Michal Simek
2 siblings, 0 replies; 4+ messages in thread
From: Michal Simek @ 2021-12-15 9:54 UTC (permalink / raw)
To: U-Boot, git; +Cc: Joe Hershberger, Ramon Fried
út 14. 12. 2021 v 13:42 odesílatel Michal Simek
<michal.simek@xilinx.com> napsal:
>
> SGMII configuration depends on proper GT setting that's why when node has
> phys property call PSGTR driver to configure it properly.
>
> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
> ---
>
> Changes in v2:
> - Handle also cases where phy reference doesn't exist which means
> that u-boot doesn't need to configure it (configured via psu_init for
> example)
> - Error out where reference exists but driver is not compiled in - reported
> by Sean
>
> drivers/net/zynq_gem.c | 19 +++++++++++++++++++
> 1 file changed, 19 insertions(+)
>
> diff --git a/drivers/net/zynq_gem.c b/drivers/net/zynq_gem.c
> index 5cbe8d28304b..ac658cd6b5e9 100644
> --- a/drivers/net/zynq_gem.c
> +++ b/drivers/net/zynq_gem.c
> @@ -12,6 +12,7 @@
> #include <common.h>
> #include <cpu_func.h>
> #include <dm.h>
> +#include <generic-phy.h>
> #include <log.h>
> #include <net.h>
> #include <netdev.h>
> @@ -716,6 +717,24 @@ static int zynq_gem_probe(struct udevice *dev)
> struct zynq_gem_priv *priv = dev_get_priv(dev);
> int ret;
>
> + if (priv->interface == PHY_INTERFACE_MODE_SGMII) {
> + struct phy phy;
> +
> + ret = generic_phy_get_by_index(dev, 0, &phy);
> + if (!ret) {
> + ret = generic_phy_init(&phy);
> + if (ret)
> + return ret;
> +
> + ret = generic_phy_power_on(&phy);
> + if (ret)
> + return ret;
> + } else if (ret != -ENOENT) {
> + debug("could not get phy (err %d)\n", ret);
> + return ret;
> + }
> + }
> +
> ret = zynq_gem_reset_init(dev);
> if (ret)
> return ret;
> --
> 2.34.1
>
Please ignore this v2. I need to send v3 because there must be a reset
between init and power on.
I will send a new version on phy driver with this function.
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP/Versal SoCs
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-12-15 9:54 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-14 12:42 [PATCH v2] net: zynq: Add support for PHY configuration in SGMII mode Michal Simek
2021-12-14 14:39 ` Ramon Fried
2021-12-14 15:54 ` Sean Anderson
2021-12-15 9:54 ` Michal Simek
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.