2020-09-04 20:15:07

by Denis Efremov (Oracle)

Subject: [PATCH 0/2] drop GZFLAGS definition

GZFLAGS is not used. KGZIP env var can be used to pass
additional flags to gzip instead.

Denis Efremov (2):
ARM: makefile: Drop GZFLAGS definition and export
csky: Drop GZFLAGS definition

arch/arm/Makefile | 3 +--
arch/csky/Makefile | 1 -
2 files changed, 1 insertion(+), 3 deletions(-)

--
2.26.2

2020-09-04 20:15:54

by Denis Efremov (Oracle)

Subject: [PATCH 1/2] ARM: makefile: Drop GZFLAGS definition and export

Drop the definition and export of GZFLAGS, because it's
not used. GZFLAGS was dropped from arm64 in commit
4cf234943dcf ("arm64: drop GZFLAGS definition and export").

Signed-off-by: Denis Efremov <[email protected]>
---
arch/arm/Makefile | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/arch/arm/Makefile b/arch/arm/Makefile
index 4e877354515f..3780063e0be0 100644
--- a/arch/arm/Makefile
+++ b/arch/arm/Makefile
@@ -20,7 +20,6 @@ ifeq ($(CONFIG_ARM_MODULE_PLTS),y)
KBUILD_LDS_MODULE += $(srctree)/arch/arm/kernel/module.lds
endif

-GZFLAGS :=-9
#KBUILD_CFLAGS +=-pipe

# Never generate .eh_frame
@@ -270,7 +269,7 @@ KBUILD_CPPFLAGS += $(patsubst %,-I$(srctree)/%include,$(machdirs) $(platdirs))
endif
endif

-export TEXT_OFFSET GZFLAGS MMUEXT
+export TEXT_OFFSET MMUEXT

core-y += arch/arm/
# If we have a machine-specific directory, then include it in the build.
--
2.26.2

2020-09-04 20:16:44

by Denis Efremov (Oracle)

Subject: [PATCH 2/2] csky: Drop GZFLAGS definition

Drop the definition of GZFLAGS because it's not used.

Signed-off-by: Denis Efremov <[email protected]>
---
arch/csky/Makefile | 1 -
1 file changed, 1 deletion(-)

diff --git a/arch/csky/Makefile b/arch/csky/Makefile
index 37f593a4bf53..715b839bf998 100644
--- a/arch/csky/Makefile
+++ b/arch/csky/Makefile
@@ -1,6 +1,5 @@
# SPDX-License-Identifier: GPL-2.0-only
OBJCOPYFLAGS :=-O binary
-GZFLAGS :=-9

ifdef CONFIG_CPU_HAS_FPU
FPUEXT = f
--
2.26.2

2020-09-05 16:48:39

by Masahiro Yamada

Subject: Re: [PATCH 1/2] ARM: makefile: Drop GZFLAGS definition and export

On Sat, Sep 5, 2020 at 5:14 AM Denis Efremov <[email protected]> wrote:
>
> Drop the definition and export of GZFLAGS, because it's
> not used. GZFLAGS was dropped from arm64 in commit
> 4cf234943dcf ("arm64: drop GZFLAGS definition and export").
>
> Signed-off-by: Denis Efremov <[email protected]>


Reviewed-by: Masahiro Yamada <[email protected]>


BTW, if you want Russell to pick up this patch,
you need to send it to:

[email protected]



> ---
> arch/arm/Makefile | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/arch/arm/Makefile b/arch/arm/Makefile
> index 4e877354515f..3780063e0be0 100644
> --- a/arch/arm/Makefile
> +++ b/arch/arm/Makefile
> @@ -20,7 +20,6 @@ ifeq ($(CONFIG_ARM_MODULE_PLTS),y)
> KBUILD_LDS_MODULE += $(srctree)/arch/arm/kernel/module.lds
> endif
>
> -GZFLAGS :=-9
> #KBUILD_CFLAGS +=-pipe
>
> # Never generate .eh_frame
> @@ -270,7 +269,7 @@ KBUILD_CPPFLAGS += $(patsubst %,-I$(srctree)/%include,$(machdirs) $(platdirs))
> endif
> endif
>
> -export TEXT_OFFSET GZFLAGS MMUEXT
> +export TEXT_OFFSET MMUEXT
>
> core-y += arch/arm/
> # If we have a machine-specific directory, then include it in the build.
> --
> 2.26.2
>


--
Best Regards
Masahiro Yamada

2020-09-05 16:49:58

by Masahiro Yamada

Subject: Re: [PATCH 2/2] csky: Drop GZFLAGS definition

On Sat, Sep 5, 2020 at 5:14 AM Denis Efremov <[email protected]> wrote:
>
> Drop the definition of GZFLAGS because it's not used.
>
> Signed-off-by: Denis Efremov <[email protected]>
> ---
> arch/csky/Makefile | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/csky/Makefile b/arch/csky/Makefile
> index 37f593a4bf53..715b839bf998 100644
> --- a/arch/csky/Makefile
> +++ b/arch/csky/Makefile
> @@ -1,6 +1,5 @@
> # SPDX-License-Identifier: GPL-2.0-only
> OBJCOPYFLAGS :=-O binary
> -GZFLAGS :=-9



Reviewed-by: Masahiro Yamada <[email protected]>




> ifdef CONFIG_CPU_HAS_FPU
> FPUEXT = f
> --
> 2.26.2
>


--
Best Regards
Masahiro Yamada