Commit 61982193 authored by Tom Rix's avatar Tom Rix Committed by Hans de Goede

platform/x86: remove unneeded break

A break is not needed if it is preceded by a return
Signed-off-by: default avatarTom Rix <trix@redhat.com>
Link: https://lore.kernel.org/r/20201019133212.12671-1-trix@redhat.comSigned-off-by: default avatarHans de Goede <hdegoede@redhat.com>
parent dac76c17
...@@ -792,7 +792,6 @@ static acpi_status AMW0_set_u32(u32 value, u32 cap) ...@@ -792,7 +792,6 @@ static acpi_status AMW0_set_u32(u32 value, u32 cap)
switch (quirks->brightness) { switch (quirks->brightness) {
default: default:
return ec_write(0x83, value); return ec_write(0x83, value);
break;
} }
default: default:
return AE_ERROR; return AE_ERROR;
......
...@@ -2467,13 +2467,11 @@ static int __sony_nc_gfx_switch_status_get(void) ...@@ -2467,13 +2467,11 @@ static int __sony_nc_gfx_switch_status_get(void)
* 0: integrated GFX (stamina) * 0: integrated GFX (stamina)
*/ */
return result & 0x1 ? SPEED : STAMINA; return result & 0x1 ? SPEED : STAMINA;
break;
case 0x015B: case 0x015B:
/* 0: discrete GFX (speed) /* 0: discrete GFX (speed)
* 1: integrated GFX (stamina) * 1: integrated GFX (stamina)
*/ */
return result & 0x1 ? STAMINA : SPEED; return result & 0x1 ? STAMINA : SPEED;
break;
case 0x0128: case 0x0128:
/* it's a more elaborated bitmask, for now: /* it's a more elaborated bitmask, for now:
* 2: integrated GFX (stamina) * 2: integrated GFX (stamina)
...@@ -2482,7 +2480,6 @@ static int __sony_nc_gfx_switch_status_get(void) ...@@ -2482,7 +2480,6 @@ static int __sony_nc_gfx_switch_status_get(void)
dprintk("GFX Status: 0x%x\n", result); dprintk("GFX Status: 0x%x\n", result);
return result & 0x80 ? AUTO : return result & 0x80 ? AUTO :
result & 0x02 ? STAMINA : SPEED; result & 0x02 ? STAMINA : SPEED;
break;
} }
return -EINVAL; return -EINVAL;
} }
......
...@@ -1260,13 +1260,10 @@ acpi_wmi_ec_space_handler(u32 function, acpi_physical_address address, ...@@ -1260,13 +1260,10 @@ acpi_wmi_ec_space_handler(u32 function, acpi_physical_address address,
switch (result) { switch (result) {
case -EINVAL: case -EINVAL:
return AE_BAD_PARAMETER; return AE_BAD_PARAMETER;
break;
case -ENODEV: case -ENODEV:
return AE_NOT_FOUND; return AE_NOT_FOUND;
break;
case -ETIME: case -ETIME:
return AE_TIME; return AE_TIME;
break;
default: default:
return AE_OK; return AE_OK;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment