[U-Boot] Please pull u-boot-mpc83xx

Wolfgang Denk wd at denx.de
Wed Jul 29 09:21:13 CEST 2009


Dear Kim Phillips,

In message <20090727185329.874ef2ae.kim.phillips at freescale.com> you wrote:
> Hi Wolfgang, please pull (mostly) support for the VME8349 board (it's
> initial patch was posted well in advance of the window) and a couple of
> fixes:
> 
> The following changes since commit 94978e19f31d225b4f7d97c4acbac1ecfaeb8f69:
>   Wolfgang Denk (1):
>         Prepare 2009.08-rc1 (again, after fixing last minute issues).
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-mpc83xx.git master
> 
> Kim Phillips (1):
>       mpc83xx: CONFIG_83XX_GENERIC_PCI is now synonymous with CONFIG_PCI; remove the former
> 
> Paul Gortmaker (1):
>       sbc8349: combine HRCW flash and u-boot image flash
> 
> Reinhard Arlt (1):
>       mpc83xx: Add esd VME8349 board support
> 
>  MAINTAINERS                   |    2 +
>  MAKEALL                       |    1 +
>  Makefile                      |    2 +
>  board/esd/vme8349/Makefile    |   54 ++++
>  board/esd/vme8349/caddy.c     |  194 +++++++++++++
>  board/esd/vme8349/caddy.h     |   77 ++++++
>  board/esd/vme8349/config.mk   |   28 ++
>  board/esd/vme8349/pci.c       |  119 ++++++++
>  board/esd/vme8349/vme8349.c   |  140 ++++++++++
>  board/sbc8349/config.mk       |    2 +-
>  cpu/mpc83xx/Makefile          |    4 +-
>  doc/README.sbc8349            |   30 ++-
>  drivers/pci/pci_auto.c        |    2 +-
>  include/configs/MPC8313ERDB.h |    1 -
>  include/configs/MPC8315ERDB.h |    3 +-
>  include/configs/MPC8323ERDB.h |    1 -
>  include/configs/MPC832XEMDS.h |    1 -
>  include/configs/MPC8349EMDS.h |    4 -
>  include/configs/MPC8349ITX.h  |    1 -
>  include/configs/MPC8360EMDS.h |    1 -
>  include/configs/MPC8360ERDK.h |    1 -
>  include/configs/MPC837XEMDS.h |    3 +-
>  include/configs/MPC837XERDB.h |    3 +-
>  include/configs/MVBLM7.h      |    1 -
>  include/configs/SIMPC8313.h   |    1 -
>  include/configs/TQM834x.h     |    1 -
>  include/configs/sbc8349.h     |    5 +-
>  include/configs/vme8349.h     |  608 +++++++++++++++++++++++++++++++++++++++++
>  28 files changed, 1254 insertions(+), 36 deletions(-)
>  create mode 100644 board/esd/vme8349/Makefile
>  create mode 100644 board/esd/vme8349/caddy.c
>  create mode 100644 board/esd/vme8349/caddy.h
>  create mode 100644 board/esd/vme8349/config.mk
>  create mode 100644 board/esd/vme8349/pci.c
>  create mode 100644 board/esd/vme8349/vme8349.c
>  create mode 100644 include/configs/vme8349.h

Applied, thanks.

But please do not reqrite the Subject of committed patches, at least
not without specifically mentioning it here.

My "patch tracking" is based on the Subject:  lines,  and  it  causes
additional  work  (and  eventually introduces errors) when you rename
the   original   (poor,    agreed)    Subject    "mpc83xx:    because
CONFIG_83XX_GENERIC_PCI  is now synonymous with CONFIG_PCI, we remove
the former" into "mpc83xx: CONFIG_83XX_GENERIC_PCI is now  synonymous
with  CONFIG_PCI; remove the former" - which is still _way_too_long_,
btw. Rather reject the patch  and  ask  for  a  resubmit.  Even  from
yourself :-)

Thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Of course there's no reason for it, it's just our policy.


More information about the U-Boot mailing list