diff mbox series

[v4,5/8] tools: imx8mimage: remove redundant code

Message ID 20200330035625.25164-5-kever.yang@rock-chips.com
State Accepted
Delegated to: Tom Rini
Headers show
Series [v4,1/8] tool: Move ALIGN_MASK to header as common MACRO | expand

Commit Message

Kever Yang March 30, 2020, 3:56 a.m. UTC
The align for fit_size has been done twice, remove the first one for it
does not make any sense.

Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
Reviewed-by: Punit Agrawal <punit1.agrawal@toshiba.co.jp>
Reviewed-by: Tom Rini <trini@konsulko.com>
---

Changes in v4: None
Changes in v3: None
Changes in v2: None

 tools/imx8mimage.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Tom Rini April 24, 2020, 5:10 p.m. UTC | #1
On Mon, Mar 30, 2020 at 11:56:21AM +0800, Kever Yang wrote:

> The align for fit_size has been done twice, remove the first one for it
> does not make any sense.
> 
> Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
> Reviewed-by: Punit Agrawal <punit1.agrawal@toshiba.co.jp>
> Reviewed-by: Tom Rini <trini@konsulko.com>

Applied to u-boot/master, thanks!
diff mbox series

Patch

diff --git a/tools/imx8mimage.c b/tools/imx8mimage.c
index 7defb13962..bc4ee793cb 100644
--- a/tools/imx8mimage.c
+++ b/tools/imx8mimage.c
@@ -341,7 +341,6 @@  static int generate_ivt_for_fit(int fd, int fit_offset, uint32_t ep,
 	}
 
 	fit_size = fdt_totalsize(&image_header);
-	fit_size = (fit_size + 3) & ~3;
 
 	fit_size = ALIGN(fit_size, ALIGN_SIZE);