diff mbox series

[PATCHv5,2/7] board: tbs2910: disable CONFIG_GZIP

Message ID 20200530032425.10670-2-GNUtoo@cyberdimension.org
State Accepted
Commit b3e5d2937972a4385ff665366253244b25f0c0a2
Delegated to: Tom Rini
Headers show
Series [PATCHv5,1/7] board: tbs2910: disable loadb and loads commands | expand

Commit Message

Denis 'GNUtoo' Carikli May 30, 2020, 3:24 a.m. UTC 
As that the resulting image is already very close to the
size limit, and that CONFIG_GZIP is not strictly required,
removing it shouldn't hurt.

With arm-linux-gnueabi-gcc 9.2.0-1 from the Parabola
GNU/Linux distribution we have the following size
reduction:
- text: 9752
- data: 0
- bss: 16
- total: 9768

Signed-off-by: Denis 'GNUtoo' Carikli <GNUtoo@cyberdimension.org>
Acked-by: Soeren Moch <smoch@web.de>
---
Changelog since v4:
-------------------
- Added the following tag:
  Acked-by: Soeren Moch <smoch@web.de>
---
 configs/tbs2910_defconfig | 1 +
 1 file changed, 1 insertion(+)

Comments

Tom Rini June 12, 2020, 9:17 p.m. UTC | #1 
On Sat, May 30, 2020 at 05:24:20AM +0200, Denis 'GNUtoo' Carikli wrote:

> As that the resulting image is already very close to the
> size limit, and that CONFIG_GZIP is not strictly required,
> removing it shouldn't hurt.
> 
> With arm-linux-gnueabi-gcc 9.2.0-1 from the Parabola
> GNU/Linux distribution we have the following size
> reduction:
> - text: 9752
> - data: 0
> - bss: 16
> - total: 9768
> 
> Signed-off-by: Denis 'GNUtoo' Carikli <GNUtoo@cyberdimension.org>
> Acked-by: Soeren Moch <smoch@web.de>

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

Patch

diff --git a/configs/tbs2910_defconfig b/configs/tbs2910_defconfig
index 2623eb73cd..cc459e6de6 100644
--- a/configs/tbs2910_defconfig
+++ b/configs/tbs2910_defconfig
@@ -88,5 +88,6 @@  CONFIG_USB_GADGET_DOWNLOAD=y
 CONFIG_I2C_EDID=y
 CONFIG_VIDEO_IPUV3=y
 CONFIG_VIDEO=y
+# CONFIG_GZIP is not set
 CONFIG_OF_LIBFDT_ASSUME_MASK=0xff
 # CONFIG_EFI_LOADER is not set