* Segher Boessenkool <[EMAIL PROTECTED]> [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: [EMAIL PROTECTED] • http://www.coresystems.de/
--
linuxbios mailing list
[email protected]
http://www.openbios.org/mailman/listinfo/linuxbios