[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