On Sun, Jul 13, 2014 at 09:45:51PM +0530, Jagan Teki wrote: > On Fri, Jun 27, 2014 at 3:48 PM, Prafulla Wadaskar <[email protected]> > wrote: > > > > > >> -----Original Message----- > >> From: Stefan Roese [mailto:[email protected]] > >> Sent: 27 June 2014 15:25 > >> To: [email protected] > >> Cc: Prafulla Wadaskar; [email protected] > >> Subject: [PATCH v1 0/25] Add Marvell Armada XP MV78460 > >> SoC support > >> > >> > >> This patch series adds support for the Marvell Armada > >> XP SoC's. Specifically > >> the MV78460. > >> > >> Basic support for the db-78460-bp evaluation board is > >> added. Supporting the > >> following interfaces: > >> - UART > >> - SPI (including SPI NOR flash) > >> - I2C > >> - Ethernet (neta) > >> > >> While doing this port, I tried to consolidate common > >> Marvell code into > >> the arch/arm/mvebu-common directory. This directory > >> should be used to > >> collect more common code for the MVEBU SoC's (Dove, > >> Kirkwood, Armada 370, > >> Armada 380, Armada XP). I started with Kirkwood and > >> some of its > >> interfaces. Dove is definitely a candidate to move some > >> of its code > >> into thise directory as well. > >> > >> Because of the renaming of some functions from kirkwood > >> to mvebu (to make > >> them better usable on other MVEBU SoCs), this patch > >> series not only > >> touches the ARM SoC specific files (in arch/arm/...). > >> But also some > >> device drivers (e.g. SPI, I2C). Separating these driver > >> specific patches > >> into different patches that are not depending on this > >> ARM patch series > >> seems hard if not impossible. Thats why I would really > >> like to get this > >> patch series to get applied completely be one > >> custodian. Not sure if > >> this could / should go through Tom directly? Only if > >> all the subsystem > >> custodians have given their "Acked-by ..." of course. > > > > Hi Stefan, > > I agree with you, since this patch series have huge changes it may go to > > Tom directly, I will review, test and provide ack/nack accordingly. > > > >> > >> Testing on Kirkwood based boards would be greatly > >> appreciated. So anyone with > >> access to some of those board, please give this patch > >> series a try. I really > >> hope that I didn't break anything while merging some of > >> the code into the > >> common mvebu directory. > > > > I will certainly help to test testing this stuff on all possible boards > > that I can access. > > > >> > >> Please note that this Armada XP port still requires the > >> "Binary Header" > >> (bin_hdr) from the Marvell U-Boot tree to be included > >> as a binary blob > >> into the resulting image (u-boot.kwb) that can be > >> booted by the MVEBU > >> BootROM. This binary bin_hdr is usually responsible for > >> the DDR3 > >> controller configuration and the DDR3 training. One way > >> to extract this > >> bin_hdr binary from an existing Marvell boot image > >> right now is to use > >> the "kwbimage" tool from Barebox. Please refer to the > >> documentation > >> thats available there for more details. > > > > Finally, last but not least, may thanks for your efforts and all the best > > for smooth migration ahead. > > > > Regards... > > Prafulla . . . > > > >> > >> Thanks, > >> Stefan > >> > >> > >> Stefan Roese (25): > >> sf: Add M25PX64 SPI NOR flash ID > >> arm: kirkwood: spi.h: Add some missing parenthesis > >> spi: kirkwood_spi.c: Some fixes and cleanup > >> spi: kirkwood_spi.c: Make global variable static > >> spi: kirkwood_spi.c: cosmetic: Fix minor coding style > >> issues > > I took these spi stuff patches, rest of spi require dependencies - So I Acked. > > Pls- Tom or someone push those as well while taking this series.
FWIW, in the future I'm fine with things like this coming in via the ARM SoC sub-tree with acks/reviewed-by's from the relevant other subsystem maintainers, to make dependencies easier. -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
