mbox

[U-Boot,47/52] ARM: remove broken "at91sam9261ek" / "at91sam9g10ek" boards

Message ID CANqCtQJGFgm-EHJk+NXHyDwJ7Dyv2Oq+MkyFxQ7msUQOjDqpJQ@mail.gmail.com
State Not Applicable
Delegated to: Reinhard Meyer
Headers show

Pull-request

git://git.denx.de/u-boot-usb.git fix-at91sam9261ek

Message

Remy Bohmer July 30, 2011, 11:12 a.m. UTC 
Hi Wolfgang,

2011/7/18 Wolfgang Denk <wd@denx.de>:
> Signed-off-by: Wolfgang Denk <wd@denx.de>
> Cc: Stelian Pop <stelian.pop@leadtechdesign.com>
> Cc: Sedji Gaouaou<sedji.gaouaou@atmel.com>
> ---
>  MAINTAINERS                               |    2 -
>  MAKEALL                                   |    2 -
>  Makefile                                  |   23 ---
>  board/atmel/at91sam9261ek/Makefile        |   56 ------
>  board/atmel/at91sam9261ek/at91sam9261ek.c |  292 -----------------------------
>  board/atmel/at91sam9261ek/config.mk       |    1 -
>  board/atmel/at91sam9261ek/led.c           |   43 -----
>  board/atmel/at91sam9261ek/partition.c     |   40 ----
>  doc/README.scrapyard                      |    2 +
>  include/configs/at91sam9261ek.h           |  240 -----------------------
>  10 files changed, 2 insertions(+), 699 deletions(-)
>  delete mode 100644 board/atmel/at91sam9261ek/Makefile
>  delete mode 100644 board/atmel/at91sam9261ek/at91sam9261ek.c
>  delete mode 100644 board/atmel/at91sam9261ek/config.mk
>  delete mode 100644 board/atmel/at91sam9261ek/led.c
>  delete mode 100644 board/atmel/at91sam9261ek/partition.c
>  delete mode 100644 include/configs/at91sam9261ek.h

(I just got back from my vacation, hence the late reply)
I do not agree that support for this board is going to be removed.
This is one of my reference boards for my USB maintenance work.
I have a series of patches pending in my u-boot-usb tree ready for a
long time that fixes this board. Several times on the mailinglist a
fix for this board has been posted, but they are not picked up by the
Atmel maintainer so far. Even the Atmel tree has been under 'rework'
for more than 6 months without any push to mainline (until recently),
this made fixing these boards a moving target to shoot at.
In short: The lack on a responsive/active custiodian is not a reason
to remove board support. I also offered to take over the custodianship
already for this board. No response so far on this subject as well.

Instead of dropping this board, you can also pull this (and add my
name as maintainer for this board):
---------------------------------------------------------------------------------------------------------
The following changes since commit dd620b26332eb1e9fc0216760909c4046ba09745:

  Merge branch 'master' of /home/wd/git/u-boot/custodians (2011-07-30
01:39:14 +0200)

are available in the git repository at:

  git://git.denx.de/u-boot-usb.git fix-at91sam9261ek

Remy Bohmer (1):
      at91sam9261ek: make operational again

 Makefile                                           |   23 ---
 arch/arm/cpu/arm926ejs/at91/at91sam9261_devices.c  |   31 ++--
 arch/arm/cpu/arm926ejs/at91/led.c                  |    1 -
 arch/arm/include/asm/arch-at91/at91_matrix.h       |   17 ++-
 arch/arm/include/asm/arch-at91/at91sam9261.h       |    1 +
 .../arm/include/asm/arch-at91/at91sam9261_matrix.h |   64 ---------
 board/atmel/at91sam9261ek/Makefile                 |    6 +-
 board/atmel/at91sam9261ek/at91sam9261ek.c          |  147 +++++++++++---------
 board/atmel/at91sam9261ek/config.mk                |    1 -
 board/atmel/at91sam9261ek/led.c                    |    8 +-
 boards.cfg                                         |    6 +
 doc/README.at91                                    |    3 +-
 include/configs/at91sam9261ek.h                    |  142 +++++++++++--------
 13 files changed, 201 insertions(+), 249 deletions(-)
 delete mode 100644 arch/arm/include/asm/arch-at91/at91sam9261_matrix.h
 deleteThe following changes since commit
dd620b26332eb1e9fc0216760909c4046ba09745:

  Merge branch 'master' of /home/wd/git/u-boot/custodians (2011-07-30
01:39:14 +0200)

are available in the git repository at:

  git://git.denx.de/u-boot-usb.git fix-at91sam9261ek

Remy Bohmer (1):
      at91sam9261ek: make operational again

 Makefile                                           |   23 ---
 arch/arm/cpu/arm926ejs/at91/at91sam9261_devices.c  |   31 ++--
 arch/arm/cpu/arm926ejs/at91/led.c                  |    1 -
 arch/arm/include/asm/arch-at91/at91_matrix.h       |   17 ++-
 arch/arm/include/asm/arch-at91/at91sam9261.h       |    1 +
 .../arm/include/asm/arch-at91/at91sam9261_matrix.h |   64 ---------
 board/atmel/at91sam9261ek/Makefile                 |    6 +-
 board/atmel/at91sam9261ek/at91sam9261ek.c          |  147 +++++++++++---------
 board/atmel/at91sam9261ek/config.mk                |    1 -
 board/atmel/at91sam9261ek/led.c                    |    8 +-
 boards.cfg                                         |    6 +
 doc/README.at91                                    |    3 +-
 include/configs/at91sam9261ek.h                    |  142 +++++++++++--------
 13 files changed, 201 insertions(+), 249 deletions(-)
 delete mode 100644 arch/arm/include/asm/arch-at91/at91sam9261_matrix.h
 delete mode 100644 board/atmel/at91sam9261ek/config.mk
 mode 100644 board/atmel/at91sam9261ek/config.mk
---------------------------------------------------------------------------------------------------------

Kind regards,

Remy

Comments

Reinhard Meyer Aug. 2, 2011, 10:54 a.m. UTC | #1 
Dear Remy Bohmer, dear Wolfgang,
> 
> 2011/7/18 Wolfgang Denk <wd@denx.de>:
> > Signed-off-by: Wolfgang Denk <wd@denx.de>
> > Cc: Stelian Pop <stelian.pop@leadtechdesign.com>
> > Cc: Sedji Gaouaou<sedji.gaouaou@atmel.com>
> > ---
> >  MAINTAINERS                               |    2 -
> >  MAKEALL                                   |    2 -
> >  Makefile                                  |   23 ---
> >  board/atmel/at91sam9261ek/Makefile        |   56 ------
> >  board/atmel/at91sam9261ek/at91sam9261ek.c |  292 -----------------------------
> >  board/atmel/at91sam9261ek/config.mk       |    1 -
> >  board/atmel/at91sam9261ek/led.c           |   43 -----
> >  board/atmel/at91sam9261ek/partition.c     |   40 ----
> >  doc/README.scrapyard                      |    2 +
> >  include/configs/at91sam9261ek.h           |  240 -----------------------
> >  10 files changed, 2 insertions(+), 699 deletions(-)
> >  delete mode 100644 board/atmel/at91sam9261ek/Makefile
> >  delete mode 100644 board/atmel/at91sam9261ek/at91sam9261ek.c
> >  delete mode 100644 board/atmel/at91sam9261ek/config.mk
> >  delete mode 100644 board/atmel/at91sam9261ek/led.c
> >  delete mode 100644 board/atmel/at91sam9261ek/partition.c
> >  delete mode 100644 include/configs/at91sam9261ek.h

As of now, this board is "unbroken" again.
I might also point out, that the Atmel folks are still the responsible
maintainers of their respective boards.
Help and testing, I am sure is welcome. If there are any issues with
Hong Xu's patch I could not detect, please report:)

Wolfgang: I have set the removal patch for this board to "non
applicable"

Best Regards,
Reinhard

PS: currently I base all on top of u-boot/master since u-boot-arm/master
is seriously broken with the OMAP patches.
Remy Bohmer Aug. 3, 2011, 8:20 a.m. UTC | #2 
Hi Reinhard,

> As of now, this board is "unbroken" again.

Thanks!

Remy