arm64: zynqmp: Increase reset assert time for TI SGMII PHY
authorHarini Katakam <harini.katakam@amd.com>
Mon, 10 Jul 2023 12:37:32 +0000 (14:37 +0200)
committerMichal Simek <michal.simek@amd.com>
Fri, 21 Jul 2023 07:00:38 +0000 (09:00 +0200)
Increase reset assert time for TI SGMII PHY on KR260 CC starting
6.1 kernel. This PHY does not come out of reset with the existing
100us pulse width as per testing on multiple carrier cards. The reset
is driven via a PCA9570 I2C expander. The expander driver was updated
to an upstream version in 6.1 where gpio_chip _set was optimized.
Delays in earlier kernels may have masked this issue. This is a safe
workaround value for assert pulse width before the discussions are
resolved with TI.

Signed-off-by: Harini Katakam <harini.katakam@amd.com>
Reviewed-by: Radhey Shyam Pandey <radhey.shyam.pandey@amd.com>
Signed-off-by: Michal Simek <michal.simek@amd.com>
Link: https://lore.kernel.org/r/8fb9f17d43a43ef504c9f29006cd686cce8ac98b.1688992653.git.michal.simek@amd.com
arch/arm/dts/zynqmp-sck-kr-g-revA.dts
arch/arm/dts/zynqmp-sck-kr-g-revB.dts

index caaf71d..d318773 100644 (file)
                        ti,tx-internal-delay = <DP83867_RGMIIDCTL_2_75_NS>;
                        ti,fifo-depth = <DP83867_PHYCR_FIFO_DEPTH_4_B_NIB>;
                        ti,dp83867-rxctrl-strap-quirk;
-                       reset-assert-us = <100>;
+                       reset-assert-us = <300>;
                        reset-deassert-us = <280>;
                        reset-gpios = <&slg7xl45106 5 GPIO_ACTIVE_LOW>;
                };
index f9d8755..69dba07 100644 (file)
                        ti,tx-internal-delay = <DP83867_RGMIIDCTL_2_75_NS>;
                        ti,fifo-depth = <DP83867_PHYCR_FIFO_DEPTH_4_B_NIB>;
                        ti,dp83867-rxctrl-strap-quirk;
-                       reset-assert-us = <100>;
+                       reset-assert-us = <300>;
                        reset-deassert-us = <280>;
                        reset-gpios = <&slg7xl45106 5 GPIO_ACTIVE_LOW>;
                };