Re: [flashrom] [PATCH] chipset_enable.c: add a message in the case the chipset enable is untested

[Paul Menzel]

Am Donnerstag, den 21.07.2011, 14:23 +0200 schrieb Stefan Tauner:
> old output:
> Calibrating delay loop... OK.
> Found chipset "Intel QS57", enabling flash write... OK.
> This chipset supports the following protocols: FWH, SPI.
>
> new non-verbose output for tested chipsets:
> Calibrating delay loop... OK.
> Found chipset "Intel QS57". Enabling flash write... OK.
> This chipset supports the following protocols: FWH, SPI.
>
> new non-verbose output for untested chipsets:
> Found chipset "Intel QS57".
> This chipset is marked as untested. If you are using an up-to-date version
> of flashrom please email a report to [e-mail] including a
> verbose (-V) log. Thank you!

A similar message should be shown somewhere else. I would put more
information in it.

This chipset is marked as untested. If you are using an up-to-date
version of flashrom http://flashrom.org/Downloads please email a
report to [e-mail] with the subject (variable with the name
(here Intel QS57) including a verbose (-V) log. Thank you!

Do we have preferences if the log should be pasted or attached?
> Enabling flash write... OK.
> This chipset supports the following protocols: FWH, SPI.
> Signed-off-by: Stefan Tauner
>
> chipset_enable.c |   19 ++++++++++++++-----
>  1 files changed, 14 insertions(+), 5 deletions(-)

[…]

Thanks,

Paul