[coreboot] SST25VF016B (2MB) flash on m57sli (IT8716F).
Carl-Daniel Hailfinger
c-d.hailfinger.devel.2006 at gmx.net
Tue Jan 22 15:40:23 CET 2008
On 22.01.2008 12:08, Harald Gutmann wrote:
> Am Dienstag, 22. Januar 2008 01:09:09 schrieben Sie:
>
>> Harald: This patch should fix your problems writing to the chip. Use
>> either "patch -l" or remove the // before
>> //while (generic_spi_read_status_register() & JEDEC_RDSR_BIT_WIP)
>>
> I've updated my flashrom tree to revision 3068 and commented that line out.
> Now i can flash the chip, like Ronald says. I wrote the chip, and read the
> file again, and compared the start and the end of the file with
> hexdump/head/tail and diff. On the first/last 50000 lines there is no
> difference between the file which was written to the chip, and the file i've
> read with flashrom from the chip.
>
> Any idea why verifying fails?
>
Yes. Will send a patch.
>> Ronald: I need an ack to commit this.
>>
>> On 20.01.2008 11:59, Ronald Hoogenboom wrote:
>>
>>> Carl-Daniel Hailfinger wrote:
>>>
>>>> Did you have time to test with the while
>>>> (generic_spi_read_status_register() & .._WIP) enabled? It would be more
>>>> correct to have it enabled.
>>>>
>>> I had that in initially (that's why it was there, but commented out) but
>>> all it got me was slower programming times and the datasheet also
>>> suggests it is ok to just do a timed wait. But in the face of
>>> genericness for other chips, it is indeed more correct to do the check
>>> for the busy bit.
>>>
>> Ronald/Harald, can you please ack the change? It is reproduced below
>> (whichspace-damaged).
>>
>> Regards,
>> Carl-Daniel
>>
>> Make sure we delay writing the next byte long enough in SPI byte
>> programming.
>> Minor formatting changes.
>>
>> Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
>>
> Acked-by: Harald Gutmann <harald.gutmann at gmx.net>
>
Thanks, r3069.
Regards,
Carl-Daniel
More information about the coreboot
mailing list