[flashrom] [PATCH] Unsignify lengths and addresses in chip functions and structs
Carl-Daniel Hailfinger
c-d.hailfinger.devel.2006 at gmx.net
Tue Sep 13 23:10:08 CEST 2011
Am 23.08.2011 17:12 schrieb Stefan Tauner:
> compile tested with gcc 4.4.5 and clang 2.8
I agree with the idea and support it.
However, I don't know how supportable this is on 16 bit platforms with
16 bit int. We need at least 24 bits to handle flash addresses, and we
could either create a new typedef chipoffset_t or specify uint32_t
explicitly. I favor a new typedef because it will make auditing easier
if we ever run on 16 bit programmers.
> Signed-off-by: Stefan Tauner <stefan.tauner at student.tuwien.ac.at>
A comment about the typedef idea would be appreciated. Anyway, this is
(even if you reject the explicit type idea)
Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
Regards,
Carl-Daniel
--
http://www.hailfinger.org/
More information about the flashrom
mailing list