interrupt_controller: ioapic: support more device power states

do nothing when enter or resume from DEVICE_PM_LOW_POWER_STATE

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

Author: kwd-doodling

drivers/interrupt_controller/intc_ioapic.c

@@ -255,7 +255,6 @@ int ioapic_suspend(const struct device *port)
 			store_flags(irq, rte_lo);
 		}
 	}
-	ioapic_device_power_state = DEVICE_PM_SUSPEND_STATE;
 	return 0;
 }
 
@@ -285,7 +284,6 @@ int ioapic_resume_from_suspend(const struct device *port)
 		ioApicRedSetHi(irq, DEFAULT_RTE_DEST);
 		ioApicRedSetLo(irq, rteValue);
 	}
-	ioapic_device_power_state = DEVICE_PM_ACTIVE_STATE;
 	return 0;
 }
 
@@ -300,10 +298,28 @@ static int ioapic_device_ctrl(const struct device *device,
 	int ret = 0;
 
 	if (ctrl_command == DEVICE_PM_SET_POWER_STATE) {
-		if (*((uint32_t *)context) == DEVICE_PM_SUSPEND_STATE) {
+		uint32_t new_state = *((uint32_t *)context);
+
+		switch (new_state) {
+		case DEVICE_PM_LOW_POWER_STATE:
+			break;
+		case DEVICE_PM_ACTIVE_STATE:
+			if (ioapic_device_power_state !=
+					DEVICE_PM_LOW_POWER_STATE) {
+				ret = ioapic_resume_from_suspend(device);
+			}
+			break;
+		case DEVICE_PM_SUSPEND_STATE:
+		case DEVICE_PM_FORCE_SUSPEND_STATE:
+		case DEVICE_PM_OFF_STATE:
 			ret = ioapic_suspend(device);
-		} else if (*((uint32_t *)context) == DEVICE_PM_ACTIVE_STATE) {
-			ret = ioapic_resume_from_suspend(device);
+			break;
+		default:
+			ret = -ENOTSUP;
+		}
+
+		if (ret == 0) {
+			ioapic_device_power_state = new_state;
 		}
 	} else if (ctrl_command == DEVICE_PM_GET_POWER_STATE) {
 		*((uint32_t *)context) = ioapic_device_power_state;