Update to match the current SPI-Py API #85
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 2 commits
December 20, 2019 13:16
MFRC522.py is the only file with CR/LF (DOS) line endings - all other files are LF-only (UNIX). Fix the anomaly. This is a whitespace- only change. Signed-off-by: Phil Elwell <[email protected]>
A change to the SPI-Py API added a device-handle parameter to the transfer method. Signed-off-by: Phil Elwell <[email protected]>
wahajmurtaza
approved these changes
Feb 20, 2020
There was a problem hiding this comment.
Did the same for connecting multiple devices, but later found this. Works on multiple devices and latest SPI-Py. Must be merged so others may not face Issue.
Before it to connect multiple devices, had to open SPI every time to read or write and then the code crashes saying too many devices to open as closing spi was not supported
|
@wahajmurtaza how do you manage to make it work with multiple devices? i'm trying to connect 2 mfrc522 to a single raspberry (using a SDA to CE0 for the first and SDA to CE1 for the second) but i can't make the second reader work. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The SPI-Py API has changed, and clients need updating to match. The patch to do so is preceded by a whitespace-only commit fixing the line endings.