2024-03-06 12:05:32

by Lukas Bulwahn

Subject: [PATCH] pps: use cflags-y instead of EXTRA_CFLAGS

Commit f77bf01425b1 ("kbuild: introduce ccflags-y, asflags-y and
ldflags-y") deprecates use of EXTRA_CFLAGS in the kernel build.

This has been cleaned up in the whole kernel tree long ago, but this one
single place must have been missed.

Replace the EXTRA_CFLAGS use by the common pattern for such debug flags.
No functional change.

Signed-off-by: Lukas Bulwahn <[email protected]>
---
Rodolfo, please ack.

Greg, please pick this minor cleanup patch.

drivers/pps/generators/Makefile | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/pps/generators/Makefile b/drivers/pps/generators/Makefile
index 2d56dd0495d5..2589fd0f2481 100644
--- a/drivers/pps/generators/Makefile
+++ b/drivers/pps/generators/Makefile
@@ -5,6 +5,4 @@

obj-$(CONFIG_PPS_GENERATOR_PARPORT) += pps_gen_parport.o

-ifeq ($(CONFIG_PPS_DEBUG),y)
-EXTRA_CFLAGS += -DDEBUG
-endif
+ccflags-$(CONFIG_PPS_DEBUG) := -DDEBUG
--
2.43.2

2024-03-06 12:22:47

by Rodolfo Giometti

Subject: Re: [PATCH] pps: use cflags-y instead of EXTRA_CFLAGS

On 06/03/24 13:05, Lukas Bulwahn wrote:
> Commit f77bf01425b1 ("kbuild: introduce ccflags-y, asflags-y and
> ldflags-y") deprecates use of EXTRA_CFLAGS in the kernel build.
>
> This has been cleaned up in the whole kernel tree long ago, but this one
> single place must have been missed.
>
> Replace the EXTRA_CFLAGS use by the common pattern for such debug flags.
> No functional change.
>
> Signed-off-by: Lukas Bulwahn <[email protected]>

Acked-by: Rodolfo Giometti <[email protected]>

> ---
> Rodolfo, please ack.
>
> Greg, please pick this minor cleanup patch.
>
> drivers/pps/generators/Makefile | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/pps/generators/Makefile b/drivers/pps/generators/Makefile
> index 2d56dd0495d5..2589fd0f2481 100644
> --- a/drivers/pps/generators/Makefile
> +++ b/drivers/pps/generators/Makefile
> @@ -5,6 +5,4 @@
>
> obj-$(CONFIG_PPS_GENERATOR_PARPORT) += pps_gen_parport.o
>
> -ifeq ($(CONFIG_PPS_DEBUG),y)
> -EXTRA_CFLAGS += -DDEBUG
> -endif
> +ccflags-$(CONFIG_PPS_DEBUG) := -DDEBUG

--
GNU/Linux Solutions e-mail: [email protected]
Linux Device Driver [email protected]
Embedded Systems phone: +39 349 2432127
UNIX programming