Skip to content
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

Added new ManID #229

Open
wants to merge 5 commits into
base: development
Choose a base branch
from
Open

Conversation

Stanimir-Petev
Copy link

@Stanimir-Petev Stanimir-Petev commented Sep 14, 2021

EN25QH16B(2A).pdf
Added EON manufacturer memory ID (0x1C) in the list of supported IDs inside the "SPIFlash.h" file and the respective preprocessor macro in the "defines.h" file.

Pull request details

  • Please check if the PR fulfills these requirements
  • The commit message/s explain/s all the changes clearly
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
  • Bug fix
  • Added feature
  • Documentation update
  • Other - Please explain here:
  • What is the current behavior? (You can also link to an open issue here)
    "FlashDiagnostics" and "TestFlash" sketches doesn't work with Eon memory EN25QH16B due to fail check of the supported Manufacturer ID in "SPIFlashIO.h ", "_chipID" method, lines 540-545 and the later check at line 563.
    A simple call of the flash.begin with a specified flash memory size as a parameter didn't make it work.

  • What is the new behavior? (if this is a feature change)
    As the manufacturer ID of Eon is added the check for supported ID passes and all other required information (namely capacity and supported) is provided and the sketches work properly

  • Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
    No

  • 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.


GitHub issue state GitHub issue title GitHub issue author GitHub issue label GitHub issue comments GitHub issue age GitHub issue last updateGitHub pull request check contexts

Marzogh and others added 5 commits April 20, 2019 22:55
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
Added EON manufacturer memory ID (0x1C) in the list of supported IDs inside the "SPIFlash.h" file and the respective preprocessor macro in the "defines.h" file.
Copy link
Owner

@Marzogh Marzogh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this :)

@Marzogh Marzogh self-assigned this Sep 16, 2021
@Marzogh Marzogh added the enhancement An enhancement to the library's function is proposed label Sep 16, 2021
@Marzogh Marzogh added this to the v3.4.1 milestone Sep 16, 2021
@Marzogh Marzogh changed the base branch from master to development September 16, 2021 04:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement to the library's function is proposed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants