On Tue, Aug 28, 2018 at 11:53 AM Jagdish Gediya <jagdish.ged...@nxp.com> wrote: > > This entry contains the Powerpc mpc85xx boot page and resetvec
nits: PowerPC Please fix the title as well > sections. > > Signed-off-by: Jagdish Gediya <jagdish.ged...@nxp.com> > --- > Changes for v2: > - Updated README for new binman entry > - Added test > > tools/binman/README.entries | 14 +++++++++++- > .../etype/powerpc_mpc85xx_bootpg_resetvec.py | 25 > ++++++++++++++++++++++ > tools/binman/ftest.py | 8 +++++++ > .../test/80_powerpc_mpc85xx_bootpg_resetvec.dts | 16 ++++++++++++++ > 4 files changed, 62 insertions(+), 1 deletion(-) > create mode 100644 tools/binman/etype/powerpc_mpc85xx_bootpg_resetvec.py > create mode 100644 tools/binman/test/80_powerpc_mpc85xx_bootpg_resetvec.dts > > diff --git a/tools/binman/README.entries b/tools/binman/README.entries > index c6e7b22..99440d6 100644 > --- a/tools/binman/README.entries > +++ b/tools/binman/README.entries > @@ -221,6 +221,18 @@ See README.x86 for information about Intel binary blobs. > > > > +Entry: powerpc-mpc85xx-bootpg-resetvec: Powerpc mpc85xx bootpg + resetvec > code for U-Boot nits: PowerPC, please fix this globally in this patch > +----------------------------------------------------------------------------------------- > + > +Properties / Entry arguments: > + - filename: Filename of u-boot-br.bin (default 'u-boot-br.bin') > + > +This enrty is valid for Powerpc mpc85xx cpus. This entry holds > +'bootpg + resetvec' code for Powerpc mpc85xx CPUs which needs to be > +placed at offset 'RESET_VECTOR_ADDRESS - 0xffc'. > + > + > + > Entry: section: Entry that contains other entries > ------------------------------------------------- > > @@ -543,7 +555,7 @@ Properties / Entry arguments: > - kernelkey: Name of the kernel key to use (inside keydir) > - preamble-flags: Value of the vboot preamble flags (typically 0) > > -Chromium OS signs the read-write firmware and kernel, writing the signature > +Chromium OS signs the read-write firmware and kernel, writing the signature > in this block. This allows U-Boot to verify that the next firmware stage > and kernel are genuine. > > diff --git a/tools/binman/etype/powerpc_mpc85xx_bootpg_resetvec.py > b/tools/binman/etype/powerpc_mpc85xx_bootpg_resetvec.py > new file mode 100644 > index 0000000..37cf408 > --- /dev/null > +++ b/tools/binman/etype/powerpc_mpc85xx_bootpg_resetvec.py > @@ -0,0 +1,25 @@ > +# SPDX-License-Identifier: GPL-2.0+ > +# Copyright 2018 NXP > +# > +# Entry-type module for the Powerpc mpc85xx bootpg and resetvec code for > U-Boot > +# > + > +from entry import Entry > +from blob import Entry_blob > + > +class Entry_powerpc_mpc85xx_bootpg_resetvec(Entry_blob): > + """Powerpc mpc85xx bootpg + resetvec code for U-Boot > + > + Properties / Entry arguments: > + - filename: Filename of u-boot-br.bin (default 'u-boot-br.bin') > + > + This enrty is valid for Powerpc mpc85xx cpus. This entry holds > + 'bootpg + resetvec' code for Powerpc mpc85xx CPUs which needs to be > + placed at offset 'RESET_VECTOR_ADDRESS - 0xffc'. > + """ > + > + def __init__(self, section, etype, node): > + Entry_blob.__init__(self, section, etype, node) > + > + def GetDefaultFilename(self): > + return 'u-boot-br.bin' > diff --git a/tools/binman/ftest.py b/tools/binman/ftest.py > index a8456c2..7a6a5ae 100644 > --- a/tools/binman/ftest.py > +++ b/tools/binman/ftest.py > @@ -39,6 +39,7 @@ U_BOOT_SPL_DTB_DATA = 'spldtb' > U_BOOT_TPL_DTB_DATA = 'tpldtb' > X86_START16_DATA = 'start16' > X86_START16_SPL_DATA = 'start16spl' > +PPC_MPC85XX_BR_DATA = 'ppcmpc85xxbr' > U_BOOT_NODTB_DATA = 'nodtb with microcode pointer somewhere in here' > U_BOOT_SPL_NODTB_DATA = 'splnodtb with microcode pointer somewhere in here' > FSP_DATA = 'fsp' > @@ -90,6 +91,7 @@ class TestFunctional(unittest.TestCase): > TestFunctional._MakeInputFile('vga.bin', VGA_DATA) > self._ResetDtbs() > TestFunctional._MakeInputFile('u-boot-x86-16bit.bin', > X86_START16_DATA) > + TestFunctional._MakeInputFile('u-boot-br.bin', PPC_MPC85XX_BR_DATA) > TestFunctional._MakeInputFile('spl/u-boot-x86-16bit-spl.bin', > X86_START16_SPL_DATA) > TestFunctional._MakeInputFile('u-boot-nodtb.bin', U_BOOT_NODTB_DATA) > @@ -756,6 +758,12 @@ class TestFunctional(unittest.TestCase): > data = self._DoReadFile('33_x86-start16.dts') > self.assertEqual(X86_START16_DATA, data[:len(X86_START16_DATA)]) > > + def testPackPowerpcMpc85xxBootpgResetvec(self): > + """Test that an image with powerpc-mpc85xx-bootpg-resetvec can be > + created""" > + data = self._DoReadFile('80_powerpc_mpc85xx_bootpg_resetvec.dts') > + self.assertEqual(PPC_MPC85XX_BR_DATA, > data[:len(PPC_MPC85XX_BR_DATA)]) > + > def _RunMicrocodeTest(self, dts_fname, nodtb_data, ucode_second=False): > """Handle running a test for insertion of microcode > > diff --git a/tools/binman/test/80_powerpc_mpc85xx_bootpg_resetvec.dts > b/tools/binman/test/80_powerpc_mpc85xx_bootpg_resetvec.dts > new file mode 100644 > index 0000000..8f4b16c > --- /dev/null > +++ b/tools/binman/test/80_powerpc_mpc85xx_bootpg_resetvec.dts > @@ -0,0 +1,16 @@ > +// SPDX-License-Identifier: GPL-2.0+ > +/* > + * Copyright 2018 NXP > + */ > + > +/dts-v1/; > + > +/ { > + #address-cells = <1>; > + #size-cells = <1>; > + > + binman { > + powerpc-mpc85xx-bootpg-resetvec { > + }; > + }; > +}; > -- Regards, Bin _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot