drivers: counter: Fix unbalanced policy state lock

ioannis-karachalios · github-actions[bot] · commit 4925ed5eeb6d · 2024-11-12T21:49:06.000Z
This commit should deal with fixing unbalanced policy state locks. When PM and PM_DEVICE are declared, default state, policy locks should be given only when active. Signed-off-by: Ioannis Karachalios <ioannis.karachalios.px@renesas.com> (cherry picked from commit a66c070)
diff --git a/drivers/counter/counter_smartbond_timer.c b/drivers/counter/counter_smartbond_timer.c
@@ -68,7 +68,9 @@ static void counter_smartbond_pm_policy_state_lock_get(const struct device *dev)
 static void counter_smartbond_pm_policy_state_lock_put(const struct device *dev)
 {
 	pm_device_runtime_put(dev);
-	pm_policy_state_lock_put(PM_STATE_STANDBY, PM_ALL_SUBSTATES);
+	if (pm_policy_state_lock_is_active(PM_STATE_STANDBY, PM_ALL_SUBSTATES)) {
+		pm_policy_state_lock_put(PM_STATE_STANDBY, PM_ALL_SUBSTATES);
+	}
 }
 
 /*

Original file line number	Diff line number	Diff line change
`@@ -68,7 +68,9 @@ static void counter_smartbond_pm_policy_state_lock_get(const struct device *dev)`
`68`	`68`	`static void counter_smartbond_pm_policy_state_lock_put(const struct device *dev)`
`69`	`69`	`{`
`70`	`70`	`pm_device_runtime_put(dev);`
`71`		`- pm_policy_state_lock_put(PM_STATE_STANDBY, PM_ALL_SUBSTATES);`
	`71`	`+ if (pm_policy_state_lock_is_active(PM_STATE_STANDBY, PM_ALL_SUBSTATES)) {`
	`72`	`+ pm_policy_state_lock_put(PM_STATE_STANDBY, PM_ALL_SUBSTATES);`
	`73`	`+ }`
`72`	`74`	`}`
`73`	`75`
`74`	`76`	`/*`