Re: [flashrom] [PATCH] Unsignify lengths and addresses in chip functions and structs

[Stefan Tauner]

On Sun, 13 Nov 2011 23:23:39 +0100
Carl-Daniel Hailfinger  wrote:
> > /* Delay after "enter/exit ID mode" commands in microseconds. */
> > int probe_timing;
> > /* Delay after "enter/exit ID mode" commands in microseconds.
> > * NB: negative values have special meanings, see TIMING_* below.
> > */
> > signed int probe_timing;
>
> AFAICS the C standard defines that int is always signed. The only
> exception is char which is implementation-defined. Please kill signed
> here, but keep your comment.

this was on purpose to make it really, really obvious to anyone reading
the code that this is not just an unsigned delay (even if he ignores
the comment). :)
because you know... one could think that and change it in a patch which
gets acked and committed by accident and produce really odd bugs ;)

will review the other comments later, thanks!