kselftest/alsa: Ensure _GNU_SOURCE is defined

broonie · shuahkh · commit 2032e61e24fe · 2024-05-29T12:23:57.000-06:00
The pcmtest driver tests use the kselftest harness which requires that _GNU_SOURCE is defined but nothing causes it to be defined. Since the KHDR_INCLUDES Makefile variable has had the required define added let's use that, this should provide some futureproofing. Fixes: daef47b ("selftests: Compile kselftest headers with -D_GNU_SOURCE") Signed-off-by: Mark Brown <broonie@kernel.org> Reviewed-by: Muhammad Usama Anjum <usama.anjum@collabora.com> Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
diff --git a/tools/testing/selftests/alsa/Makefile b/tools/testing/selftests/alsa/Makefile
@@ -1,7 +1,7 @@
 # SPDX-License-Identifier: GPL-2.0
 #
 
-CFLAGS += $(shell pkg-config --cflags alsa)
+CFLAGS += $(shell pkg-config --cflags alsa) $(KHDR_INCLUDES)
 LDLIBS += $(shell pkg-config --libs alsa)
 ifeq ($(LDLIBS),)
 LDLIBS += -lasound

Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,7 @@`
`1`	`1`	`# SPDX-License-Identifier: GPL-2.0`
`2`	`2`	`#`
`3`	`3`
`4`		`-CFLAGS += $(shell pkg-config --cflags alsa)`
	`4`	`+CFLAGS += $(shell pkg-config --cflags alsa) $(KHDR_INCLUDES)`
`5`	`5`	`LDLIBS += $(shell pkg-config --libs alsa)`
`6`	`6`	`ifeq ($(LDLIBS),)`
`7`	`7`	`LDLIBS += -lasound`