> -----Original Message----- > From: Valentin Longchamp [mailto:valentin.longch...@keymile.com] > Sent: 17 August 2012 14:13 > To: Prafulla Wadaskar > Cc: u-boot@lists.denx.de; holger.bru...@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.longch...@keymile.com] > >> Sent: 16 August 2012 16:55 > >> To: u-boot@lists.denx.de > >> Cc: holger.bru...@keymile.com; Prafulla Wadaskar; > >> valentin.longch...@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.longch...@keymile.com> > >> Cc: Holger Brunck <holger.bru...@keymile.com> > >> Cc: Prafulla Wadaskar <prafu...@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 . . . _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
