[U-Boot] [PATCH V4 0/4] Marvell GbE multiple SoC support

[Prafulla Wadaskar]

> -----Original Message-----
> From: u-boot-bounces at lists.denx.de 
> [mailto:u-boot-bounces at lists.denx.de] On Behalf Of Albert Aribaud
> Sent: Tuesday, July 13, 2010 1:54 AM
> To: u-boot at lists.denx.de
> Subject: [U-Boot] [PATCH V4 0/4] Marvell GbE multiple SoC support
> 
> NOTES
> 
> 1) This patchset will only apply above a previously posted and
> Acked-by patch available on the mailing list archives e.g. at:
> <http://lists.denx.de/pipermail/u-boot/2010-July/073590.html>
> ("kirkwood_egiga: updates: fix DRAM mapping and typo").
> 
> 2) This patchset will build for edminiv2 but requires a bugfix
> patch, also Acked-by and available on archives as well at e.g.
> <http://lists.denx.de/pipermail/u-boot/2010-July/073789.html>
> ("kirkwood_egiga: bugfix: add DMA sequence points").
> 
> 3) This patchset will only apply above a previously posted, not
> yet Acked-by patch available on the mailing list archives e.g. at:
> <http://lists.denx.de/pipermail/u-boot/2010-July/073838.html>
> ("kirkwood_egiga: CONFIG_SKIP_LOCAL_MAC_RANDOMIZATION").
> 
> PATCHSET HISTORY
> 
> V1:	First submission.
> V2:	Reorganization as per Prafulla Wadasakar's suggestion.
> 	Used -C in format-patch to render renames correctly.
> 	Removal of DRAM patch posted standalone.
> 	Removal of unrelated changes.
> V3:	Removal of DMA bugfix patch posted standalone
> 	Introduced MAC non-randomization config option
> 	Added short description to each patch.
> V4:	Renaming of mv_egiga to mvgbe throughout.
> 	Removal of MAC non-randomization patch posted standalone.
> 	Removal of unrelated edminiv2 changes.
> 	Move of MAX_MVGBE_DEVS from driver to SoC files.
> 
> SHORT STORY
> 
> This patchset separates the Marvell Gigabit Ethernet (mvgbe) from
> kirkwood, adds orion5x support in mvgbe, and adds mvgbe support
> in edminiv2. It has been successfully tested on an OpenRD-Client
> after each of the four commits and on an EDMini V2 after the last
> commit; it has been run through checkpatch.pl, with 6 false-positive
> errors on patch 2 and 1 on patch 4.
> 
> LONG STORY
> 
> This is a set of four atomic commits.
> 
> the first commit renames kirkwood_egiga.[ch] to mvgbe.[ch] in
> drivers/net/ (also modifies Makefile) to make sur the renaming
> remains visible in git at 100% similarity.
> 
> The second commit removes all dependency of mvgbe on kirkwood.
> Kirkwood references in all mvgbe symbols are replaced with mvgbe
> references throughout the source tree. Some cosmetic changes have
> also been made to minimize checkpatch errors (although 6 false-
> positives remain).
> 
> The third commit adds orion5x support to mvgbe.
> 
> The fourth commit adds edminiv2 support for the orion5x GbE 
> controller.
> 
> Prior to submission, all commits have been tested on a kirkwood-based
> OpenRD-Client for regression avoidance, and the last commit has been
> tested on an orion5x-based ED Mini V2 for functionality validation.
> 
> Also, all patches were run through checkpatch.pl --no-tree; there are
> 6 errors emitted for patch 2, and 1 for patch 4, all corresponding to
> the same line: in all config files involved, checkpatch.pl erroneously
> considers "#define CONFIG_MVGBE_PORTS {...}" as a macro expanding to
> statements whereas it actually expands to an array initializer.
> 
> Albert Aribaud (4):
>   net: rename: kirkwood_egiga as mvgbe
>   mvgbe: support SoCs other than kirkwood
>   mvgbe: add support for orion5x GbE controller
>   edminiv2: add ethernet support
> 
>  arch/arm/cpu/arm926ejs/kirkwood/cpu.c         |    4 +-
>  arch/arm/cpu/arm926ejs/orion5x/cpu.c          |    8 +
>  arch/arm/include/asm/arch-kirkwood/kirkwood.h |    5 +
>  arch/arm/include/asm/arch-orion5x/orion5x.h   |    4 +
>  board/LaCie/edminiv2/edminiv2.c               |   36 ++
>  board/LaCie/edminiv2/edminiv2.h               |   41 ++
>  drivers/net/Makefile                          |    2 +-
>  drivers/net/kirkwood_egiga.h                  |  505 
> -------------------------
>  drivers/net/{kirkwood_egiga.c => mvgbe.c}     |  350 
> +++++++++--------
>  drivers/net/mvgbe.h                           |  505 
> +++++++++++++++++++++++++
>  include/configs/edminiv2.h                    |   19 +-
>  include/configs/guruplug.h                    |    4 +-
>  include/configs/km_arm.h                      |    4 +-
>  include/configs/mv88f6281gtw_ge.h             |    4 +-
>  include/configs/openrd_base.h                 |    4 +-
>  include/configs/rd6281a.h                     |    4 +-
>  include/configs/sheevaplug.h                  |    4 +-
>  include/netdev.h                              |    2 +-
>  18 files changed, 815 insertions(+), 690 deletions(-)
>  create mode 100644 board/LaCie/edminiv2/edminiv2.h
>  delete mode 100644 drivers/net/kirkwood_egiga.h
>  rename drivers/net/{kirkwood_egiga.c => mvgbe.c} (63%)
>  create mode 100644 drivers/net/mvgbe.h
> 

Acked-by: Prafulla Wadaskar <prafulla at marvell.com>

Regards..
Prafulla . .