Carl-Daniel Hailfinger wrote:
There's another question I have for you: How should we handle the situation if the opcode table has been locked by the BIOS? One way would be to load the locked opcode table from the southbridge and proceed as normal. If any opcodes we want are not in the table, fail the transaction with an error message. program_opcodes() could check the lock bit and return failure, then another function would malloc curopcodes and read the chipset opcodes into it.
Sounds reasonable.
By the way, some people have volunteered to test flashrom on ICH7 and I think ICH7/ICH8 support would be rather easy to add to the ICH9 driver, resulting in improved testing coverage. It's your decision, though, and I am very thankful you wrote the ICH9 driver.
How different are they wrt their SPI interface?