diff mbox series

boot: image-pre-load: drop unused CONFIG_SYS_BOOTM_LEN

Message ID 20220507132306.10165-1-peng.fan@oss.nxp.com
State Accepted
Commit 3800b318c5303633f056bc6ab387ef01906cee57
Delegated to: Tom Rini
Headers show
Series boot: image-pre-load: drop unused CONFIG_SYS_BOOTM_LEN | expand

Commit Message

Peng Fan (OSS) May 7, 2022, 1:23 p.m. UTC 
From: Peng Fan <peng.fan@nxp.com>

CONFIG_SYS_BOOTM_LEN is not used in this file, drop it.

Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
 boot/image-pre-load.c | 5 -----
 1 file changed, 5 deletions(-)

Comments

Tom Rini May 8, 2022, 3:30 p.m. UTC | #1 
On Sat, May 07, 2022 at 09:23:05PM +0800, Peng Fan (OSS) wrote:

> From: Peng Fan <peng.fan@nxp.com>
> 
> CONFIG_SYS_BOOTM_LEN is not used in this file, drop it.
> 
> Signed-off-by: Peng Fan <peng.fan@nxp.com>

Reviewed-by: Tom Rini <trini@konsulko.com>
Peng Fan June 3, 2022, 6:44 a.m. UTC | #2 
Tom,

> Subject: [PATCH] boot: image-pre-load: drop unused CONFIG_SYS_BOOTM_LEN

You gave an R-b, but still not pick this up?

Thanks,
Peng.

> 
> From: Peng Fan <peng.fan@nxp.com>
> 
> CONFIG_SYS_BOOTM_LEN is not used in this file, drop it.
> 
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> ---
>  boot/image-pre-load.c | 5 -----
>  1 file changed, 5 deletions(-)
> 
> diff --git a/boot/image-pre-load.c b/boot/image-pre-load.c index
> 78d89069a98..5ab9ae18746 100644
> --- a/boot/image-pre-load.c
> +++ b/boot/image-pre-load.c
> @@ -23,11 +23,6 @@ DECLARE_GLOBAL_DATA_PTR;
>  #define IMAGE_PRE_LOAD_PROP_PUBLIC_KEY		"public-key"
>  #define IMAGE_PRE_LOAD_PROP_MANDATORY		"mandatory"
> 
> -#ifndef CONFIG_SYS_BOOTM_LEN
> -/* use 8MByte as default max gunzip size */
> -#define CONFIG_SYS_BOOTM_LEN	0x800000
> -#endif
> -
>  /*
>   * Information in the device-tree about the signature in the header
>   */
> --
> 2.36.0
Tom Rini June 3, 2022, 1:21 p.m. UTC | #3 
On Fri, Jun 03, 2022 at 06:44:56AM +0000, Peng Fan wrote:
> Tom,
> 
> > Subject: [PATCH] boot: image-pre-load: drop unused CONFIG_SYS_BOOTM_LEN
> 
> You gave an R-b, but still not pick this up?

It's not breaking anything is it?  I'll grab it to -next when that opens
next week if so.
Tom Rini June 7, 2022, 4:45 p.m. UTC | #4 
On Sat, May 07, 2022 at 09:23:05PM +0800, Peng Fan (OSS) wrote:

> From: Peng Fan <peng.fan@nxp.com>
> 
> CONFIG_SYS_BOOTM_LEN is not used in this file, drop it.
> 
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> Reviewed-by: Tom Rini <trini@konsulko.com>

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

Patch

diff --git a/boot/image-pre-load.c b/boot/image-pre-load.c
index 78d89069a98..5ab9ae18746 100644
--- a/boot/image-pre-load.c
+++ b/boot/image-pre-load.c
@@ -23,11 +23,6 @@  DECLARE_GLOBAL_DATA_PTR;
 #define IMAGE_PRE_LOAD_PROP_PUBLIC_KEY		"public-key"
 #define IMAGE_PRE_LOAD_PROP_MANDATORY		"mandatory"
 
-#ifndef CONFIG_SYS_BOOTM_LEN
-/* use 8MByte as default max gunzip size */
-#define CONFIG_SYS_BOOTM_LEN	0x800000
-#endif
-
 /*
  * Information in the device-tree about the signature in the header
  */