Idwer noticed a few problems... these patches try to fix them.
Idwer please try one at the time and report if they work as intended:
the first one should fix the "no error" message,
the second one should stop the loop after 10k tries,
the third one should fix the underlying problem for good.
Stefan Tauner (3):
Fix error message in sp_openserport().
Break endless loop in serialport_write().
Fix sp_openserport users on Windows.
buspirate_spi.c | 2 +-
pony_spi.c | 2 +-
serial.c | 15 ++++++++++++---
serprog.c | 2 +-
4 files changed, 15 insertions(+), 6 deletions(-)
Kind regards, Stefan Tauner
flashrom v0.9.6.1-r1563 on Linux 3.5.0-19-generic (i686)
flashrom is free software, get the source code at http://www.flashrom.org
flashrom was built with libpci 3.1.9, GCC 4.7.1, little endian
Command line (3 args): flashrom -V -p internal
Calibrating delay loop... OS timer resolution is 2 usecs, 1728M loops
per second, 10 myus = 11 us, 100 myus = 101 us, 1000 myus = 998 us,
10000 myus = 14494 us, 8 myus = 9 us, OK.
Initializing internal programmer
No coreboot table found.
DMI string system-manufacturer: "Acer, inc."
DMI string system-product-name: "TravelMate 6231 "
DMI string system-version: "Not Applicable"
DMI string baseboard-manufacturer: "Acer, Inc."
DMI string baseboard-product-name: "Vico "
DMI string baseboard-version: "Not Applicable"
DMI string chassis-type: "Other"
DMI chassis-type is not specific enough.
W836xx enter config mode worked or we were already in config mode.
W836xx leave config mode had no effect.
Active config mode, unknown reg 0x20 ID: fc.
Please send the output of "flashrom -V" to flashrom(a)flashrom.org with
W836xx: your board name: flashrom -V
as the subject to help us finish support for your Super I/O. Thanks.
WARNING! You may be running flashrom on an unsupported laptop. We could
not detect this for sure because your vendor has not setup the SMBIOS
tables correctly. You can enforce execution by adding
'-p internal:laptop=this_is_not_a_laptop' to the command line, but
please read the following warning if you are not sure.
Laptops, notebooks and netbooks are difficult to support and we
recommend to use the vendor flashing utility. The embedded controller
(EC) in these machines often interacts badly with flashing.
See http://www.flashrom.org/Laptops for details.
If flash is shared with the EC, erase is guaranteed to brick your laptop
and write may brick your laptop.
Read and probe may irritate your EC and cause fan failure, backlight
failure and sudden poweroff.
You have been warned.
Error: Programmer initialization failed.
On Sat, 20 Oct 2012 23:31:52 -0400
John Schmerge <jbschmerge(a)gmail.com> wrote:
> Hi Stefan, i'm sorry for the rather slow response, i've been taking
> some time away from everything tech related. Someone you may wish to
> reach out to to ask about this is Ian Geiser at Devon IT (my former
> employer)... His email address is igeiser(a)devonit.com
is Devon still using flashrom with (newer) VIA chipsets?
We have included a number of improvements/support for additional
models in r1578*, but were not able to test them on real hardware but
VX900 with SPI. If you have access to affected boards, we would be glad
if you could at least run some probing runs (and send us the output).
Kind regards/Mit freundlichen Grüßen, Stefan Tauner
Am Samstag, den 24.11.2012, 18:44 +0100 schrieb Stefan Tauner:
> More clear "variable" names, better explanation if now programmer is selected etc.
> Signed-off-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at>
> cli_classic.c | 78 +++++++++++++++++++++++++--------------------------------
> flashrom.c | 1 -
> print.c | 1 +
> 3 files changed, 35 insertions(+), 45 deletions(-)
> diff --git a/cli_classic.c b/cli_classic.c
> index ccf75e9..0d28d15 100644
> --- a/cli_classic.c
> +++ b/cli_classic.c
> @@ -33,58 +33,42 @@
> static void cli_classic_usage(const char *name)
> - printf("Usage: flashrom [-h|-R|-L|"
> + printf("Please note that the command line interface for flashrom has changed between\n"
> + "0.9.5 and 0.9.6 and will change again before flashrom 1.0.\n\n");
> + printf("Usage: %s [-h|-R|-L|"
> #if CONFIG_PRINT_WIKI == 1
> - "-z|"
> + "-z|"
Indentation is changed?
I quickly looked over it and could not find any suspicious. It is hard
though as there are a lot of white space changes.
Acked-by: Paul Menzel <paulepanter(a)users.sourceforge.net>