Support #357
closedFeature #353: Release v1.3
Add Spansion S25FL512S
100%
Description
Ref commit https://github.com/flashrom/flashrom/commit/05c629be2964bcee368c03d805747da15281856d
What to look at: flash would be fully erased with programmers that lack native 4BA command support? Needs investigation.
Updated by Nico Huber over 2 years ago
- Status changed from New to In Progress
- Assignee set to Nico Huber
- Related links updated (diff)
It turned out that the flash chip entry missed to advertise the extended-address register (EAR) because support for the instructions used by this flash chip was hard to see (one would have to use the .wrea_override
field which isn't very obvious). Both ISSI and Spansion support the same alternative instructions to access the EAR (0x17/0x16). Beside this and the default (0xc5/0xc8), no other instruction sets are known to be used. So I wrote patches to replace this field with a feature flag for the alternative instruction set.
Updated by Nico Huber over 2 years ago
- Status changed from In Progress to Resolved
Patches are merged, let's see if we can find somebody to test.
Updated by Felix Singer about 2 years ago
- Tracker changed from Feature to Support
Updated by Felix Singer about 2 years ago
- Target version changed from 1.3 to main
Updated by Felix Singer about 2 years ago
- Status changed from Resolved to Needs Testing
Nico, I assume this still needs to be tested?
Updated by Nico Huber about 2 years ago
Felix Singer wrote in #note-6:
Nico, I assume this still needs to be tested?
We use the .tested
field in the flash database to keep track of the test status. So there's no need to do so here, is there?
Updated by Anastasia Klimchuk about 2 years ago
- Status changed from Needs Testing to Resolved