cmake: Fix how compile definition is added

SebastianBoe · galak · commit 47914cb291b3 · 2019-10-02T07:38:26.000-07:00
The compile definition ENET_ENHANCEDBUFFERDESCRIPTOR_MODE has been
added by modifying the build environment of the 'zephyr' library. But
modifying this library is not supported.

The intention was to modify the global build environment, so we change
to use the zephyr_ API instead of the zephyr_library_ API as this API
will behave as intended.

Signed-off-by: Sebastian Bøe &lt;sebastian.boe@nordicsemi.no&gt;
diff --git a/mcux/drivers/kinetis/CMakeLists.txt b/mcux/drivers/kinetis/CMakeLists.txt
@@ -6,8 +6,9 @@
 
 zephyr_include_directories(.)
 
-zephyr_library_compile_definitions_ifdef(
-  CONFIG_PTP_CLOCK_MCUX ENET_ENHANCEDBUFFERDESCRIPTOR_MODE
+zephyr_compile_definitions_ifdef(
+  CONFIG_PTP_CLOCK_MCUX
+  ENET_ENHANCEDBUFFERDESCRIPTOR_MODE
 )
 
 zephyr_sources_ifdef(CONFIG_HAS_MCUX_CACHE    fsl_cache.c)

Original file line number	Diff line number	Diff line change
`@@ -6,8 +6,9 @@`
`6`	`6`
`7`	`7`	`zephyr_include_directories(.)`
`8`	`8`
`9`		`-zephyr_library_compile_definitions_ifdef(`
`10`		`- CONFIG_PTP_CLOCK_MCUX ENET_ENHANCEDBUFFERDESCRIPTOR_MODE`
	`9`	`+zephyr_compile_definitions_ifdef(`
	`10`	`+ CONFIG_PTP_CLOCK_MCUX`
	`11`	`+ ENET_ENHANCEDBUFFERDESCRIPTOR_MODE`
`11`	`12`	`)`
`12`	`13`
`13`	`14`	`zephyr_sources_ifdef(CONFIG_HAS_MCUX_CACHE fsl_cache.c)`