* [U-Boot] [PATCH] ARM: imx6q_logic: Correct phy fixup for broken ethernet
@ 2019-01-12 23:32 Adam Ford
2019-02-16 10:35 ` [U-Boot] " sbabic at denx.de
2019-04-03 4:58 ` [U-Boot] [PATCH] " Jagan Teki
0 siblings, 2 replies; 4+ messages in thread
From: Adam Ford @ 2019-01-12 23:32 UTC (permalink / raw)
To: u-boot
The Ethernet has been broken for some time. This patch unifies
this board with a few others that use a similar approach to
enabling phy. This fixes ar8031 Ethernet controller so it works.
Signed-off-by: Adam Ford <aford173@gmail.com>
diff --git a/board/logicpd/imx6/imx6logic.c b/board/logicpd/imx6/imx6logic.c
index cda76a8084..eb4849117d 100644
--- a/board/logicpd/imx6/imx6logic.c
+++ b/board/logicpd/imx6/imx6logic.c
@@ -60,57 +60,6 @@ static iomux_v3_cfg_t const uart3_pads[] = {
MX6_PAD_EIM_EB3__UART3_RTS_B | MUX_PAD_CTRL(UART_PAD_CTRL),
};
-#ifndef CONFIG_SPL_BUILD
-static void fixup_enet_clock(void)
-{
- struct iomuxc *iomuxc_regs = (struct iomuxc *)IOMUXC_BASE_ADDR;
- struct gpio_desc nint;
- struct gpio_desc reset;
- int ret;
-
- /* Set Ref Clock to 50 MHz */
- enable_fec_anatop_clock(0, ENET_50MHZ);
-
- /* Set GPIO_16 as ENET_REF_CLK_OUT */
- setbits_le32(&iomuxc_regs->gpr[1], IOMUXC_GPR1_ENET_CLK_SEL_MASK);
-
- /* Request GPIO Pins to reset Ethernet with new clock */
- ret = dm_gpio_lookup_name("GPIO4_7", &nint);
- if (ret) {
- printf("Unable to lookup GPIO4_7\n");
- return;
- }
-
- ret = dm_gpio_request(&nint, "eth0_nInt");
- if (ret) {
- printf("Unable to request eth0_nInt\n");
- return;
- }
-
- /* Ensure nINT is input or PHY won't startup */
- dm_gpio_set_dir_flags(&nint, GPIOD_IS_IN);
-
- ret = dm_gpio_lookup_name("GPIO4_9", &reset);
- if (ret) {
- printf("Unable to lookup GPIO4_9\n");
- return;
- }
-
- ret = dm_gpio_request(&reset, "eth0_reset");
- if (ret) {
- printf("Unable to request eth0_reset\n");
- return;
- }
-
- /* Reset LAN8710A PHY */
- dm_gpio_set_dir_flags(&reset, GPIOD_IS_OUT);
- dm_gpio_set_value(&reset, 0);
- udelay(150);
- dm_gpio_set_value(&reset, 1);
- mdelay(50);
-}
-#endif
-
static void setup_iomux_uart(void)
{
imx_iomux_v3_setup_multiple_pads(uart1_pads, ARRAY_SIZE(uart1_pads));
@@ -141,8 +90,33 @@ static void setup_nand_pins(void)
imx_iomux_v3_setup_multiple_pads(nand_pads, ARRAY_SIZE(nand_pads));
}
+static int ar8031_phy_fixup(struct phy_device *phydev)
+{
+ unsigned short val;
+
+ /* To enable AR8031 output a 125MHz clk from CLK_25M */
+ phy_write(phydev, MDIO_DEVAD_NONE, 0xd, 0x7);
+ phy_write(phydev, MDIO_DEVAD_NONE, 0xe, 0x8016);
+ phy_write(phydev, MDIO_DEVAD_NONE, 0xd, 0x4007);
+
+ val = phy_read(phydev, MDIO_DEVAD_NONE, 0xe);
+ val &= 0xffe3;
+ val |= 0x18;
+ phy_write(phydev, MDIO_DEVAD_NONE, 0xe, val);
+
+ /* introduce tx clock delay */
+ phy_write(phydev, MDIO_DEVAD_NONE, 0x1d, 0x5);
+ val = phy_read(phydev, MDIO_DEVAD_NONE, 0x1e);
+ val |= 0x0100;
+ phy_write(phydev, MDIO_DEVAD_NONE, 0x1e, val);
+
+ return 0;
+}
+
int board_phy_config(struct phy_device *phydev)
{
+ ar8031_phy_fixup(phydev);
+
if (phydev->drv->config)
phydev->drv->config(phydev);
@@ -160,9 +134,6 @@ int overwrite_console(void)
int board_early_init_f(void)
{
-#ifndef CONFIG_SPL_BUILD
- fixup_enet_clock();
-#endif
setup_iomux_uart();
setup_nand_pins();
return 0;
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [U-Boot] ARM: imx6q_logic: Correct phy fixup for broken ethernet
2019-01-12 23:32 [U-Boot] [PATCH] ARM: imx6q_logic: Correct phy fixup for broken ethernet Adam Ford
@ 2019-02-16 10:35 ` sbabic at denx.de
2019-04-03 4:58 ` [U-Boot] [PATCH] " Jagan Teki
1 sibling, 0 replies; 4+ messages in thread
From: sbabic at denx.de @ 2019-02-16 10:35 UTC (permalink / raw)
To: u-boot
> The Ethernet has been broken for some time. This patch unifies
> this board with a few others that use a similar approach to
> enabling phy. This fixes ar8031 Ethernet controller so it works.
> Signed-off-by: Adam Ford <aford173@gmail.com>
> diff --git a/board/logicpd/imx6/imx6logic.c b/board/logicpd/imx6/imx6logic.c
> index cda76a8084..eb4849117d 100644
> --- a/board/logicpd/imx6/imx6logic.c
> +++ b/board/logicpd/imx6/imx6logic.c
> @@ -60,57 +60,6 @@ static iomux_v3_cfg_t const uart3_pads[] = {
> MX6_PAD_EIM_EB3__UART3_RTS_B | MUX_PAD_CTRL(UART_PAD_CTRL),
> };
>
> -#ifndef CONFIG_SPL_BUILD
> -static void fixup_enet_clock(void)
> -{
> - struct iomuxc *iomuxc_regs = (struct iomuxc *)IOMUXC_BASE_ADDR;
> - struct gpio_desc nint;
> - struct gpio_desc reset;
> - int ret;
> -
> - /* Set Ref Clock to 50 MHz */
> - enable_fec_anatop_clock(0, ENET_50MHZ);
> -
> - /* Set GPIO_16 as ENET_REF_CLK_OUT */
> - setbits_le32(&iomuxc_regs->gpr[1], IOMUXC_GPR1_ENET_CLK_SEL_MASK);
> -
> - /* Request GPIO Pins to reset Ethernet with new clock */
> - ret = dm_gpio_lookup_name("GPIO4_7", &nint);
> - if (ret) {
> - printf("Unable to lookup GPIO4_7\n");
> - return;
> - }
> -
> - ret = dm_gpio_request(&nint, "eth0_nInt");
> - if (ret) {
> - printf("Unable to request eth0_nInt\n");
> - return;
> - }
> -
> - /* Ensure nINT is input or PHY won't startup */
> - dm_gpio_set_dir_flags(&nint, GPIOD_IS_IN);
> -
> - ret = dm_gpio_lookup_name("GPIO4_9", &reset);
> - if (ret) {
> - printf("Unable to lookup GPIO4_9\n");
> - return;
> - }
> -
> - ret = dm_gpio_request(&reset, "eth0_reset");
> - if (ret) {
> - printf("Unable to request eth0_reset\n");
> - return;
> - }
> -
> - /* Reset LAN8710A PHY */
> - dm_gpio_set_dir_flags(&reset, GPIOD_IS_OUT);
> - dm_gpio_set_value(&reset, 0);
> - udelay(150);
> - dm_gpio_set_value(&reset, 1);
> - mdelay(50);
> -}
> -#endif
> -
> static void setup_iomux_uart(void)
> {
> imx_iomux_v3_setup_multiple_pads(uart1_pads, ARRAY_SIZE(uart1_pads));
> @@ -141,8 +90,33 @@ static void setup_nand_pins(void)
> imx_iomux_v3_setup_multiple_pads(nand_pads, ARRAY_SIZE(nand_pads));
> }
>
> +static int ar8031_phy_fixup(struct phy_device *phydev)
> +{
> + unsigned short val;
> +
> + /* To enable AR8031 output a 125MHz clk from CLK_25M */
> + phy_write(phydev, MDIO_DEVAD_NONE, 0xd, 0x7);
> + phy_write(phydev, MDIO_DEVAD_NONE, 0xe, 0x8016);
> + phy_write(phydev, MDIO_DEVAD_NONE, 0xd, 0x4007);
> +
> + val = phy_read(phydev, MDIO_DEVAD_NONE, 0xe);
> + val &= 0xffe3;
> + val |= 0x18;
> + phy_write(phydev, MDIO_DEVAD_NONE, 0xe, val);
> +
> + /* introduce tx clock delay */
> + phy_write(phydev, MDIO_DEVAD_NONE, 0x1d, 0x5);
> + val = phy_read(phydev, MDIO_DEVAD_NONE, 0x1e);
> + val |= 0x0100;
> + phy_write(phydev, MDIO_DEVAD_NONE, 0x1e, val);
> +
> + return 0;
> +}
> +
> int board_phy_config(struct phy_device *phydev)
> {
> + ar8031_phy_fixup(phydev);
> +
> if (phydev->drv->config)
> phydev->drv->config(phydev);
>
> @@ -160,9 +134,6 @@ int overwrite_console(void)
>
> int board_early_init_f(void)
> {
> -#ifndef CONFIG_SPL_BUILD
> - fixup_enet_clock();
> -#endif
> setup_iomux_uart();
> setup_nand_pins();
> return 0;
Applied to u-boot-imx, master, thanks !
Best regards,
Stefano Babic
--
=====================================================================
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================
^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] ARM: imx6q_logic: Correct phy fixup for broken ethernet
2019-01-12 23:32 [U-Boot] [PATCH] ARM: imx6q_logic: Correct phy fixup for broken ethernet Adam Ford
2019-02-16 10:35 ` [U-Boot] " sbabic at denx.de
@ 2019-04-03 4:58 ` Jagan Teki
2019-04-03 11:36 ` Adam Ford
1 sibling, 1 reply; 4+ messages in thread
From: Jagan Teki @ 2019-04-03 4:58 UTC (permalink / raw)
To: u-boot
Hi Adam,
On Sun, Jan 13, 2019 at 5:02 AM Adam Ford <aford173@gmail.com> wrote:
>
> The Ethernet has been broken for some time. This patch unifies
> this board with a few others that use a similar approach to
> enabling phy. This fixes ar8031 Ethernet controller so it works.
>
> Signed-off-by: Adam Ford <aford173@gmail.com>
>
> diff --git a/board/logicpd/imx6/imx6logic.c b/board/logicpd/imx6/imx6logic.c
> index cda76a8084..eb4849117d 100644
> --- a/board/logicpd/imx6/imx6logic.c
> +++ b/board/logicpd/imx6/imx6logic.c
> @@ -60,57 +60,6 @@ static iomux_v3_cfg_t const uart3_pads[] = {
> MX6_PAD_EIM_EB3__UART3_RTS_B | MUX_PAD_CTRL(UART_PAD_CTRL),
> };
>
> -#ifndef CONFIG_SPL_BUILD
> -static void fixup_enet_clock(void)
> -{
> - struct iomuxc *iomuxc_regs = (struct iomuxc *)IOMUXC_BASE_ADDR;
> - struct gpio_desc nint;
> - struct gpio_desc reset;
> - int ret;
> -
> - /* Set Ref Clock to 50 MHz */
> - enable_fec_anatop_clock(0, ENET_50MHZ);
I just wonder why the fec working w/o ene_ref set rate to 50MHz, any idea?
^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] ARM: imx6q_logic: Correct phy fixup for broken ethernet
2019-04-03 4:58 ` [U-Boot] [PATCH] " Jagan Teki
@ 2019-04-03 11:36 ` Adam Ford
0 siblings, 0 replies; 4+ messages in thread
From: Adam Ford @ 2019-04-03 11:36 UTC (permalink / raw)
To: u-boot
On Tue, Apr 2, 2019 at 11:58 PM Jagan Teki <jagan@amarulasolutions.com> wrote:
>
> Hi Adam,
>
> On Sun, Jan 13, 2019 at 5:02 AM Adam Ford <aford173@gmail.com> wrote:
> >
> > The Ethernet has been broken for some time. This patch unifies
> > this board with a few others that use a similar approach to
> > enabling phy. This fixes ar8031 Ethernet controller so it works.
> >
> > Signed-off-by: Adam Ford <aford173@gmail.com>
> >
> > diff --git a/board/logicpd/imx6/imx6logic.c b/board/logicpd/imx6/imx6logic.c
> > index cda76a8084..eb4849117d 100644
> > --- a/board/logicpd/imx6/imx6logic.c
> > +++ b/board/logicpd/imx6/imx6logic.c
> > @@ -60,57 +60,6 @@ static iomux_v3_cfg_t const uart3_pads[] = {
> > MX6_PAD_EIM_EB3__UART3_RTS_B | MUX_PAD_CTRL(UART_PAD_CTRL),
> > };
> >
> > -#ifndef CONFIG_SPL_BUILD
> > -static void fixup_enet_clock(void)
> > -{
> > - struct iomuxc *iomuxc_regs = (struct iomuxc *)IOMUXC_BASE_ADDR;
> > - struct gpio_desc nint;
> > - struct gpio_desc reset;
> > - int ret;
> > -
> > - /* Set Ref Clock to 50 MHz */
> > - enable_fec_anatop_clock(0, ENET_50MHZ);
>
> I just wonder why the fec working w/o ene_ref set rate to 50MHz, any idea?
The original source was setup for a different ethernet controller
100Mb instead of 1000Mb. I got a bit ahead of myself when I pushed
the original board to U-Boot, but I thought it was going to market
sooner. That revision of the board was never released, and eventually
Logic PD migrated to the AR8031 controller, so the original code was
completely wrong. The patch you're referencing was to make the AR8031
controller actually work. Since the older hardware was never
released, I didn't mention it in the commit.
adam
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-04-03 11:36 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-01-12 23:32 [U-Boot] [PATCH] ARM: imx6q_logic: Correct phy fixup for broken ethernet Adam Ford
2019-02-16 10:35 ` [U-Boot] " sbabic at denx.de
2019-04-03 4:58 ` [U-Boot] [PATCH] " Jagan Teki
2019-04-03 11:36 ` Adam Ford
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.