PM: wakeup: Remove unnecessary else from device_init_wakeup()

DhruvaG2000 · rafaeljw · commit 3642c7ed5231 · 2024-04-03T16:51:37.000+02:00
Checkpatch warns that else is generally not necessary after a return
condition which exists in the if part of this function.

Hence, just to abide by what checkpatch recommends, follow it's
guidelines.

Signed-off-by: Dhruva Gole &lt;d-gole@ti.com&gt;
Signed-off-by: Rafael J. Wysocki &lt;rafael.j.wysocki@intel.com&gt;
diff --git a/include/linux/pm_wakeup.h b/include/linux/pm_wakeup.h
@@ -234,11 +234,10 @@ static inline int device_init_wakeup(struct device *dev, bool enable)
 	if (enable) {
 		device_set_wakeup_capable(dev, true);
 		return device_wakeup_enable(dev);
-	} else {
-		device_wakeup_disable(dev);
-		device_set_wakeup_capable(dev, false);
-		return 0;
 	}
+	device_wakeup_disable(dev);
+	device_set_wakeup_capable(dev, false);
+	return 0;
 }
 
 #endif /* _LINUX_PM_WAKEUP_H */

Original file line number	Diff line number	Diff line change
`@@ -234,11 +234,10 @@ static inline int device_init_wakeup(struct device *dev, bool enable)`
`234`	`234`	`if (enable) {`
`235`	`235`	`device_set_wakeup_capable(dev, true);`
`236`	`236`	`return device_wakeup_enable(dev);`
`237`		`- } else {`
`238`		`- device_wakeup_disable(dev);`
`239`		`- device_set_wakeup_capable(dev, false);`
`240`		`- return 0;`
`241`	`237`	`}`
	`238`	`+ device_wakeup_disable(dev);`
	`239`	`+ device_set_wakeup_capable(dev, false);`
	`240`	`+ return 0;`
`242`	`241`	`}`
`243`	`242`
`244`	`243`	`#endif /* _LINUX_PM_WAKEUP_H */`