Skip to content

Commit b697b81

Browse files
committed
ACPI: processor: idle: Drop unnecessary statements and parens
Drop a redundant "else", a "return" statement at the end of a void function and redundant parentheses around an unsigent int variable name from the ACPI processor idle driver. No expected functional impact. Signed-off-by: Rafael J. Wysocki <[email protected]>
1 parent 05e6b43 commit b697b81

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

drivers/acpi/processor_idle.c

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -324,7 +324,7 @@ static void acpi_processor_power_verify_c3(struct acpi_processor *pr,
324324
* the erratum), but this is known to disrupt certain ISA
325325
* devices thus we take the conservative approach.
326326
*/
327-
else if (errata.piix4.fdma) {
327+
if (errata.piix4.fdma) {
328328
acpi_handle_debug(pr->handle,
329329
"C3 not supported on PIIX4 with Type-F DMA\n");
330330
return;
@@ -384,8 +384,6 @@ static void acpi_processor_power_verify_c3(struct acpi_processor *pr,
384384
* handle BM_RLD is to set it and leave it set.
385385
*/
386386
acpi_write_bit_register(ACPI_BITREG_BUS_MASTER_RLD, 1);
387-
388-
return;
389387
}
390388

391389
static int acpi_cst_latency_cmp(const void *a, const void *b)
@@ -459,7 +457,7 @@ static int acpi_processor_power_verify(struct acpi_processor *pr)
459457

460458
lapic_timer_propagate_broadcast(pr);
461459

462-
return (working);
460+
return working;
463461
}
464462

465463
static int acpi_processor_get_cstate_info(struct acpi_processor *pr)

0 commit comments

Comments
 (0)