drivers: led: Create led_core.c

In order to develop a sotfware implementation for led blinking,
prepare the led API.
This move z_impl_led_on and z_impl_led_off code to led_core.c
so we could use and update them for blinking a led.

Signed-off-by: Alexandre Bailon <abailon@baylibre.com>

Author: anobli

drivers/led/CMakeLists.txt

@@ -25,6 +25,8 @@ zephyr_library_sources_ifdef(CONFIG_PCA9633 pca9633.c)
 zephyr_library_sources_ifdef(CONFIG_TLC59108 tlc59108.c)
 # zephyr-keep-sorted-stop
 
+zephyr_library_sources(led_core.c)
+
 zephyr_library_sources_ifdef(CONFIG_LED_SHELL   led_shell.c)
 
 zephyr_library_sources_ifdef(CONFIG_USERSPACE   led_handlers.c)

drivers/led/led_core.c

@@ -0,0 +1,72 @@
+/*
+ * Copyright (c) 2025 BayLibre SAS
+ *
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+#include <zephyr/drivers/led.h>
+
+int z_impl_led_on(const struct device *dev, uint32_t led)
+{
+	const struct led_driver_api *api = (const struct led_driver_api *)dev->api;
+
+	if (api->set_brightness == NULL && api->on == NULL) {
+		return -ENOSYS;
+	}
+
+	if (api->on == NULL) {
+		return api->set_brightness(dev, led, LED_BRIGHTNESS_MAX);
+	}
+
+	return api->on(dev, led);
+}
+
+int z_impl_led_off(const struct device *dev, uint32_t led)
+{
+	const struct led_driver_api *api = (const struct led_driver_api *)dev->api;
+
+	if (api->set_brightness == NULL && api->off == NULL) {
+		return -ENOSYS;
+	}
+
+	if (api->off == NULL) {
+		return api->set_brightness(dev, led, 0);
+	}
+
+	return api->off(dev, led);
+}
+
+int z_impl_led_set_brightness(const struct device *dev, uint32_t led, uint8_t value)
+{
+	const struct led_driver_api *api = (const struct led_driver_api *)dev->api;
+
+	if (api->set_brightness == NULL) {
+		if (api->on == NULL || api->off == NULL) {
+			return -ENOSYS;
+		}
+	}
+
+	if (value > LED_BRIGHTNESS_MAX) {
+		return -EINVAL;
+	}
+
+	if (api->set_brightness == NULL) {
+		if (value) {
+			return api->on(dev, led);
+		} else {
+			return api->off(dev, led);
+		}
+	}
+
+	return api->set_brightness(dev, led, value);
+}
+
+int z_impl_led_blink(const struct device *dev, uint32_t led, uint32_t delay_on, uint32_t delay_off)
+{
+	const struct led_driver_api *api = (const struct led_driver_api *)dev->api;
+
+	if (api->blink == NULL) {
+		return -ENOSYS;
+	}
+	return api->blink(dev, led, delay_on, delay_off);
+}

include/zephyr/drivers/led.h

@@ -143,18 +143,6 @@ __subsystem struct led_driver_api {
 __syscall int led_blink(const struct device *dev, uint32_t led,
 			    uint32_t delay_on, uint32_t delay_off);
 
-static inline int z_impl_led_blink(const struct device *dev, uint32_t led,
-				   uint32_t delay_on, uint32_t delay_off)
-{
-	const struct led_driver_api *api =
-		(const struct led_driver_api *)dev->api;
-
-	if (api->blink == NULL) {
-		return -ENOSYS;
-	}
-	return api->blink(dev, led, delay_on, delay_off);
-}
-
 /**
  * @brief Get LED information
  *
@@ -200,34 +188,6 @@ static inline int z_impl_led_get_info(const struct device *dev, uint32_t led,
 __syscall int led_set_brightness(const struct device *dev, uint32_t led,
 				     uint8_t value);
 
-static inline int z_impl_led_set_brightness(const struct device *dev,
-					    uint32_t led,
-					    uint8_t value)
-{
-	const struct led_driver_api *api =
-		(const struct led_driver_api *)dev->api;
-
-	if (api->set_brightness == NULL) {
-		if (api->on == NULL || api->off == NULL) {
-			return -ENOSYS;
-		}
-	}
-
-	if (value > LED_BRIGHTNESS_MAX) {
-		return -EINVAL;
-	}
-
-	if (api->set_brightness == NULL) {
-		if (value) {
-			return api->on(dev, led);
-		} else {
-			return api->off(dev, led);
-		}
-	}
-
-	return api->set_brightness(dev, led, value);
-}
-
 /**
  * @brief Write/update a strip of LED channels
  *
@@ -327,22 +287,6 @@ static inline int z_impl_led_set_color(const struct device *dev, uint32_t led,
  */
 __syscall int led_on(const struct device *dev, uint32_t led);
 
-static inline int z_impl_led_on(const struct device *dev, uint32_t led)
-{
-	const struct led_driver_api *api =
-		(const struct led_driver_api *)dev->api;
-
-	if (api->set_brightness == NULL && api->on == NULL) {
-		return -ENOSYS;
-	}
-
-	if (api->on == NULL) {
-		return api->set_brightness(dev, led, LED_BRIGHTNESS_MAX);
-	}
-
-	return api->on(dev, led);
-}
-
 /**
  * @brief Turn off an LED
  *
@@ -357,22 +301,6 @@ static inline int z_impl_led_on(const struct device *dev, uint32_t led)
  */
 __syscall int led_off(const struct device *dev, uint32_t led);
 
-static inline int z_impl_led_off(const struct device *dev, uint32_t led)
-{
-	const struct led_driver_api *api =
-		(const struct led_driver_api *)dev->api;
-
-	if (api->set_brightness == NULL && api->off == NULL) {
-		return -ENOSYS;
-	}
-
-	if (api->off == NULL) {
-		return api->set_brightness(dev, led, 0);
-	}
-
-	return api->off(dev, led);
-}
-
 /*
  * LED DT helpers.
  */