From 7a98a62f7b2c544b6f8d2ee4c8289175db5bc0c7 Mon Sep 17 00:00:00 2001 From: Sean Rhodes Date: Sun, 30 Nov 2025 20:48:31 +0000 Subject: [PATCH] drivers/intel/gma: Unify coding style The breaks for `if` and `else` are inconsistent; remove all breaks for these. Change-Id: Ie76f38387fd5ef330b432c0462cb1101571c73db Signed-off-by: Sean Rhodes Reviewed-on: https://review.coreboot.org/c/coreboot/+/90286 Reviewed-by: Angel Pons Tested-by: build bot (Jenkins) Reviewed-by: Matt DeVillier --- .../gma/acpi/configure_brightness_levels.asl | 15 +++++---------- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/src/drivers/intel/gma/acpi/configure_brightness_levels.asl b/src/drivers/intel/gma/acpi/configure_brightness_levels.asl index 11a7dd7725..4400f5159c 100644 --- a/src/drivers/intel/gma/acpi/configure_brightness_levels.asl +++ b/src/drivers/intel/gma/acpi/configure_brightness_levels.asl @@ -66,8 +66,7 @@ /* also set valid bit */ BCLP = Local1 | 0x80000000 - If (ARDY == 0) - { + If (ARDY == 0) { Return (Ones) } @@ -77,8 +76,7 @@ ASLE = 0x1 Local0 = 0x20 - While (Local0 > 0) - { + While (Local0 > 0) { Sleep (1) If ((ASLC & 2) == 0) { /* Request has been processed, check status: */ @@ -118,8 +116,7 @@ Method (XBQC, 0, NotSerialized) { /* Prevent DivideByZero if backlight control isn't enabled */ - If (BCLM == 0) - { + If (BCLM == 0) { Return (0) } /* Local0: current percentage */ @@ -152,14 +149,12 @@ { BRLV = Arg0 BRVA = 1 - If (^BOX3.XBCM (Arg0) == Ones) - { + If (^BOX3.XBCM (Arg0) == Ones) { /* * Only touch the legacy PWM registers after the graphics driver * reprograms them during boot/resume (BCLM stays zero until then). */ - If (BCLM != 0) - { + If (BCLM != 0) { ^LEGA.XBCM (Arg0) } }