From mboxrd@z Thu Jan 1 00:00:00 1970 From: Simon Glass Date: Fri, 14 Sep 2018 12:53:20 +0200 Subject: [U-Boot] [PATCH v3 3/8] binman: Add a new "skip-at-start" property in Section class In-Reply-To: <1535990712-8359-4-git-send-email-jagdish.gediya@nxp.com> References: <1535990712-8359-1-git-send-email-jagdish.gediya@nxp.com> <1535990712-8359-4-git-send-email-jagdish.gediya@nxp.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 3 September 2018 at 18:05, Jagdish Gediya wrote: > Currently binman calculates '_skip_at_start' based on 'end-at-4gb' > property and it is used for x86 images. > > For PowerPC mpc85xx based CPU, CONFIG_SYS_TEXT_BASE is the entry > offset of the first entry. It can be 0xeff40000 or 0xfff40000 for > nor flash boot, 0x201000 for sd boot etc, so "_skip_at_start" > should be set to CONFIG_SYS_TEXT_BASE. > > 'end-at-4gb' property is not applicable where CONFIG_SYS_TEXT_BASE + > Image size != 4gb. > > Add new property 'skip-at-start' in Section class so that > '_skip_at_start' can be calculated either based on 'end-at-4gb' > or based on "skip-at-start". > > Add a test case to check that 'skip-at-start' and 'end-at-4gb' > property can't be used together. > > Signed-off-by: Jagdish Gediya > Reviewed-by: Bin Meng > --- > Changes for v2: > - Renamed 'start-pos' property to 'skip-at-start' > - Updated README > > Changes for v3: > - Modification as per Simon Glass's comments > - Added test case for 'skip-at-start' property > > tools/binman/README | 9 +++++++++ > tools/binman/bsection.py | 15 +++++++++++---- > tools/binman/ftest.py | 8 ++++++++ > .../test/80_4gb_and_skip_at_start_together.dts | 21 +++++++++++++++++++++ > 4 files changed, 49 insertions(+), 4 deletions(-) > create mode 100644 tools/binman/test/80_4gb_and_skip_at_start_together.dts Reviewed-by: Simon Glass