From: Ian Campbell Date: Sat, 02 May 2015 15:08:30 +0100 Subject: [U-Boot] [PATCH 1/5] nand: sunxi: change BLOCK_SIZE in mksunxiboot to match NAND block size In-Reply-To: <1430319781-15375-2-git-send-email-dkochmanski@turtle-solutions.eu> References: <1430319781-15375-1-git-send-email-dkochmanski@turtle-solutions.eu> <1430319781-15375-2-git-send-email-dkochmanski@turtle-solutions.eu> Message-ID: <1430575710.15640.127.camel@hellion.org.uk> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Wed, 2015-04-29 at 17:02 +0200, Daniel Kochma?ski wrote: > This change is necessary to calculate correct checksum for NAND > boot. Works both for MMC and NAND. Without it BROM rejects boot image > as invalid (bad checksum). (Changes block size from 0x200 to 0x2000). I'm inferring here that for MMC boot this checksum is ignored by the BROM? If that is the case then I think it should be explicitly mentioned. As well as the checksum this also changes the header's length field, FWIW. > Signed-off-by: Daniel Kochma?ski > Cc: Ian Campbell > Cc: Hans De Goede > --- > > tools/mksunxiboot.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tools/mksunxiboot.c b/tools/mksunxiboot.c > index 0035f6e..bef0c79 100644 > --- a/tools/mksunxiboot.c > +++ b/tools/mksunxiboot.c > @@ -65,7 +65,7 @@ int gen_check_sum(struct boot_file_head *head_p) > > #define SUN4I_SRAM_SIZE 0x7600 /* 0x7748+ is used by BROM */ > #define SRAM_LOAD_MAX_SIZE (SUN4I_SRAM_SIZE - sizeof(struct boot_file_head)) > -#define BLOCK_SIZE 512 > +#define BLOCK_SIZE 0x2000 > > struct boot_img { > struct boot_file_head header;