diff mbox

[U-Boot] Bananapi: Fix ethernet device initialization

Message ID 1433682296-41814-1-git-send-email-maxin.john@enea.com
State Superseded
Delegated to: Joe Hershberger
Headers show

Commit Message

maxin.john@enea.com June 7, 2015, 1:04 p.m. UTC
Ethernet device initialization fails with the following message:
Net:   PH: dir_output: error: gpio PH23 not reserved
Phy 1 not found
PHY reset timed out
eth0: ethernet@01c50000

Tested on Banana Pi board.

Signed-off-by: Maxin B. John <maxin.john@enea.com>
---
 arch/arm/cpu/armv7/sunxi/board.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Paul Kocialkowski June 7, 2015, 1:12 p.m. UTC | #1
Hi,

> Ethernet device initialization fails with the following message:
> Net:   PH: dir_output: error: gpio PH23 not reserved
> Phy 1 not found
> PHY reset timed out
> eth0: ethernet@01c50000
> 
> Tested on Banana Pi board.
> 
> Signed-off-by: Maxin B. John <maxin.john@enea.com>
> ---
>  arch/arm/cpu/armv7/sunxi/board.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm/cpu/armv7/sunxi/board.c b/arch/arm/cpu/armv7/sunxi/board.c
> index a82c8b9..030a711 100644
> --- a/arch/arm/cpu/armv7/sunxi/board.c
> +++ b/arch/arm/cpu/armv7/sunxi/board.c
> @@ -223,6 +223,7 @@ int cpu_eth_init(bd_t *bis)
>  	__maybe_unused int rc;
>  
>  #ifdef CONFIG_MACPWR
> +	gpio_request(CONFIG_MACPWR, "MAC Power");

I suggest using the string "macpwr" instead to be consistent with:
board/sunxi/ahci.c:	gpio_request(CONFIG_SATAPWR, "satapwr");

Good find otherwise!

>  	gpio_direction_output(CONFIG_MACPWR, 1);
>  	mdelay(200);
>  #endif
maxin.john@enea.com June 7, 2015, 1:25 p.m. UTC | #2
Hi Paul,

On Sun, Jun 07, 2015 at 03:12:41PM +0200, Paul Kocialkowski wrote:
> Hi,
> 
> > Ethernet device initialization fails with the following message:
> > Net:   PH: dir_output: error: gpio PH23 not reserved
> > Phy 1 not found
> > PHY reset timed out
> > eth0: ethernet@01c50000
> > 
> > Tested on Banana Pi board.
> > 
> > Signed-off-by: Maxin B. John <maxin.john@enea.com>
> > ---
> >  arch/arm/cpu/armv7/sunxi/board.c | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/arch/arm/cpu/armv7/sunxi/board.c b/arch/arm/cpu/armv7/sunxi/board.c
> > index a82c8b9..030a711 100644
> > --- a/arch/arm/cpu/armv7/sunxi/board.c
> > +++ b/arch/arm/cpu/armv7/sunxi/board.c
> > @@ -223,6 +223,7 @@ int cpu_eth_init(bd_t *bis)
> >  	__maybe_unused int rc;
> >  
> >  #ifdef CONFIG_MACPWR
> > +	gpio_request(CONFIG_MACPWR, "MAC Power");
> 
> I suggest using the string "macpwr" instead to be consistent with:
> board/sunxi/ahci.c:	gpio_request(CONFIG_SATAPWR, "satapwr");
> 
> Good find otherwise!

Thanks for the comments. Will send a v2 soon.

> >  	gpio_direction_output(CONFIG_MACPWR, 1);
> >  	mdelay(200);
> >  #endif

Best Regards,
Maxin
diff mbox

Patch

diff --git a/arch/arm/cpu/armv7/sunxi/board.c b/arch/arm/cpu/armv7/sunxi/board.c
index a82c8b9..030a711 100644
--- a/arch/arm/cpu/armv7/sunxi/board.c
+++ b/arch/arm/cpu/armv7/sunxi/board.c
@@ -223,6 +223,7 @@  int cpu_eth_init(bd_t *bis)
 	__maybe_unused int rc;
 
 #ifdef CONFIG_MACPWR
+	gpio_request(CONFIG_MACPWR, "MAC Power");
 	gpio_direction_output(CONFIG_MACPWR, 1);
 	mdelay(200);
 #endif