From: Mathias Krause <mathias.krause@secunet.com>
Date: Wed, 8 Feb 2012 09:32:57 +0000 (+0100)
Subject: libpayload: code cosmetics
X-Git-Url: http://wien.tomnetworks.com/gitweb/?p=coreboot.git;a=commitdiff_plain;h=50759ed4ff438db6f3b093984fbb79d6445ebcb3

libpayload: code cosmetics

Be consistend with coding style at least within a function -- don't mix
sizeof with plain values.

Change-Id: Iefb5b7fe4f54977f5505fc9cea65c9c4af3e7f3a
Signed-off-by: Mathias Krause <mathias.krause@secunet.com>
Reviewed-on: http://review.coreboot.org/617
Tested-by: build bot (Jenkins)
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
---

diff --git a/payloads/libpayload/drivers/options.c b/payloads/libpayload/drivers/options.c
index 8161c22e1..7c762516f 100644
--- a/payloads/libpayload/drivers/options.c
+++ b/payloads/libpayload/drivers/options.c
@@ -309,7 +309,7 @@ int set_option_from_string(const struct nvram_accessor *nvram, struct cb_cmos_op
 	switch (cmos_entry->config) {
 		case 'h':
 			/* only works on little endian */
-			raw = malloc(8);
+			raw = malloc(sizeof(u64));
 			*(u64*)raw = strtoull(value, NULL, 0);
 			break;
 		case 's':