[coreboot] r3474 - in trunk/payloads/libpayload: drivers i386 include
stepan at coresystems.de
Sat Aug 9 21:20:28 CEST 2008
Jordan Crouse wrote:
>> Weird, it compiles fine here. I strongly suggest changing the Makefile,
>> adding -Wno-pointer-sign. It's a dumb warning.
> Nak. This is not a dumb warning - I have seen it expose some real
Yes? Please exemplify
> And the memory compare functions should probably be taking void *.
Just checked. Normal libc memcmp takes void. I thought libpayload
functions were modelled after these. If not, that should be fixed anyways.
void * is a good idea here.
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: info at coresystems.de • http://www.coresystems.de/
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866
More information about the coreboot