Skip to content

Conversation

@pdgendt
Copy link
Contributor

@pdgendt pdgendt commented Nov 27, 2024

Add wrapper DEVICE_API macro to all fpga_driver_api instances.

Copy link
Member

@cfriedt cfriedt left a comment

Choose a reason for hiding this comment

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

Looks good, but I would suggest holding off until @benediktibk 's ice40 PR is merged and then rebasing to avoid churn.

@pdgendt
Copy link
Contributor Author

pdgendt commented Nov 28, 2024

Looks good, but I would suggest holding off until @benediktibk 's ice40 PR is merged and then rebasing to avoid churn.

Sure, also note that the DEVICE_API macro is already in main, new drivers can already make use of it.

@pdgendt pdgendt force-pushed the linker-api-fpga branch 2 times, most recently from f4642e5 to bef0872 Compare November 28, 2024 21:07
Copy link
Contributor

Choose a reason for hiding this comment

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

Doesn't belong to this commit :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Whoops

Add wrapper DEVICE_API macro to all fpga_driver_api instances.

Signed-off-by: Pieter De Gendt <[email protected]>
@pdgendt
Copy link
Contributor Author

pdgendt commented Nov 29, 2024

Removed flash driver, rebased and added ice40 fpga drivers.

@kartben kartben merged commit 0c75641 into zephyrproject-rtos:main Nov 30, 2024
24 checks passed
@pdgendt pdgendt deleted the linker-api-fpga branch November 30, 2024 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: Flash area: FPGA Field-Programmable Gate Array (FPGA) platform: Microchip SAM Microchip SAM Platform (formerly Atmel SAM) platform: Xilinx AMD Xilinx

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants