Input: pm8941-pwrkey - fix comma vs semicolon issue

dtor · dtor · commit 5af9f79b41b2 · 2021-07-23T17:37:14.000-07:00
There is absolutely no reason to use comma operator in this code, 2 separate statements make much more sense. Reviewed-by: Stephen Boyd <swboyd@chromium.org> Link: https://lore.kernel.org/r/YPsa1qCBn/SAmE5x@google.com Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
diff --git a/drivers/input/misc/pm8941-pwrkey.c b/drivers/input/misc/pm8941-pwrkey.c
@@ -284,7 +284,7 @@ static int pm8941_pwrkey_probe(struct platform_device *pdev)
 	}
 
 	if (pwrkey->data->supports_ps_hold_poff_config) {
-		pwrkey->reboot_notifier.notifier_call = pm8941_reboot_notify,
+		pwrkey->reboot_notifier.notifier_call = pm8941_reboot_notify;
 		error = register_reboot_notifier(&pwrkey->reboot_notifier);
 		if (error) {
 			dev_err(&pdev->dev, "failed to register reboot notifier: %d\n",

Original file line number	Diff line number	Diff line change
`@@ -284,7 +284,7 @@ static int pm8941_pwrkey_probe(struct platform_device *pdev)`
`284`	`284`	`}`
`285`	`285`
`286`	`286`	`if (pwrkey->data->supports_ps_hold_poff_config) {`
`287`		`- pwrkey->reboot_notifier.notifier_call = pm8941_reboot_notify,`
	`287`	`+ pwrkey->reboot_notifier.notifier_call = pm8941_reboot_notify;`
`288`	`288`	`error = register_reboot_notifier(&pwrkey->reboot_notifier);`
`289`	`289`	`if (error) {`
`290`	`290`	`dev_err(&pdev->dev, "failed to register reboot notifier: %d\n",`