From 1f4e78f651375aec51bcd04f42bfb2db7c13c084 Mon Sep 17 00:00:00 2001 From: Angel Pons Date: Thu, 26 Aug 2021 09:53:14 +0200 Subject: [PATCH] mb/google/butterfly: Put braces around `else` branch Ensure braces are consistent on all branches of a conditional statement, as per the coding style. Tested with BUILD_TIMELESS=1, Google Butterfly remains identical. Change-Id: I34f3b22486e0f0712bc248477acb43012b21c5ee Signed-off-by: Angel Pons Reviewed-on: https://review.coreboot.org/c/coreboot/+/57163 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel Reviewed-by: Felix Held --- src/mainboard/google/butterfly/mainboard.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/mainboard/google/butterfly/mainboard.c b/src/mainboard/google/butterfly/mainboard.c index 6759b86a1d..6145dc73d4 100644 --- a/src/mainboard/google/butterfly/mainboard.c +++ b/src/mainboard/google/butterfly/mainboard.c @@ -150,8 +150,9 @@ static void program_keyboard_type(u32 search_address, u32 search_length) if (offset != search_length) kbd_type = EC_KBD_JP; } - } else + } else { printk(BIOS_DEBUG, "Error: Could not locate VPD area\n"); + } printk(BIOS_DEBUG, "Setting Keyboard type in EC to "); printk(BIOS_DEBUG, (kbd_type == EC_KBD_JP) ? "Japanese" : "English");