[U-Boot] [PATCH v7 16/19] arm926ejs: Remove deprecated and now unused NAND SPL

Albert ARIBAUD
2013-02-17T16:08:00Z

Hi Albert,

On Sun, 17 Feb 2013 17:04:54 +0100, Albert ARIBAUD
<albert.u.boot@aribaud.net> wrote:

> Hi Beno?t,
>
> On Sun, 17 Feb 2013 16:51:37 +0100 (CET), Beno?t Th?baudeau
> <benoit.thebaudeau@advansee.com> wrote:
>
> > Hi Albert, Tom, Zhong,
> >
> > On Friday, February 15, 2013 9:54:22 PM, Beno?t Th?baudeau wrote:
> > > Signed-off-by: Beno?t Th?baudeau <benoit.thebaudeau@advansee.com>
> > > ---
> > > Changes in v7: None
> > > Changes in v6:
> > > - New patch.
> > >
> > > Changes in v5: None
> > > Changes in v4: None
> > > Changes in v3: None
> > > Changes in v2: None
> > >
> > > arch/arm/cpu/arm926ejs/start.S | 10 ----------
> > > 1 file changed, 10 deletions(-)
> >
> > I would like to get your feedback regarding the status of the Samsung SMDK6400
> > board:
> > - It is not in boards.cfg, so, according to commit 1285a28, support for it
> > should already have been removed a long time ago. It also seems to be the
> > only board remaining in the main Makefile.
> > - It uses the deprecated NAND SPL.
> > - MAKEALL does not test its build, which has been broken for a while.
> > - If it were removed or fixed, ARM1176's start.S' relocate_code() could be made
> > identical to all the other implementations of this function, so all this
> > duplicated code could be moved to a common location like crt0.S. Besides
> > that, it would be possible to completely get rid of the legacy NAND SPL on
> > ARM.
> >
> > I have no intention of fixing this board, but dropping it and cleaning up ARM
> > after that would be easy.
>
> Cc:ing the board maintainers, as they should be the first ones to be
> asked whether they intend to bring the board into boards.cfg and fix
> it, or whether it should be dropped.

Scratch this: correct maintainer CC:ed now (how about changing the
MAINTAINERS file format so that we can tell for sure without having to
ceck the first entry?)

> > Best regards,
> > Beno?t
>
> Amicalement,

Amicalement,
--
Albert.