[U-Boot] [PATCH 1/2] arm/km: add mv88e6352 configuration for kmnusa

Prafulla Wadaskar prafulla at marvell.com
Fri Aug 17 10:53:44 CEST 2012



> -----Original Message-----
> From: Valentin Longchamp [mailto:valentin.longchamp at keymile.com]
> Sent: 17 August 2012 14:13
> To: Prafulla Wadaskar
> Cc: u-boot at lists.denx.de; holger.brunck at keymile.com
> Subject: Re: [PATCH 1/2] arm/km: add mv88e6352 configuration for
> kmnusa
> 
> On 08/17/2012 09:59 AM, Prafulla Wadaskar wrote:
> >
> >
> >> -----Original Message-----
> >> From: Valentin Longchamp [mailto:valentin.longchamp at keymile.com]
> >> Sent: 16 August 2012 16:55
> >> To: u-boot at lists.denx.de
> >> Cc: holger.brunck at keymile.com; Prafulla Wadaskar;
> >> valentin.longchamp at keymile.com
> >> Subject: [PATCH 1/2] arm/km: add mv88e6352 configuration for kmnusa
> >>
> >> The kmnusa board uses a mv88e6352 switch that is connected to the
> main
> >> eth interface of the kirkwood. Therefore the switch must be
> configured
> >> so that the kirkwood's egiga eth inferface can be used.
> >>
> >> Signed-off-by: Valentin Longchamp <valentin.longchamp at keymile.com>
> >> Cc: Holger Brunck <holger.brunck at keymile.com>
> >> Cc: Prafulla Wadaskar <prafulla at marvell.com>
> >> ---
> >>  board/keymile/km_arm/km_arm.c |   63
> >> +++++++++++++++++++++++++++++++++++++++++
> >>  1 files changed, 63 insertions(+), 0 deletions(-)
> >>
> >> diff --git a/board/keymile/km_arm/km_arm.c
> >> b/board/keymile/km_arm/km_arm.c
> >> index 2b2ca39..c0069c1 100644
> >> --- a/board/keymile/km_arm/km_arm.c
> >> +++ b/board/keymile/km_arm/km_arm.c
> >> @@ -385,6 +385,69 @@ void reset_phy(void)
> >>  	/* reset the phy */
> >>  	miiphy_reset(name, CONFIG_PHY_BASE_ADR);
> >>  }
> >> +#elif defined(CONFIG_KM_PIGGY4_88E6352)
> >> +
> >> +#include <mv88e6352.h>
> >> +
> >> +#if defined(CONFIG_KM_NUSA)
> >> +struct mv88e_sw_reg extsw_conf[] = {
> >> +	/* port 0, PIGY4, autoneg */
> >> +	/* first the fix for the 1000Mbits Autoneg, this is from
> >> +	 * a Marvell errata, the regs are undocumented */
> >
> > May you please follow standard c-style commenting?
> >
> 
> OK, I will change this comment. Is it OK if I just repost this updated
> patch as
> a reply to this message since it's a very minor change or do you need
> the whole
> series ?

I think v2 for this patch only should work.

Regards...
Prafulla . . .


More information about the U-Boot mailing list