platform/x86: remove unneeded break
authorTom Rix <trix@redhat.com>
Mon, 19 Oct 2020 13:32:12 +0000 (06:32 -0700)
committerHans de Goede <hdegoede@redhat.com>
Wed, 28 Oct 2020 11:32:06 +0000 (12:32 +0100)
A break is not needed if it is preceded by a return

Signed-off-by: Tom Rix <trix@redhat.com>
Link: https://lore.kernel.org/r/20201019133212.12671-1-trix@redhat.com
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
drivers/platform/x86/acer-wmi.c
drivers/platform/x86/sony-laptop.c
drivers/platform/x86/wmi.c

index 49f4b73be513f515af3838e4af451c2071e945a7..1c2084c74a5704a961dee397dba40e486f44d281 100644 (file)
@@ -792,7 +792,6 @@ static acpi_status AMW0_set_u32(u32 value, u32 cap)
                switch (quirks->brightness) {
                default:
                        return ec_write(0x83, value);
-                       break;
                }
        default:
                return AE_ERROR;
index e5a1b553340813dd140459084b4d80089dfd301e..7048133749221e079e69a72244acbc9c34f222b0 100644 (file)
@@ -2467,13 +2467,11 @@ static int __sony_nc_gfx_switch_status_get(void)
                 * 0: integrated GFX (stamina)
                 */
                return result & 0x1 ? SPEED : STAMINA;
-               break;
        case 0x015B:
                /* 0: discrete GFX (speed)
                 * 1: integrated GFX (stamina)
                 */
                return result & 0x1 ? STAMINA : SPEED;
-               break;
        case 0x0128:
                /* it's a more elaborated bitmask, for now:
                 * 2: integrated GFX (stamina)
@@ -2482,7 +2480,6 @@ static int __sony_nc_gfx_switch_status_get(void)
                dprintk("GFX Status: 0x%x\n", result);
                return result & 0x80 ? AUTO :
                        result & 0x02 ? STAMINA : SPEED;
-               break;
        }
        return -EINVAL;
 }
index d5e84946a1daa99cf896f067dff57ab06d55b5f5..c669676ea8e8a6448e089161bd631f9e1b753e4c 100644 (file)
@@ -1260,13 +1260,10 @@ acpi_wmi_ec_space_handler(u32 function, acpi_physical_address address,
        switch (result) {
        case -EINVAL:
                return AE_BAD_PARAMETER;
-               break;
        case -ENODEV:
                return AE_NOT_FOUND;
-               break;
        case -ETIME:
                return AE_TIME;
-               break;
        default:
                return AE_OK;
        }