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

Fixed writing unaligned sections that may overlap page boundaries of SPI-flash (GF180) #89

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

Conversation

shaos
Copy link

@shaos shaos commented Aug 2, 2023

Re-created PR for updated repo

One example that was written incorrectly by old version of utility:

...
@000009B0
00 00 00 00 01 00 00 00 00 00 00 00 02 00 00 00 
03 00 00 00 02 00 00 00 00 00 00 00 01 00 00 00 
00 00 00 00 02 00 00 00 03 00 00 00 02 00 00 00 
00 00 00 00 04 00 00 00 00 00 00 00 02 00 00 00 
06 00 00 00 02 00 00 00 00 00 00 00 04 00 00 00 
00 00 00 00 02 00 00 00 06 00 00 00 02 00 00 00 
12 00 00 00 02 00 00 00 00 00 00 00 08 00 00 00 
00 00 00 00 60 00 00 00 20 00 00 00 60 00 00 00 
40 00 00 00 60 00 00 00 62 00 00 00 E2 00 00 00 
62 00 00 00 60 00 00 00 62 00 00 00 E2 00 00 00 
E0 00 00 00 60 00 00 00 61 00 00 00 60 00 00 00 
61 00 00 00 60 00 00 00 FF FF FF FF 00 00 00 00 

Previously utility tried to write 256 bytes overlapping to next page 0xA00 and that extensive part was not written properly, corrupting beginning of the current page.

Now fixed utility is doing

  • erase the flash (entirely as before);
  • read file to buffer (filling holes with FFs and at the end up to the next page boundary);
  • write buffer to flash page by page (so no going through page boundaries while writing);
  • verify by reading flash page by page (comparing with buffer, not file).

@shaos
Copy link
Author

shaos commented Sep 19, 2023

Updated my branch one more time to match current state of efabless/caravel_board

Updating my repo to current state
add gerbers for panelized breakout; reorganize hardware directories
@shaos
Copy link
Author

shaos commented Oct 28, 2023

Updated my branch one more time to match current state of efabless/caravel_board

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant