-
Notifications
You must be signed in to change notification settings - Fork 232
sdwire3: initial implementation #1781
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
base: master
Are you sure you want to change the base?
sdwire3: initial implementation #1781
Conversation
0282d3b to
42af1ca
Compare
|
I how far does the SDWire3 differ from the SDWire and why can't the existing support for SDWire not be reused? |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #1781 +/- ##
========================================
- Coverage 45.2% 45.1% -0.1%
========================================
Files 172 173 +1
Lines 13569 13702 +133
========================================
+ Hits 6136 6184 +48
- Misses 7433 7518 +85
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
|
It is not using a FTDI for the configuration. It's using the sdwire version of Badgered https://github.com/Badger-Embedded/sdwire-cli. |
Emantor
left a comment
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.
Otherwise looks good.
doc/configuration.rst
Outdated
| USBSDWire3Device | ||
| ~~~~~~~~~~~~~~~~ | ||
| A :any:`USBSDWire3Device` resource describes a Tizen | ||
| `SD Wire device <https://web.archive.org/web/20240121081917/https://wiki.tizen.org/SDWire>`_. |
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.
This should link to an SDWire3 website, not the original sdwire website from tizen.
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.
My bad. Fixed that
Signed-off-by: Peter Johennecken <[email protected]>
42af1ca to
2679ed2
Compare
Adds a driver for the Badgerd SDWire3 based device SD Card multiplexer.
This device is very similar to the already existing SDWire by Tizen.
See: https://badgerd.nl/sdwirec/
Description
Checklist