diff mbox series

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

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

Commit Message

Kever Yang March 26, 2020, 10:09 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>
---

Changes in v3: None
Changes in v2: None

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

Comments

Tom Rini March 27, 2020, 6:16 p.m. UTC | #1 
On Thu, Mar 26, 2020 at 06:09:13PM +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: Tom Rini <trini@konsulko.com>
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);