[PATCH] powerpc: Use generic PHY driver for Marvell 88E1111 PHY on GE Fanuc SBC610

[PATCH] powerpc: Use generic PHY driver for Marvell 88E1111 PHY on GE Fanuc SBC610

[Martyn Welch]

The Marvell PHY driver is currently being used for the 88E1111 on the
SBC610. This driver is causing the link to run in 10/Half mode, the generic
PHY driver is correctly configuring the PHY as 1000/Full.

Edit default config to use generic PHY driver.
---

 arch/powerpc/configs/86xx/gef_sbc610_defconfig |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/powerpc/configs/86xx/gef_sbc610_defconfig b/arch/powerpc/configs/86xx/gef_sbc610_defconfig
index 2da13e0..07ccaf8 100644
--- a/arch/powerpc/configs/86xx/gef_sbc610_defconfig
+++ b/arch/powerpc/configs/86xx/gef_sbc610_defconfig
@@ -838,7 +838,7 @@ CONFIG_PHYLIB=y
 #
 # MII PHY device drivers
 #
-CONFIG_MARVELL_PHY=y
+# CONFIG_MARVELL_PHY is not set
 # CONFIG_DAVICOM_PHY is not set
 # CONFIG_QSEMI_PHY is not set
 # CONFIG_LXT_PHY is not set

[PATCH] powerpc: Use generic PHY driver for Marvell 88E1111 PHY on GE Fanuc SBC610

[Martyn Welch]

The Marvell PHY driver is currently being used for the 88E1111 on the
SBC610. This driver is causing the link to run in 10/Half mode, the generic
PHY driver is correctly configuring the PHY as 1000/Full.

Edit default config to use generic PHY driver.

Signed-off-by: Martyn Welch <martyn.welch@gefanuc.com>
---

Agh! My mistake - forgot to add "Signed-off", sorry.

 arch/powerpc/configs/86xx/gef_sbc610_defconfig |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/powerpc/configs/86xx/gef_sbc610_defconfig b/arch/powerpc/configs/86xx/gef_sbc610_defconfig
index 2da13e0..07ccaf8 100644
--- a/arch/powerpc/configs/86xx/gef_sbc610_defconfig
+++ b/arch/powerpc/configs/86xx/gef_sbc610_defconfig
@@ -838,7 +838,7 @@ CONFIG_PHYLIB=y
 #
 # MII PHY device drivers
 #
-CONFIG_MARVELL_PHY=y
+# CONFIG_MARVELL_PHY is not set
 # CONFIG_DAVICOM_PHY is not set
 # CONFIG_QSEMI_PHY is not set
 # CONFIG_LXT_PHY is not set

Re: [PATCH] powerpc: Use generic PHY driver for Marvell 88E1111 PHY on GE Fanuc SBC610

[Kumar Gala]

On Nov 18, 2008, at 4:55 AM, Martyn Welch wrote:

> The Marvell PHY driver is currently being used for the 88E1111 on the
> SBC610. This driver is causing the link to run in 10/Half mode, the  
> generic
> PHY driver is correctly configuring the PHY as 1000/Full.
>
> Edit default config to use generic PHY driver.
>
> Signed-off-by: Martyn Welch <martyn.welch@gefanuc.com>
> ---
>
> Agh! My mistake - forgot to add "Signed-off", sorry.
>
> arch/powerpc/configs/86xx/gef_sbc610_defconfig |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)

hmm.. why not fix the issue with the 88e1111 driver.

- k

Re: [PATCH] powerpc: Use generic PHY driver for Marvell 88E1111 PHY on GE Fanuc SBC610

[Martyn Welch]

On Tue, 18 Nov 2008 07:39:21 -0600
Kumar Gala <galak@kernel.crashing.org> wrote:
> 
> On Nov 18, 2008, at 4:55 AM, Martyn Welch wrote:
> 
> > The Marvell PHY driver is currently being used for the 88E1111 on the
> > SBC610. This driver is causing the link to run in 10/Half mode, the  
> > generic
> > PHY driver is correctly configuring the PHY as 1000/Full.
> >
> > Edit default config to use generic PHY driver.
> >
> > Signed-off-by: Martyn Welch <martyn.welch@gefanuc.com>
> > ---
> >
> > Agh! My mistake - forgot to add "Signed-off", sorry.
> >
> > arch/powerpc/configs/86xx/gef_sbc610_defconfig |    2 +-
> > 1 files changed, 1 insertions(+), 1 deletions(-)
> 
> hmm.. why not fix the issue with the 88e1111 driver.
> 

Two reasons really:

1) We don't use any of the non-generic features of the PHY.
2) Networking is really _not_ my forte.

As a result, it's not itching ;-)

Martyn

-- 
Martyn Welch MEng MPhil MIET (Principal Software Engineer)   T:+44(0)1327322748
GE Fanuc Intelligent Platforms Ltd,        |Registered in England and Wales
Tove Valley Business Park, Towcester,      |(3828642) at 100 Barbirolli Square,
Northants, NN12 6PF, UK T:+44(0)1327359444 |Manchester,M2 3AB  VAT:GB 729849476

Re: [PATCH] powerpc: Use generic PHY driver for Marvell 88E1111 PHY on GE Fanuc SBC610

[Kumar Gala]

On Nov 18, 2008, at 4:55 AM, Martyn Welch wrote:

> The Marvell PHY driver is currently being used for the 88E1111 on the
> SBC610. This driver is causing the link to run in 10/Half mode, the  
> generic
> PHY driver is correctly configuring the PHY as 1000/Full.
>
> Edit default config to use generic PHY driver.
>
> Signed-off-by: Martyn Welch <martyn.welch@gefanuc.com>
> ---
>
> Agh! My mistake - forgot to add "Signed-off", sorry.
>
> arch/powerpc/configs/86xx/gef_sbc610_defconfig |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)

applied to merge.

(I still think you should fix it to work with the proper driver).

- k