Support #371
closedCheck XMC spi flash chips definitions
90%
Description
Chips to check:
XM25QU64C
XM25QU128C
XM25QU256C
XM25QH64C
XM25QH128C
XM25QH256C
Specific questions:
1) Looks like copy-pasta from Winbond chips.
2) Might miss some block erasers.
3) Missing 4BA_WRITE, is this correct?
Updated by Anastasia Klimchuk over 2 years ago
- Assignee set to Anastasia Klimchuk
Updated by Anastasia Klimchuk about 2 years ago
After the discussion on the meeting, some answers to initial questions:
1) Yes, and this is WAI because these chips are copy-pasta of Winbond
3) This is correct.
Remaining question need to be answered is 2) which means checking the datasheets and compare with chip definitions for block erasers.
Updated by Felix Singer about 2 years ago
- Tracker changed from Feature to Support
- Target version changed from 1.3 to main
Updated by Liam Flaherty about 2 years ago
- Assignee changed from Anastasia Klimchuk to Liam Flaherty
Updated by Liam Flaherty about 2 years ago
2) I have checked the erasers, they are all correct and none are missing.
On #3, the smaller variants of the chip (<256Mbit) do not support 4BA_WRITE, however the XM25Qx256C chips do appear to. They do also work in a "backwards compatible" mode that doesn't require the 4-byte addressing (see section "5.1.5 3-Byte/4-Byte Address Modes" of the datasheet), so they can work without 4BA_WRITE beings set as well.
Updated by Anastasia Klimchuk about 2 years ago
Looks like this is resolved, can be closed. Let me double-check.
Updated by Felix Singer about 2 years ago
I understand most of the chip definitions are correct. Is anything left to do on the chip definitions for XM25QH256C and XM25QU256C regarding the 4BA mode?
Updated by Felix Singer about 2 years ago
- Status changed from New to Response Needed
Updated by Liam Flaherty about 2 years ago
I will add the FEATURE_4BA_WRITE flag for these chips.
Updated by Liam Flaherty almost 2 years ago
- Status changed from Response Needed to Resolved
4BA submitted under https://review.coreboot.org/c/flashrom/+/69713