[U-Boot] [PATCH v1 0/25] Add Marvell Armada XP MV78460 SoC support

Tom Rini
2014-07-14T14:51:26Z

On Sun, Jul 13, 2014 at 09:45:51PM +0530, Jagan Teki wrote:
> On Fri, Jun 27, 2014 at 3:48 PM, Prafulla Wadaskar <prafulla@marvell.com> wrote:
> >
> >
> >> -----Original Message-----
> >> From: Stefan Roese [mailto:sr at denx.de]
> >> Sent: 27 June 2014 15:25
> >> To: u-boot at lists.denx.de
> >> Cc: Prafulla Wadaskar; trini at ti.com
> >> 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140714/1a27fd02/attachment.pgp>