Skip to content

Conversation

@pdgendt
Copy link
Contributor

@pdgendt pdgendt commented Nov 28, 2024

Commit e63c6cd introduced device API
macros to be used by driver implementations. The DEVICE_API macro
ensures the passed API instance is placed in the corresponding iterable
section to allow for runtime checks.

Add wrapper DEVICE_API macro to all its_driver_api instances.

Copy link
Contributor

@ithinuel ithinuel left a comment

Choose a reason for hiding this comment

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

Hi, thank you for your contribution.

Could you please add some explanation of what motivated this change and what does it imply/what is its impact? (doing so in the commit message would be great)

@pdgendt
Copy link
Contributor Author

pdgendt commented Nov 28, 2024

Could you please add some explanation of what motivated this change and what does it imply/what is its impact? (doing so in the commit message would be great)

Done

Commit e63c6cd introduced device API
macros to be used by driver implementations. The DEVICE_API macro
ensures the passed API instance is placed in the corresponding iterable
section to allow for runtime checks.

Add wrapper DEVICE_API macro to all its_driver_api instances.

Signed-off-by: Pieter De Gendt <[email protected]>
@zephyrbot zephyrbot added the size: XS A PR changing only a single line of code label Nov 28, 2024
@pdgendt pdgendt requested review from ithinuel and kartben November 28, 2024 21:04
@pdgendt
Copy link
Contributor Author

pdgendt commented Dec 4, 2024

@ithinuel please revisit

@pdgendt
Copy link
Contributor Author

pdgendt commented Dec 5, 2024

@kartben PTAL

@kartben kartben merged commit b775c0d into zephyrproject-rtos:main Dec 5, 2024
26 checks passed
@pdgendt pdgendt deleted the linker-api-its branch December 5, 2024 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: Interrupt Controller platform: ARM Arm Limited size: XS A PR changing only a single line of code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants