pm: device: Change atomic flags type

Just using a simple atomic for flags instead of using an array.

While is neat using ATOMIC_DEFINE for future proof. The reality is
that it brings some problem for the wakeup source implementation
that needs to statically initialize it during the device definition.

Signed-off-by: Flavio Ceolin <[email protected]>

Author: Flavio Ceolin

include/pm/device.h

@@ -96,7 +96,7 @@ struct pm_device {
 	/** Device pm enable flag */
 	bool enable : 1;
 	/* Device PM status flags. */
-	ATOMIC_DEFINE(flags, PM_DEVICE_FLAG_COUNT);
+	atomic_t flags;
 	/** Device usage count */
 	uint32_t usage;
 	/** Device power state */

subsys/pm/device.c

@@ -184,7 +184,7 @@ bool pm_device_is_any_busy(void)
 	devc = z_device_get_all_static(&devs);
 
 	for (const struct device *dev = devs; dev < (devs + devc); dev++) {
-		if (atomic_test_bit(dev->pm->flags, PM_DEVICE_FLAG_BUSY)) {
+		if (atomic_test_bit(&dev->pm->flags, PM_DEVICE_FLAG_BUSY)) {
 			return true;
 		}
 	}
@@ -194,15 +194,15 @@ bool pm_device_is_any_busy(void)
 
 bool pm_device_is_busy(const struct device *dev)
 {
-	return atomic_test_bit(dev->pm->flags, PM_DEVICE_FLAG_BUSY);
+	return atomic_test_bit(&dev->pm->flags, PM_DEVICE_FLAG_BUSY);
 }
 
 void pm_device_busy_set(const struct device *dev)
 {
-	atomic_set_bit(dev->pm->flags, PM_DEVICE_FLAG_BUSY);
+	atomic_set_bit(&dev->pm->flags, PM_DEVICE_FLAG_BUSY);
 }
 
 void pm_device_busy_clear(const struct device *dev)
 {
-	atomic_clear_bit(dev->pm->flags, PM_DEVICE_FLAG_BUSY);
+	atomic_clear_bit(&dev->pm->flags, PM_DEVICE_FLAG_BUSY);
 }