[U-Boot] [PATCH 6/8][RESEND] powerpc: dts: add u-boot.dtsi and mpc85xx-u-boot.dtsi

[Bin Meng]

On Wed, Aug 22, 2018 at 2:18 AM, Jagdish Gediya <jagdish.gediya at 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 at 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