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

Benoît Thébaudeau
2013-02-17T16:25:33Z

On Sunday, February 17, 2013 5:08:00 PM, Albert ARIBAUD wrote:
> 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

Yes, I had already Cc'ed Zhong.

> (how about changing the
> MAINTAINERS file format so that we can tell for sure without having to
> ceck the first entry?)

That would be good. E.g., a colon could be added after the last e-mail address
of each group of board maintainers, or the empty line between the e-mail
addresses and the boards could be removed.

Best regards,
Beno?t