[PATCH v2] drm/xe: replace 'grouped target' in Makefile with pattern rule

Lucas De Marchi lucas.demarchi at intel.com
Thu Feb 22 05:49:15 UTC 2024


On Wed, Feb 21, 2024 at 11:55:56AM +0200, Dafna Hirschfeld wrote:
>Since 'grouped target' is used only in 'make' 4.3, it should
>be avoided. Replace it with 'multi-target pattern rule' which
>has the same behavior.
>
>Signed-off-by: Dafna Hirschfeld <dhirschfeld at habana.ai>
>---
>v2 changes: instead of removing 'grouped target' we replace it with pattern-rule
>here is v1: https://lore.kernel.org/intel-xe/20240219135845.1463992-1-dhirschfeld@habana.ai/
>
> drivers/gpu/drm/xe/Makefile | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/drivers/gpu/drm/xe/Makefile b/drivers/gpu/drm/xe/Makefile
>index c531210695db..64903841fbd4 100644
>--- a/drivers/gpu/drm/xe/Makefile
>+++ b/drivers/gpu/drm/xe/Makefile
>@@ -42,7 +42,7 @@ generated_oob := $(obj)/generated/xe_wa_oob.c $(obj)/generated/xe_wa_oob.h
> quiet_cmd_wa_oob = GEN     $(notdir $(generated_oob))
>       cmd_wa_oob = mkdir -p $(@D); $^ $(generated_oob)
>
>-$(generated_oob) &: $(obj)/xe_gen_wa_oob $(srctree)/$(src)/xe_wa_oob.rules
>+$(obj)/generated/%.c $(obj)/generated/%.h:  $(obj)/xe_gen_wa_oob $(srctree)/$(src)/xe_wa_oob.rules

that means we can only ever have generated files for xe_wa_oob and
nothing else (or we would have to change this line too.

did you see any issue with the line I suggested?

Lucas De Marchi

> 	$(call cmd,wa_oob)
>
> uses_generated_oob := \
>-- 
>2.34.1
>


More information about the Intel-xe mailing list