Skip to content

Conversation

@yishai1999
Copy link
Contributor

Add wrapper DEVICE_API macro to all led_driver_api instances.

@pdgendt
Copy link
Contributor

pdgendt commented Nov 28, 2024

I think you missed drivers/led/ht16k33.c

pdgendt
pdgendt previously approved these changes Nov 28, 2024
simonguinot
simonguinot previously approved these changes Nov 28, 2024
@simonguinot simonguinot self-requested a review November 28, 2024 15:22
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suggestion: this notation is quite heavy. Since we are using it several times, couldn't we just get the api pointer ?

@simonguinot simonguinot self-requested a review November 28, 2024 15:23
fabiobaltieri
fabiobaltieri previously approved these changes Nov 29, 2024
@fabiobaltieri fabiobaltieri dismissed their stale review November 29, 2024 17:43

woops, missed the last comment

Add wrapper DEVICE_API macro to all led_driver_api instances.

Signed-off-by: Yishai Jaffe <[email protected]>
Filter for led devices when looking them up in dynamic shell commands.

Signed-off-by: Yishai Jaffe <[email protected]>
@pdgendt
Copy link
Contributor

pdgendt commented Dec 2, 2024

@simonguinot PTAL

@fabiobaltieri fabiobaltieri merged commit 589be4e into zephyrproject-rtos:main Dec 2, 2024
24 checks passed
@yishai1999 yishai1999 deleted the linker-api-led branch December 18, 2024 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: LED Label to identify LED subsystem area: Sensors Sensors platform: Microchip MEC Microchip MEC Platform

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants