drivers: counter: Fix unbalanced policy state lock

ioannis-karachalios · ioannis-karachalios · commit 39c8274ba2c8 · 2024-11-10T22:37:03.000+02:00
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 &lt;ioannis.karachalios.px@renesas.com&gt;
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`	`/*`