-
Notifications
You must be signed in to change notification settings - Fork 3k
Adding Samsung Bioprocessor S1SBP6A target #13392
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
Conversation
@heuisam, 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.
I found few styling issues, otherwise looks fine to me
extern const PinMap PinMap_SPI_SSEL[]; | ||
#endif | ||
|
||
#endif |
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 add a newline at the end of file (other files in this pull request have it)
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.
Thank you for your comment. I'll fix it.
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.
If you fix, please comment in the pull request - what has been updated. I can see it was forced pushed but github still shows here "no new line"
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.
I'm sorry it was missed.
I'll update it and add comment for changes.
targets/targets.json
Outdated
"S1SBP6A": { | ||
"inherits": ["Target"], | ||
"core": "Cortex-M4F", | ||
"supported_toolchains": ["GCC_ARM", "ARMC6"], |
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.
can these be on new lines same as device_has
?
Also line 6571
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.
Thank you. I'll change it.
The TARGET_Samsung had security_subsystem and SIDK_S5JS100. If I add another target that does not use security_subsystem, I get a build error. This commit moves security_subsyste to TARGET_SIDK_S5JS100. Signed-off-by: Heuisam Kwag <[email protected]>
CI started |
Jenkins CI Test : ❌ FAILEDBuild Number: 1 | 🔒 Jenkins CI Job | 🌐 Logs & ArtifactsCLICK for Detailed Summary
|
Signed-off-by: Heuisam Kwag <[email protected]>
Change List
|
CI restarted |
Jenkins CI Test : ✔️ SUCCESSBuild Number: 2 | 🔒 Jenkins CI Job | 🌐 Logs & ArtifactsCLICK for Detailed Summary
|
Summary of changes
Adding a new target of HW development kit using Samsung Bioprocessor S1SBP6A to mMbed-OS.
S1SBP6A is a low-power bio-porcessor targeting multiple bio-sensor data acquisition and processing for healthcare and other applications.
Moving security_subsystem into TARGET_SIDK_S5JS100 to fix build error when S1SBP6A target is added.
Main author : Heuisam Kwag [email protected]
Documentation
Pull request type
Test results
arm
gcc
Reviewers