Patrick Georgi submitted this change.

View Change

Approvals: build bot (Jenkins): Verified HAOUAS Elyes: Looks good to me, approved
drivers/i2c/at24rf08c: Format according to coding style

1. Move opening bracket to line above
2. Remove space after `printk` statements

Change-Id: Ia12a4ed6ab2fb2c9848a2688b41fcfa70ab001b0
Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/38392
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: HAOUAS Elyes <ehaouas@noos.fr>
---
M src/drivers/i2c/at24rf08c/at24rf08c.c
1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/drivers/i2c/at24rf08c/at24rf08c.c b/src/drivers/i2c/at24rf08c/at24rf08c.c
index daf04a9..11a6fd2 100644
--- a/src/drivers/i2c/at24rf08c/at24rf08c.c
+++ b/src/drivers/i2c/at24rf08c/at24rf08c.c
@@ -29,10 +29,9 @@
if (dev->path.type != DEVICE_PATH_I2C || dev->path.i2c.device != 0x5c)
return;

- printk (BIOS_DEBUG, "Locking EEPROM RFID\n");
+ printk(BIOS_DEBUG, "Locking EEPROM RFID\n");

- for (i = 0; i < 8; i++)
- {
+ for (i = 0; i < 8; i++) {
/* After a register write AT24RF08C sometimes stops responding.
Retry several times in case of failure. */
for (j = 0; j < 100; j++)
@@ -40,7 +39,7 @@
break;
}

- printk (BIOS_DEBUG, "init EEPROM done\n");
+ printk(BIOS_DEBUG, "init EEPROM done\n");
}

static struct device_operations at24rf08c_operations = {

To view, visit change 38392. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ia12a4ed6ab2fb2c9848a2688b41fcfa70ab001b0
Gerrit-Change-Number: 38392
Gerrit-PatchSet: 3
Gerrit-Owner: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Reviewer: Arthur Heymans <arthur@aheymans.xyz>
Gerrit-Reviewer: HAOUAS Elyes <ehaouas@noos.fr>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-MessageType: merged