> -----Original Message----- > From: Valentin Longchamp [mailto:valentin.longch...@keymile.com] > Sent: 21 June 2012 18:40 > To: Prafulla Wadaskar > Cc: prafu...@mavell.com; holger.bru...@keymile.com; u- > b...@lists.denx.de > Subject: Re: [U-Boot] [PATCH 12/20] arm/km: add support for external > switchconfiguration > > Hi Prafulla, > > On 06/12/2012 06:39 AM, Prafulla Wadaskar wrote: > > > > > >> -----Original Message----- > >> From: u-boot-boun...@lists.denx.de [mailto:u-boot- > >> boun...@lists.denx.de] On Behalf Of Valentin Longchamp > >> Sent: 07 June 2012 15:37 > >> To: prafu...@mavell.com > >> Cc: Valentin Longchamp; holger.bru...@keymile.com; u- > >> b...@lists.denx.de > >> Subject: [U-Boot] [PATCH 12/20] arm/km: add support for external > >> switch configuration > >> > >> This can be used if we do not want to use an EEPROM for the > >> configuration. > >> > >> Signed-off-by: Valentin Longchamp <valentin.longch...@keymile.com> > >> --- > >> board/keymile/common/common.h | 7 -- > >> board/keymile/km_arm/managed_switch.c | 169 > >> +++++++++++++++++++++++++++++++-- > >> board/keymile/km_arm/managed_switch.h | 99 +++++++++++++++++++ > > > > What is managed switch? Which chip it supports? Why it is sitting > here and not in generic folder? > > > > In this case, the switch is a Marvell 88E52xxx (don't remember the > exact model) > that is connected to the GE port of the Kirkwood and that gets > configured by the > Kirkwood through a MDIO link, using indirect addressing. > > We do not have the pretention to write a generic driver for such > switches/addressing, but we need this functionnality on some boards, > that's why > we have kept it in our own folder and not in a generic driver folder. > > I will add the above precisions to the commit message for an optional > v3 of the > "updates for the Keymile Marvell boards" series so that this is more > clear.
Dear Valentin We must keep develop it as generic driver. Regards... Prafulla . . . _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
