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

Bug report - Compatibility with nRF52840 #225

Open
2 tasks
TheSecondEmperor opened this issue Jul 12, 2021 · 0 comments
Open
2 tasks

Bug report - Compatibility with nRF52840 #225

TheSecondEmperor opened this issue Jul 12, 2021 · 0 comments
Assignees
Labels
possible bug Possibly a bug. Needs verification before moving to *bug* status.

Comments

@TheSecondEmperor
Copy link

TheSecondEmperor commented Jul 12, 2021

Any bug report raised 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 is provided. If no information is provided for over a month after the 'Not enough information' label is applied, the issue will be closed.

Pre-bug report checklist:

Do this checklist before filing an bug report:

  • Is this something you can debug and fix? Send a pull request! Bug fixes and documentation fixes are welcome.
  • Is this an idea for a feature? Post it as a Feature request. NOT a bug report.
  • [ x ] Is this a bug that you cannot fix? Go ahead with filing a bug report below

Bug Report

Describe the bug

Include a clear and concise description of what the bug is.
I am trying to use the flash chip W25Q128JVSIQ to communicate with an Arduino Nano 33 BLE which isn't explicitly supported by this library. However, I added
|| defined(ARDUINO_ARCH_NRF52840) to line 106 of SPIMemory.h which solved the util.delay.h errors I was getting when compiling the TestFlash.ino sketch which was suggested by the Arduino Forum in this thread.
The trouble is that after uploading the sketch the microcontroller fails to communicate with the flash chip indicated by error "2" in the Diagnostics of the serial monitor. I changed some things in the sketch as indicated below to help identify the chip. I also uncommented line 33 the SFDP line in SPIMemory.h which is supported by this chip as described in its datasheet. Chip Select has been set to 1 as I believe TX on the Nano 33 BLE is pin 1. I have tested continuity with all of the pins on the flash.

  • Make sure you have run FlashDiagnostics.ino with #define RUNDIAGNOSTICS uncommented in SPIFlash.h. Paste the relevant sections of the output from your Serial console when you run FlashDiagnostics.ino.here:
Initialising..........
Chip Diagnostics initiated.

Check your wiring. Flash chip is non-responsive.
If this does not help resolve/clarify this issue, please raise an issue at http://www.github.com/Marzogh/SPIMemory/issues with the details of what your were doing when this error occurred
The Flash chip does not support SFDP.
Unable to identify chip. Are you sure this chip is supported?
If this does not help resolve/clarify this issue, please raise an issue at http://www.github.com/Marzogh/SPIMemory/issues with the details of what your were doing when this error occurred
2
  • If you have a problem with a particular function, call the flash.error() function (after you have made sure you have started up your Serial port with a Serial.begin(BAUD) ). Provide details of the function, the data given to/ expected from the function and the error code here: (Please repeat this for every function you have an error with)
    - Function: (e.g. writeByte())
    - Data: (e.g. 3.14 or Struct)
    - Error code: (e.g. 0x0A)

To Reproduce

Provide a minimal code snippet example that reproduces the bug (If you're using one of the examples that came with the library, just tell us which one instead of pasting the entire example's code). Please make sure you wrap any code in the proper code blocks like below

TestFlash.ino,
line 100 is changed to,

SPIFlash flash(1);

line 106 has been given this addition,

|| defined(ARDUINO_ARCH_NRF52840)

line 117 was given the addition,

 Serial.println(flash.error());

Expected behavior

A clear and concise description of what you expected to happen.

I expected the sketch to upload and then print the scripts in the setup as well as the commands. The initialization failed with the "2" error indicating a failure to communicate with the chip. After entering "1" into the command line, I expected the JEDECID to be presented. Instead, the JEDECID was represented as zeroes.

Screenshots

If applicable, add screenshots to help explain your problem.
Screenshot (157)

Wiring diagram

If applicable, add a wiring diagram to help explain your problem.

Software environment (please complete the following information):

  • Library version: [v3.0.1]
  • Arduino IDE version: [1.8.15]
  • OS: [Windows 10]
  • Micro controller platform: [Arduino Nano 33 BLE]
  • Flash/Fram memory module: [IS25LP064A-JBLE]

Additional context

Add any other context about the problem here.


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 update

@TheSecondEmperor TheSecondEmperor added the possible bug Possibly a bug. Needs verification before moving to *bug* status. label Jul 12, 2021
@TheSecondEmperor TheSecondEmperor changed the title Bug report - *IS25LP064A compatibility with Nano 33 BLE* Bug report - Compatibility with nRF52840 Jul 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
possible bug Possibly a bug. Needs verification before moving to *bug* status.
Projects
None yet
Development

No branches or pull requests

2 participants