[LinuxBIOS] r2489 - trunk/LinuxBIOSv2/util/flashrom
Stefan Reinauer
stepan at coresystems.de
Sun Nov 5 21:34:05 CET 2006
* Segher Boessenkool <segher at kernel.crashing.org> [061105 20:49]:
> Trivial in this case, but in general: can we please add
> the name or document number (not just the URL) for datasheets
> and other documentation? Vendors just _love_ to rearrange
> their websites (and some pull their old docs altogether);
> if we have the name/number (or even just the filename of
> the pdf) at least there's a chance to google for it again
> (either on the vendor's site or on some archive).
good hint. I added this to the devel guidelines.
> >+ Set bit 2: BIOSCS# Write Protect Enable (1=enable,
> >0=disable). */
>
> The bit is misnamed in the datasheet: it's the "write enable"
> bit, not the "write protect enable" (which would be "write disable").
nitpicking award 2006 ;-)
> >+ printf("tried to set 0x%x to 0x%x on %s failed (WARNING
> >ONLY) \n", xbcs, new, name);
>
> Warning only? It's pretty damn fatal :-) Or does this mean
> "trying to continue anyway" (which might not be such a great
> plan for a utility that can brick your system)?
it actually is. For the Via Epia you have to continue, because this
test basically fails every second time. But flashing works like a charme.
It's not that bad either, because if it really fails, you fail to write
to flash. In which case you dont brick your system.
> Would be nice if this message could be made a bit less cryptic,
> for example, "WARNING: failed setting %s register %#x to %#x\n".
> Yes I realise this is probably copied all over the place ;-)
Good idea. Or should we just drop this message all together? Or make it
printf_debug? If its an error, writing will fail later on anyways.
--
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/
More information about the coreboot
mailing list