Pull request u-boot-fdt.git / FDT bugfix patches
Dear Wolfgang,
On the FDT front, we have two outstanding bugfix patches:
-------------------------------------------------------------------
Explanation: I authored 43ddd9c820fec44816188f53346b464e20b3142d and missed some pieces which Kim was kind enough to clean up. (You pulled 43ddd directly into u-boot.)
-------------------------------------------------------------------
<http://article.gmane.org/gmane.comp.boot-loaders.u-boot/38849>
From: Kim Phillips <kim.phillips <at> freescale.com>
Subject: [PATCH] remove remaining CONFIG_OF_HAS_{UBOOT_ENV, BD_T} code
Newsgroups: gmane.comp.boot-loaders.u-boot
Date: 2008-03-28 22:37:49 GMT (4 days and 55 minutes ago)
finish off what commit 43ddd9c820fec44816188f53346b464e20b3142d, "Remove deprecated CONFIG_OF_HAS_UBOOT_ENV and CONFIG_OF_HAS_BD_T" started.
Signed-off-by: Kim Phillips <kim.phillips <at> freescale.com>
-------------------------------------------------------------------
-------------------------------------------------------------------
I also created a 8xx patch to fix a compile warning for 8xx boards that don't use fdt.o. This looks good to me, but I don't have a 8xx board to test it on. PLEASE VERIFY.
-------------------------------------------------------------------
<http://article.gmane.org/gmane.comp.boot-loaders.u-boot/38885/>
From: Jerry Van Baren <gvb.uboot <at> gmail.com>
Subject: [PATCH] Fix 8xx build to conditionally compile fdt.o
Newsgroups: gmane.comp.boot-loaders.u-boot
Date: 2008-03-30 02:06:31 GMT (2 days, 21 hours and 30 minutes ago)
Change to COBJS-y method so that the fdt.o library can be properly conditionally compiled. Without this change, the mpc8xx boards that don't use CONFIG_OF_LIBFDT still build the fdt.o, causing code bloat and compile warnings.
Signed-off-by: Gerald Van Baren <vanbaren <at> cideas.com>
-------------------------------------------------------------------
If it is handier, you can pull both from my u-boot-fdt repo:
The following changes since commit 3596d55eb22703d3f4f1b839fe4b000fabe081b3:
Gerald Van Baren (1):
Merge git://www.denx.de/git/u-boot into uboot
are available in the git repository at:
git://git.denx.de/u-boot-fdt.git master
Gerald Van Baren (1):
Fix 8xx build to conditionally compile fdt.o
Kim Phillips (1):
remove remaining CONFIG_OF_HAS_{UBOOT_ENV,BD_T} code
README | 13 ------ common/ft_build.c | 113 --------------------------------------------------- cpu/mpc8xx/Makefile | 24 +++++++++-- lib_ppc/bootm.c | 12 ----- 4 files changed, 19 insertions(+), 143 deletions(-)
Thanks,
gvb
Quoted 3 lines · Jerry Van Baren
Dear Wolfgang,
On the FDT front, we have <del>two</del>THREE outstanding bugfix patches:
I confirmed my error, the <nnnn> notation is indeed 32 bit cells per the DTC documentation (and his other changes are valuable improvements). As a result, I've added Andy's patch to my queued-to-pull list.
Thanks,
gvb
The following changes since commit 3596d55eb22703d3f4f1b839fe4b000fabe081b3:
Gerald Van Baren (1):
Merge git://www.denx.de/git/u-boot into uboot
are available in the git repository at:
git://git.denx.de/u-boot-fdt.git master
Andy Fleming (1):
Fix fdt set command to conform to dts spec
Jean-Christophe PLAGNIOL-VILLARD (1):
MPC8xx: Fix libfdt support introduced in commit 77ff7b74
Kim Phillips (1):
remove remaining CONFIG_OF_HAS_{UBOOT_ENV,BD_T} code
README | 13 ----- common/cmd_fdt.c | 137 +++++++++++++++++++++++++-------------------------- common/ft_build.c | 113 ------------------------------------------ cpu/mpc8xx/Makefile | 33 +++++++++---- lib_ppc/bootm.c | 12 ----- 5 files changed, 90 insertions(+), 218 deletions(-)
Wolfgang Denk
AppliedQuoted 20 lines · In message <[email protected]> you wrote:
Quoted 3 lines · Jerry Van Baren
Dear Wolfgang,
On the FDT front, we have <del>two</del>THREE outstanding bugfix patches:
I confirmed my error, the <nnnn> notation is indeed 32 bit cells per the DTC documentation (and his other changes are valuable improvements). As a result, I've added Andy's patch to my queued-to-pull list.
Thanks,
gvbThe following changes since commit
3596d55eb22703d3f4f1b839fe4b000fabe081b3:
Gerald Van Baren (1):
Mergegit://www.denx.de/git/u-bootinto ubootare available in the git repository at:
git://git.denx.de/u-boot-fdt.gitmaster
Quoted 8 lines
Andy Fleming (1):
Fix fdt set command to conform to dts specJean-Christophe PLAGNIOL-VILLARD (1):
MPC8xx: Fix libfdt support introduced in commit77ff7b74Kim Phillips (1):
remove remaining CONFIG_OF_HAS_{UBOOT_ENV,BD_T} code
Applied, thanks.
Best regards,
Wolfgang Denk
Signature · 6 lines
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
Eeeek!
'eval' on strings should have been named 'evil'. -- Tom Phoenix in
<[email protected]>