Skip to content

QSPI bug with Portenta C33 #84

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.

Sign up for GitHub

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

Jump to bottom
Closed
umbynos opened this issue Aug 3, 2023 · 2 comments · Fixed by hathach/tinyusb#2245
Closed

QSPI bug with Portenta C33 #84

umbynos opened this issue Aug 3, 2023 · 2 comments · Fixed by hathach/tinyusb#2245
Assignees
@maidnl
Labels
conclusion: resolved Issue was resolved topic: code Related to content of the project itself type: imperfection Perceived defect in any part of project

Comments

@umbynos
Copy link
Contributor

umbynos commented Aug 3, 2023

While experimenting with Portenta C33 and certificate upload I discovered a bug 🐛 :
When a file is uploaded in the external QSPI of the board somehow it's corrupted.
To reproduce:

  1. flash the board with the UsbMsd/examples/MSD_QSPI/ example
  2. upload a file to the board using the new example in Add new example (we'll need it in the fwuploader) #2 or simply by copying to the mounted FS ( cp SSH_1.pdf /media/umberto/EB9F-C47C/SSH_1.pdf) ( the file I uploaded is here: SSH_1.pdf
  3. reset the board
  4. download the file from the mounted drive ( cp /media/umberto/EB9F-C47C/SSH_1.pdf SSH_1_new.pdf) (the file I got is here:
    SSH_1_new.pdf
  5. the files are different:
$ md5sum SSH_1.pdf
fba0dc994d857d5eee569ce2057b1b54  SSH_1.pdf
$ md5sum SSH_1_new.pdf 
b273108e7e4616afa33a5094d4c00068  SSH_1_new.pdf
  1. vbindiff SSH_1.pdf SSH_1_new.pdf:
    image

I don't know if that USBS in the vbindiff output could be a hint...
@umbynos umbynos added type: imperfection Perceived defect in any part of project topic: code Related to content of the project itself labels Aug 3, 2023
@facchinm
Copy link
Member

facchinm commented Aug 4, 2023

To be retested on top of #85

@maidnl maidnl mentioned this issue Sep 6, 2023
Merged
@pennam
Copy link
Contributor

pennam commented Sep 18, 2023

Works without issues with latest core release (1.0.4)

@pennam pennam closed this as completed Sep 18, 2023
@per1234 per1234 added the conclusion: resolved Issue was resolved label Sep 18, 2023
pennam pushed a commit to pennam/ArduinoCore-renesas that referenced this issue Jan 5, 2024
@facchinm
Merge pull request arduino#84 from bcmi-labs/analogWave_sample
7979e8c 
analogWave: add basic example
cristidragomir97 pushed a commit to cristidragomir97/ArduinoCore-renesas that referenced this issue May 20, 2024
@facchinm
Merge pull request arduino#84 from bcmi-labs/analogWave_sample
b5cbe76 
analogWave: add basic example
Former-commit-id: 7979e8c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@maidnl maidnl

Labels
conclusion: resolved Issue was resolved topic: code Related to content of the project itself type: imperfection Perceived defect in any part of project
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix msd extraneus bytes on reading arduino/tinyusb
5 participants
@per1234 @facchinm @pennam @umbynos @maidnl