On Sat, 2015-08-22 at 20:13 +0200, Hans de Goede wrote: > We only ever use syndrome mode for the partitions which contain the SPL, > as that is required for the BROM to be able to read the SPL. > > Instead of using some arbritray limit for deciding whether or not to > use syndrome, be smart and check if u-boot-dtb.bin is directly behind > the SPL, if it is not then it is on its own partition and we should not > use syndrome. > > Note the reason why we only use syndrome mode is because it comes with > weaker randomization,
"...why we only use syndrome mode for the SPL is because..." > introducing a risc for more bit errors, risk > - int syndrome = offs < CONFIG_NAND_SUNXI_SPL_SYNDROME_PARTITIONS_END; > - int i; > + int i, syndrome; > + > + if (CONFIG_SYS_NAND_U_BOOT_OFFS == CONFIG_SPL_PAD_TO) > + syndrome = 1; /* u-boot-dtb.bin appended to SPL */ > + else > + syndrome = 0; /* u-boot-dtb.bin on its own partition */ Is it not possible for a separate partition to begin at CONFIG_SPL_PAD_TO, or would padding not be used in that case? -Scott _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
