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

Benoît Thébaudeau benoit.thebaudeau at advansee.com
Sun Feb 17 17:25:33 CET 2013


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 at aribaud.net> wrote:
> 
> > Hi Benoît,
> > 
> > On Sun, 17 Feb 2013 16:51:37 +0100 (CET), Benoît Thébaudeau
> > <benoit.thebaudeau at 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 at 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


More information about the U-Boot mailing list