[coreboot] r3300 - trunk/util/flashrom
Stefan Reinauer
stepan at coresystems.de
Wed May 14 00:08:25 CEST 2008
Carl-Daniel Hailfinger wrote:
> On 13.05.2008 23:53, Stefan Reinauer wrote:
>
>> * svn at coreboot.org <svn at coreboot.org> [080512 23:19]:
>>
>>
>>> Author: hailfinger
>>> Date: 2008-05-12 23:19:53 +0200 (Mon, 12 May 2008)
>>> New Revision: 3300
>>>
>>> Modified: trunk/util/flashrom/flash.h
>>> ===================================================================
>>> --- trunk/util/flashrom/flash.h 2008-05-12 14:25:31 UTC (rev 3299)
>>> +++ trunk/util/flashrom/flash.h 2008-05-12 21:19:53 UTC (rev 3300)
>>> @@ -66,6 +66,7 @@
>>> #define TEST_OK_READ (1<<1)
>>> #define TEST_OK_ERASE (1<<2)
>>> #define TEST_OK_WRITE (1<<3)
>>> +#define TEST_OK_PREW (TEST_OK_PROBE|TEST_OK_READ|TEST_OK_ERASE|TEST_OK_WRITE)
>>>
>>>
>> Can we call this TEST_OK_ALL or something? PREW is not exactly
>> intuitive.
>>
>>
>
> ProbeReadEraseWrite is the best I could come up with. Once we support
> sector-based unlocking and sector-based erasing, we might add
> TEST_OK_UNLOCK and TEST_OK_SECTOR_ERASE. Having TEST_OK_ALL as alias
> would mean we assume any new functionality is supported on all chips.
> That would defeat the purpose of such support statements.
>
Then let's not use PREW. Or we will have to end up providing all
different kinds of combinations.
--
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: info at coresystems.de • http://www.coresystems.de/
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866
More information about the coreboot
mailing list