On Fri, Oct 20, 2023 at 08:09:12PM +0200, Ahmad Fatoum wrote:
> The XZ decompressor in barebox has BCJ filter support for THUMB, but
> like Linux, it's unused and instead the ARM BCJ filter is used.
>
> Fixing that lets us save a few kilobytes, which is especially useful for
> legacy configuration that fit a whole barebox into limited on-chip SRAM
> (and not only the prebootloader). For example, with this patch applied
> barebox-am33xx-beaglebone-mlo.img is shrinked 3K from 109236 to 106028 bytes.
>
> Signed-off-by: Ahmad Fatoum
> ---
> lib/decompress_unxz.c | 2 ++
> scripts/xz_wrap.sh| 4
> 2 files changed, 6 insertions(+)
Applied, thanks
Sascha
>
> diff --git a/lib/decompress_unxz.c b/lib/decompress_unxz.c
> index 132ab4a239b3..ad6a5f20ba28 100644
> --- a/lib/decompress_unxz.c
> +++ b/lib/decompress_unxz.c
> @@ -133,6 +133,8 @@
> #ifdef CONFIG_ARM
> #ifdef CONFIG_CPU_64
> #define XZ_DEC_ARM64
> +#elif defined CONFIG_THUMB2_BAREBOX
> +#define XZ_DEC_ARMTHUMB
> #else
> #define XZ_DEC_ARM
> #endif
> diff --git a/scripts/xz_wrap.sh b/scripts/xz_wrap.sh
> index 5b5f3adcff6d..a6373a748168 100755
> --- a/scripts/xz_wrap.sh
> +++ b/scripts/xz_wrap.sh
> @@ -20,6 +20,10 @@ case $SRCARCH in
> sparc) BCJ=--sparc ;;
> esac
>
> +if grep -q '^CONFIG_THUMB2_BAREBOX=y$' include/config/auto.conf; then
> + BCJ=--armthumb
> +fi
> +
> # clear BCJ filter if unsupported
> xz -H | grep -q -- $BCJ || BCJ=
>
> --
> 2.39.2
>
>
>
--
Pengutronix e.K. | |
Steuerwalder Str. 21 | http://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0|
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917- |
