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
drivers: hwinfo: implemented hardware info support for Smartbond #64125
drivers: hwinfo: implemented hardware info support for Smartbond #64125
Conversation
e89b7cb
to
fa87c66
Compare
This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time. |
It's hard to make anyone here to review things, let's prolong for another 60 days. |
fa87c66
to
0216019
Compare
@kasjer thanks for the quick fixes. Were you able to test on HW? |
@kasjer are there boards in the tree where you could add the hwinfo as supported features? |
@alexanderwachter I tested it on da1469x_dk_pro, could you give me a hint on how add supported feature (I'm very new to zephyr) |
Just add - hwinfo here: |
Only reset cause is supported as there is no common unique id present on those chips. Unique ID can be put in OTP but there is no single specification for this. Signed-off-by: Jerzy Kasenberg <jerzy.kasenberg@codecoup.pl>
0216019
to
22b92e6
Compare
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.
Thanks @kasjer sorry for the delay!
@alexanderwachter I added hwinfo feature to both boards. I had to rebase branch to include da14695_dk_usb board though so force-pushed changes may look unrelated |
No problem I added this because I really needed to know reset cause in work on other PR so there was no rush, thanks for review and for your help with polishing things. |
Only reset cause is supported as there is no common unique id present on those chips.
Unique ID can be put in OTP but there is no single specification for this.