Project

General

Profile

Actions

Support #357

closed

Feature #353: Release v1.3

Add Spansion S25FL512S

Added by Anastasia Klimchuk over 2 years ago. Updated about 2 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Release prep
Target version:
Start date:
04/28/2022
Due date:
% Done:

100%

Estimated time:
Affected versions:
Needs backport to:
Affected hardware:
Affected OS:

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.

Actions #1

Updated by Felix Singer over 2 years ago

  • Related links updated (diff)
Actions #2

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.

Actions #3

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.

Actions #4

Updated by Felix Singer about 2 years ago

  • Tracker changed from Feature to Support
Actions #5

Updated by Felix Singer about 2 years ago

  • Target version changed from 1.3 to main
Actions #6

Updated by Felix Singer about 2 years ago

  • Status changed from Resolved to Needs Testing

Nico, I assume this still needs to be tested?

Actions #7

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?

Actions #8

Updated by Anastasia Klimchuk about 2 years ago

  • Status changed from Needs Testing to Resolved
Actions #9

Updated by Anastasia Klimchuk about 2 years ago

  • % Done changed from 0 to 100
Actions

Also available in: Atom PDF