diff options
author | Mathias Krause <mathias.krause@secunet.com> | 2012-02-17 11:53:28 +0100 |
---|---|---|
committer | Stefan Reinauer <stefan.reinauer@coreboot.org> | 2012-02-17 19:24:11 +0100 |
commit | 632ab1ff6501e87b2b51da88f9dd9ca7b1b90e49 (patch) | |
tree | d8e36e22157d511c77abe4ed689e96dc93165e27 /src/northbridge/amd/amdfam10/raminit.h | |
parent | f01291dd64175b0c03a218f5ee4ad306ad779183 (diff) |
libpayload: fix compiler warning for first_cmos_entry()
The 'name' argument to lookup_cmos_entry() is declared to be 'char *'
but we pass an empty string ("") which is 'const char[]' so the compiler
legitimatly warns about discarded qualifiers here. Fix this by passing
NULL as 'name'.
Minor nitpick: The NULL test in lookup_cmos_entry() is superfluous as our
implementation of strnlen() can handle NULL pointers gracefully. But for
an average C hacker it just doesn't feel right not to do so.
Change-Id: I592917d12d8fa840804c0d19e38b844427064fef
Signed-off-by: Mathias Krause <mathias.krause@secunet.com>
Reviewed-on: http://review.coreboot.org/651
Tested-by: build bot (Jenkins)
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Diffstat (limited to 'src/northbridge/amd/amdfam10/raminit.h')
0 files changed, 0 insertions, 0 deletions