[U-Boot] [PATCH] scripts/Makefile.lib: Make __dtb_dt_begin align on a 4 byte boundary

2014-11-20 Thread Bin Meng 
With CONFIG_OF_EMBED, __dtb_dt_begin must be aligned on a 4 byte
boundary otherwise fdtdec_prepare_fdt() will fail.

Signed-off-by: Bin Meng bmeng...@gmail.com
---
 scripts/Makefile.lib | 1 +
 1 file changed, 1 insertion(+)

diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib
index 9d1383a..aead250 100644
--- a/scripts/Makefile.lib
+++ b/scripts/Makefile.lib
@@ -260,6 +260,7 @@ quiet_cmd_dt_S_dtb= DTB $@
 cmd_dt_S_dtb=  \
 (  \
echo '.section .dtb.init.rodata,a';   \
+   echo '.p2align 2';  \
echo '.global __dtb_$(*F)_begin';   \
echo '__dtb_$(*F)_begin:';  \
echo '.incbin $ ';   \
-- 
1.8.2.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] scripts/Makefile.lib: Make __dtb_dt_begin align on a 4 byte boundary

2014-11-20 Thread Masahiro Yamada 
Hi Bin,

I think this issue is covered by Simon's patch:
http://patchwork.ozlabs.org/patch/411062/



On Thu, 20 Nov 2014 17:45:06 +0800
Bin Meng bmeng...@gmail.com wrote:

 With CONFIG_OF_EMBED, __dtb_dt_begin must be aligned on a 4 byte
 boundary otherwise fdtdec_prepare_fdt() will fail.
 
 Signed-off-by: Bin Meng bmeng...@gmail.com
 ---
  scripts/Makefile.lib | 1 +
  1 file changed, 1 insertion(+)
 
 diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib
 index 9d1383a..aead250 100644
 --- a/scripts/Makefile.lib
 +++ b/scripts/Makefile.lib
 @@ -260,6 +260,7 @@ quiet_cmd_dt_S_dtb= DTB $@
  cmd_dt_S_dtb=\
  (\
   echo '.section .dtb.init.rodata,a';   \
 + echo '.p2align 2';  \
   echo '.global __dtb_$(*F)_begin';   \
   echo '__dtb_$(*F)_begin:';  \
   echo '.incbin $ ';   \
 -- 
 1.8.2.1
 
 ___
 U-Boot mailing list
 U-Boot@lists.denx.de
 http://lists.denx.de/mailman/listinfo/u-boot


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] scripts/Makefile.lib: Make __dtb_dt_begin align on a 4 byte boundary

2014-11-20 Thread Bin Meng 
Hi Masahiro,

On Thu, Nov 20, 2014 at 6:04 PM, Masahiro Yamada
yamad...@jp.panasonic.com wrote:
 Hi Bin,

 I think this issue is covered by Simon's patch:
 http://patchwork.ozlabs.org/patch/411062/


Thanks for the info. I did not notice that patch before.

Regards,
Bin
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot