[flashrom] [PATCH] Unsignify lengths and addresses in chip functions and structs
stefan.tauner at student.tuwien.ac.at
Wed Nov 23 10:14:41 CET 2011
On Wed, 23 Nov 2011 00:19:32 +0100
Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net> wrote:
> Am 22.11.2011 23:09 schrieb Stefan Tauner:
> > should i commit the attached version?
> Please do, the ack is still valid.
> I wonder whether we should enable the sign warnings once the cleanups
> and remaining conversions are in. The big problem might be platform
> (OS/library version) dependent changes which we don't know of, but OTOH
> we won't know until we try it.
i have been playing around with -Wsign-conversion for this patch... and
at least on my gcc version it is way over-sensitive, for example
ichspi.c: In function ‘ich7_run_opcode’:
ichspi.c:742: warning: negative integer implicitly converted to unsigned type
temp32 = REGREAD32(ICH7_REG_SPIA) & ~0x00FFFFFF;
together with -Werror -Wsign-conversion will probably explode somewhere
even if we try hard to get it right on dev platforms.
Kind regards/Mit freundlichen Grüßen, Stefan Tauner
More information about the flashrom