[U-Boot-Users] Pull request u-boot-fdt.git / FDT bugfix patches

[U-Boot-Users] Pull request u-boot-fdt.git / FDT bugfix patches

[Jerry Van Baren]

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

[U-Boot-Users] Pull request u-boot-fdt.git / FDT bugfix patches

[Jerry Van Baren]

Jerry Van Baren wrote:
> 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(-)

[U-Boot-Users] Pull request u-boot-fdt.git / FDT bugfix patches

[Wolfgang Denk]

In message <47F3A61A.9070003@gmail.com> you wrote:
> Jerry Van Baren wrote:
> > 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

Applied, 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
Eeeek!
'eval' on strings should have been named 'evil'.    -- Tom Phoenix in
        <Pine.GSO.3.96.980526121813.27437N-100000@user2.teleport.com>