[coreboot-gerrit] Patch set updated for coreboot: src/drivers: Remove whitespace after memcpy & memset

HAOUAS Elyes (ehaouas@noos.fr) gerrit at coreboot.org
Mon Oct 3 22:25:09 CEST 2016


HAOUAS Elyes (ehaouas at noos.fr) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/16866

-gerrit

commit 5461e39fa42e329d8aa9b2032729e88572ff0789
Author: Elyes HAOUAS <ehaouas at noos.fr>
Date:   Mon Oct 3 22:19:16 2016 +0200

    src/drivers: Remove whitespace after memcpy & memset
    
    Change-Id: If79eb706b6d44f7c34dfe31a1545f5850870b334
    Signed-off-by: Elyes HAOUAS <ehaouas at noos.fr>
---
 src/drivers/i2c/at24rf08c/lenovo_serials.c | 14 +++++++-------
 src/drivers/intel/gma/vbt.c                |  8 ++++----
 2 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/src/drivers/i2c/at24rf08c/lenovo_serials.c b/src/drivers/i2c/at24rf08c/lenovo_serials.c
index 881bf53..d0d006f 100644
--- a/src/drivers/i2c/at24rf08c/lenovo_serials.c
+++ b/src/drivers/i2c/at24rf08c/lenovo_serials.c
@@ -104,7 +104,7 @@ const char *lenovo_mainboard_partnumber(void)
 	if (already_read)
 		return result;
 
-	memset (result, 0, sizeof (result));
+	memset(result, 0, sizeof (result));
 	at24rf08c_read_string(0, 0x27, 7, result);
 
 	already_read = 1;
@@ -129,17 +129,17 @@ void smbios_mainboard_set_uuid(u8 *uuid)
 
 
 	if (already_read) {
-		memcpy (uuid, result, 16);
+		memcpy(uuid, result, 16);
 		return;
 	}
 
-	memset (result, 0, sizeof (result));
+	memset(result, 0, sizeof (result));
 
 	dev = dev_find_slot_on_smbus(1, 0x56);
 	if (dev == NULL) {
 		printk(BIOS_WARNING, "EEPROM not found\n");
 		already_read = 1;
-		memset (uuid, 0, 16);
+		memset(uuid, 0, 16);
 		return;
 	}
 
@@ -155,7 +155,7 @@ void smbios_mainboard_set_uuid(u8 *uuid)
 				break;
 		}
 		if (t < 0) {
-			memset (result, 0, sizeof (result));
+			memset(result, 0, sizeof (result));
 			break;
 		}
 		result[remap[i]] = t;
@@ -163,7 +163,7 @@ void smbios_mainboard_set_uuid(u8 *uuid)
 
 	already_read = 1;
 
-	memcpy (uuid, result, 16);
+	memcpy(uuid, result, 16);
 }
 
 const char *smbios_mainboard_version(void)
@@ -176,7 +176,7 @@ const char *smbios_mainboard_version(void)
 	if (already_read)
 		return result;
 
-	memset (result, 0, sizeof (result));
+	memset(result, 0, sizeof (result));
 
 	dev = at24rf08c_find_bank(2);
 	if (dev == NULL) {
diff --git a/src/drivers/intel/gma/vbt.c b/src/drivers/intel/gma/vbt.c
index 9ec09d3..430aa9b 100644
--- a/src/drivers/intel/gma/vbt.c
+++ b/src/drivers/intel/gma/vbt.c
@@ -32,7 +32,7 @@ static size_t generate_vbt(const struct i915_gpu_controller_info *conf,
 	struct bdb_general_features *genfeat;
 	u8 *ptr;
 
-	memset (head, 0, sizeof (*head));
+	memset(head, 0, sizeof (*head));
 
 	memset(head->signature, ' ', sizeof(head->signature));
 	memcpy(head->signature, idstr, MIN(strlen(idstr),
@@ -42,8 +42,8 @@ static size_t generate_vbt(const struct i915_gpu_controller_info *conf,
 	head->bdb_offset = sizeof (*head);
 
 	bdb_head = (struct bdb_header *) (head + 1);
-	memset (bdb_head, 0, sizeof (*bdb_head));
-	memcpy (bdb_head->signature, "BIOS_DATA_BLOCK ", 16);
+	memset(bdb_head, 0, sizeof (*bdb_head));
+	memcpy(bdb_head->signature, "BIOS_DATA_BLOCK ", 16);
 	bdb_head->version = 0xa8;
 	bdb_head->header_size = sizeof (*bdb_head);
 
@@ -55,7 +55,7 @@ static size_t generate_vbt(const struct i915_gpu_controller_info *conf,
 	ptr += 3;
 
 	genfeat = (struct bdb_general_features *) ptr;
-	memset (genfeat, 0, sizeof (*genfeat));
+	memset(genfeat, 0, sizeof (*genfeat));
 	genfeat->panel_fitting = 3;
 	genfeat->flexaim = 1;
 	genfeat->download_ext_vbt = 1;



More information about the coreboot-gerrit mailing list