[PATCH] arm: mach-imx: Makefile: Extend u-boot-nand.imx padding

2023-10-16 Thread sbabic
> From: Hiago De Franco 
> Extend the padding process of u-boot-nand.imx target by adding 10k bytes
> of zeros to the end of the binary using the 'dd' command.
> The existing padding method did not generate a functional binary,
> as discussed in more detail in this thread [1]. Instead, we adopt the
> end-padding calculation method documented in 'board/doc/colibri_imx7.rst'
> as a reference, which is relevant for iMX7 with NAND storage.
> Adding 10k bytes of zeros provides an approximate value that makes the
> proper padding for these NAND devices.
> [1] 
> https://lore.kernel.org/all/cac4tdfuqffqzrqfv5age_xtbfy1agr2sepn_fzedexhwjdr...@mail.gmail.com/
> Signed-off-by: Hiago De Franco 
> Reviewed-by: Fabio Estevam 
Applied to u-boot-imx, master, thanks !

Best regards,
Stefano Babic

-- 
=
DENX Software Engineering GmbH,Managing Director: Erika Unter  
HRB 165235 Munich,   Office: Kirchenstr.5, 82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=


Re: [PATCH] arm: mach-imx: Makefile: Extend u-boot-nand.imx padding

2023-10-07 Thread Fabio Estevam
On Tue, Oct 3, 2023 at 6:27 PM Hiago De Franco  wrote:
>
> From: Hiago De Franco 
>
> Extend the padding process of u-boot-nand.imx target by adding 10k bytes
> of zeros to the end of the binary using the 'dd' command.
>
> The existing padding method did not generate a functional binary,
> as discussed in more detail in this thread [1]. Instead, we adopt the
> end-padding calculation method documented in 'board/doc/colibri_imx7.rst'
> as a reference, which is relevant for iMX7 with NAND storage.
>
> Adding 10k bytes of zeros provides an approximate value that makes the
> proper padding for these NAND devices.
>
> [1] 
> https://lore.kernel.org/all/cac4tdfuqffqzrqfv5age_xtbfy1agr2sepn_fzedexhwjdr...@mail.gmail.com/
>
> Signed-off-by: Hiago De Franco 

Reviewed-by: Fabio Estevam 


[PATCH] arm: mach-imx: Makefile: Extend u-boot-nand.imx padding

2023-10-03 Thread Hiago De Franco
From: Hiago De Franco 

Extend the padding process of u-boot-nand.imx target by adding 10k bytes
of zeros to the end of the binary using the 'dd' command.

The existing padding method did not generate a functional binary,
as discussed in more detail in this thread [1]. Instead, we adopt the
end-padding calculation method documented in 'board/doc/colibri_imx7.rst'
as a reference, which is relevant for iMX7 with NAND storage.

Adding 10k bytes of zeros provides an approximate value that makes the
proper padding for these NAND devices.

[1] 
https://lore.kernel.org/all/cac4tdfuqffqzrqfv5age_xtbfy1agr2sepn_fzedexhwjdr...@mail.gmail.com/

Signed-off-by: Hiago De Franco 
---
 arch/arm/mach-imx/Makefile | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/arch/arm/mach-imx/Makefile b/arch/arm/mach-imx/Makefile
index aebfa6517b..6904cf3880 100644
--- a/arch/arm/mach-imx/Makefile
+++ b/arch/arm/mach-imx/Makefile
@@ -142,7 +142,9 @@ u-boot.imx: u-boot.bin u-boot.cfgout $(PLUGIN).bin FORCE
$(call if_changed,mkimage)
 
 quiet_cmd_u-boot-nand_imx = GEN $@
-cmd_u-boot-nand_imx = (dd bs=1024 count=1 if=/dev/zero 2>/dev/null) | cat - $< 
> $@
+cmd_u-boot-nand_imx = (dd bs=1024 count=1 if=/dev/zero 2>/dev/null) | cat - $< 
> $@.zero-padded ; \
+ (dd bs=10k count=1 if=/dev/zero 2>/dev/null) | cat 
$@.zero-padded - > $@ ; \
+ rm -f $@.zero-padded
 
 u-boot-nand.imx: u-boot.imx FORCE
$(call if_changed,u-boot-nand_imx)
-- 
2.40.1