Skip to contentSkip to conversation

Pull request u-boot-fdt.git / FDT bugfix patches

U-Boot-Users Applied Jerry Van Baren 2 participants 3 messages Apr 1, 2008 – Apr 7, 2008

lore
Share & tools
lore
#0 lore raw

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

#1 lore raw
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(-)
#2 lore raw
Quoted 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,
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

Quoted 8 lines

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

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]>