Bug fix: Make fastread to work with FRAM chip #203
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
Updated pull requests and issue templates
Latest stable changes from development - Custom SPI pins and multiple SPI support for ESP32
Final code for v3.4.0 from Development
Final code for v3.4.0 from Stable
The dummy byte needed for fastread operation need to be sent after the two-bytes address
Use _beginSPI() instead of _nextByte
7 tasks
Marzogh
added
bugfix
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.
Any pull request raised here MUST be submitted according to here MUST be submitted according to this template or it will be flagged with 'Not enough information'. No action will be taken till all the prerequisite information (according to the right template) is provided. If no information is provided for over a month after the 'Not enough information' label is applied, the issue will be closed.
Pull request details
What is the current behavior? (You can also link to an open issue here)
Fast read doesn't work with a FRAM chip. See comments on Add support for Microchip SST26VF016B & SST26VF032B and make fastread working #173
What is the new behavior? (if this is a feature change)
Fastread work again ...
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Other information:
DO NOT DELETE OR EDIT anything below this
Note 1: Make sure to add all the information needed to understand the bug so that someone can help. If any essential information is missing we'll add the 'Needs more information' label and close the issue until there is enough information.
Note 2: For support questions (for example, tutorials on how to use the library), please use the Arduino Forums. This repository's issues are reserved for feature requests and bug reports.