* Segher Boessenkool segher@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 (WARNINGONLY) \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.