On Wed, Aug 22, 2018 at 2:18 AM, Jagdish Gediya jagdish.gediya@nxp.com wrote:
add required binman node inside dts to use binman to prepare
nits: Add
u-boot-dtb.bin
Signed-off-by: Jagdish Gediya jagdish.gediya@nxp.com
arch/powerpc/dts/mpc85xx-u-boot.dtsi | 16 ++++++++++++++++ arch/powerpc/dts/u-boot.dtsi | 31 +++++++++++++++++++++++++++++++ 2 files changed, 47 insertions(+) create mode 100644 arch/powerpc/dts/mpc85xx-u-boot.dtsi create mode 100644 arch/powerpc/dts/u-boot.dtsi
diff --git a/arch/powerpc/dts/mpc85xx-u-boot.dtsi b/arch/powerpc/dts/mpc85xx-u-boot.dtsi new file mode 100644 index 0000000..637db11 --- /dev/null +++ b/arch/powerpc/dts/mpc85xx-u-boot.dtsi @@ -0,0 +1,16 @@ +// SPDX-License-Identifier: GPL-2.0+ +/*
- Copyright 2018 NXP
- */
+#include <u-boot.dtsi>
+/ { +#if defined(CONFIG_BINMAN)
binman {u-boot-with-ucode-ptr {optional-ucode;};};+#endif
Please merge this into the u-boot.dtsi, as you already used the #ifdef there for MPC85xx only.
+}; diff --git a/arch/powerpc/dts/u-boot.dtsi b/arch/powerpc/dts/u-boot.dtsi new file mode 100644 index 0000000..ebe160f --- /dev/null +++ b/arch/powerpc/dts/u-boot.dtsi @@ -0,0 +1,31 @@ +// SPDX-License-Identifier: GPL-2.0+ +/*
- Copyright 2018 NXP
- */
+#include <config.h>
+/ { +#if defined(CONFIG_BINMAN) && defined(CONFIG_MPC85xx)
binman {filename = "u-boot-dtb.bin";start-pos = <CONFIG_SYS_TEXT_BASE>;sort-by-pos;pad-byte = <0xff>;size = <CONFIG_SYS_MONITOR_LEN>;u-boot-with-ucode-ptr {offset = <CONFIG_SYS_TEXT_BASE>;};u-boot-dtb-with-ucode {align = <256>;};+#if !defined(CONFIG_SYS_MPC85XX_NO_RESETVEC) && !defined(CONFIG_NAND)
mpc85xx-bootpg-resetvec {offset = <(CONFIG_RESET_VECTOR_ADDRESS - 0xffc)>;};+#endif
};+#endif +};
Regards, Bin