From: Stephen Warren <swarren@xxxxxxxxxx>
Currently, U-Boot's itest command cannot read from address 0. On Tegra20,
soc['ram-base'] can be 0, and is the address passed to itest. Modify the
code to use an offset from this address, so as not to use address 0.
Note that in Tegra30 and later, the RAM base is non-zero, so this issue
does not occur. This issue also only affects the "flash" command, and not
the "exec" command.
A patch is also in progress to fix the itest command in U-Boot. However,
fixing tegra-uboot-flasher not to use address 0 seems reasonable too, and
will immediately solve the issue irrespective of which U-Boot code-base is
in use.
Signed-off-by: Stephen Warren <swarren@xxxxxxxxxx>
---
Untested so far since I'm working from home today due to snowfall and can't
remotely plug in my old Tegra20 HW. I'll test before I apply this tomorrow.
tegra-uboot-flasher | 9 +++++----
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/tegra-uboot-flasher b/tegra-uboot-flasher
index 0fefe21dc211..153db0dafb8e 100755
--- a/tegra-uboot-flasher
+++ b/tegra-uboot-flasher
@@ -206,14 +206,15 @@ def func_flash():
cmd = ['fdtput', '-p', '-t', 'x', u_boot_dtb_runflash, '/config', 'bootdelay', '0xfffffffe']
run(workdir, cmd)
+ crc_addr = soc['ram-base'] + 4
bootcmd = ''
bootcmd += 'echo >>> Verifying image in RAM... ; '
- bootcmd += 'crc32 0x%08x 0x%08x 0x%08x ; ' % (flash_image_addr, flash_img_size, soc['ram-base'])
- bootcmd += 'if itest.l *0x%08x != 0x%x; then echo CRC MISMATCH of initial image; exit; fi ; ' % (soc['ram-base'], flash_img_crc32_bs)
+ bootcmd += 'crc32 0x%08x 0x%08x 0x%08x ; ' % (flash_image_addr, flash_img_size, crc_addr)
+ bootcmd += 'if itest.l *0x%08x != 0x%x; then echo CRC MISMATCH of initial image; exit; fi ; ' % (crc_addr, flash_img_crc32_bs)
bootcmd += gen_flashcmd(flash_image_addr, readback_addr, flash_img_size)
bootcmd += 'echo >>> Verifying image from flash... ; '
- bootcmd += 'crc32 0x%08x 0x%08x 0x%08x ; ' % (readback_addr, flash_img_size, soc['ram-base'])
- bootcmd += 'if itest.l *0x%08x != 0x%x; then echo CRC MISMATCH of readback image; exit; fi ; ' % (soc['ram-base'], flash_img_crc32_bs)
+ bootcmd += 'crc32 0x%08x 0x%08x 0x%08x ; ' % (readback_addr, flash_img_size, crc_addr)
+ bootcmd += 'if itest.l *0x%08x != 0x%x; then echo CRC MISMATCH of readback image; exit; fi ; ' % (crc_addr, flash_img_crc32_bs)
bootcmd += 'echo >>> Setting up environment... ; '
bootcmd += 'env default -f -a ; '
# Perhaps U-Boot should set $boardname based on the ID EEPROM; then we wouldn't need this
--
2.7.0
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
