-
Notifications
You must be signed in to change notification settings - Fork 3k
Analog Devices: refactor CMake targets #14244
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Analog Devices: refactor CMake targets #14244
Conversation
@0xc0170, thank you for your changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please rename the CMake target names so they follow the convention and can be picked up by CMake.
I'll rebase all of my PRs soon, on it |
d8fd18d
to
ab7067d
Compare
Rebased and fixed |
Pull request has been modified.
bsp/xint/adi_xint.c | ||
|
||
${STARTUP_FILE} | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please link mbed-aducm4050
with mbed-analog-devices
here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
device/startup_ADuCM4050.c | ||
) | ||
|
||
target_link_libraries(mbed-ev-cog-ad4050lz INTERFACE mbed-analog-devices mbed-aducm4050) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
target_link_libraries(mbed-ev-cog-ad4050lz INTERFACE mbed-analog-devices mbed-aducm4050) | |
target_link_libraries(mbed-ev-cog-ad4050lz INTERFACE mbed-aducm4050) |
bsp/xint/adi_xint.c | ||
|
||
${STARTUP_FILE} | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please link mbed-aducm3029 with mbed-analog-devices here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
Fixed all |
Jenkins CI Test : ✔️ SUCCESSBuild Number: 1 | 🔒 Jenkins CI Job | 🌐 Logs & ArtifactsCLICK for Detailed Summary
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please rename the new files to CMakeLists.txt
Pull request has been modified.
Fix all CMakeLists now |
CI started |
Jenkins CI Test : ✔️ SUCCESSBuild Number: 2 | 🔒 Jenkins CI Job | 🌐 Logs & ArtifactsCLICK for Detailed Summary
|
CMake targets matching Mbed OS targets, refactor. Tested for both of these end-targets
Summary of changes
Impact of changes
Migration actions required
Documentation
Pull request type
Test results
Reviewers