[flashrom] [PATCH] Silence ICH preop debug message
Luc Verhaegen
libv at skynet.be
Thu Oct 22 13:05:44 CEST 2009
On Thu, Oct 22, 2009 at 04:01:27AM +0200, Carl-Daniel Hailfinger wrote:
> Since we don't have any debug level printing infrastructure yet, I
> propose to kill the obnoxious debug message in ichspi.c which was added
> to check for correct PREOP handling. We know the code works fine (after
> getting a few reports over 100 MB long) and there's no point in keeping
> it around anymore.
> If there is any desire, we can reinstate it as print_spew or whatever
> once the debug level infrastructure is merged, but at that point we
> probably just are happy that the debug output isn't there anymore.
>
> Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
>
> Index: flashrom-ich_preop_silence/ichspi.c
> ===================================================================
> --- flashrom-ich_preop_silence/ichspi.c (Revision 752)
> +++ flashrom-ich_preop_silence/ichspi.c (Arbeitskopie)
> @@ -756,11 +756,8 @@
> * opcode of the next command?
> */
> if ((oppos != -1) && (preoppos != -1) &&
> - (curopcodes->opcode[oppos].atomic - 1 == preoppos)) {
> - printf_debug("opcode 0x%02x will be run as PREOP\n",
> - cmds->writearr[0]);
> + (curopcodes->opcode[oppos].atomic - 1 == preoppos))
> continue;
> - }
> }
>
> ret = ich_spi_send_command(cmds->writecnt, cmds->readcnt,
>
>
> --
> Developer quote of the week:
> "We are juggling too many chainsaws and flaming arrows and tigers."
Simple enough for me to fully grasp, and yes, we should not have logs
that noisy any more.
Acked-by: Luc Verhaegen <libv at skynet.be>
More information about the flashrom
mailing list