From: kettenis <kettenis@openbsd.org>
Date: Sun, 14 Jul 2024 10:48:55 +0000 (+0000)
Subject: Upon first suspend, turn off all power resources that haven't been
X-Git-Url: http://artulab.com/gitweb/?a=commitdiff_plain;h=bfa04e3703e141960b303fcf48b6b5b4430e67b1;p=openbsd

Upon first suspend, turn off all power resources that haven't been
referenced.

ok deraadt@
---

diff --git a/sys/dev/acpi/acpipwrres.c b/sys/dev/acpi/acpipwrres.c
index 27d7965df76..05e1a09d14b 100644
--- a/sys/dev/acpi/acpipwrres.c
+++ b/sys/dev/acpi/acpipwrres.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: acpipwrres.c,v 1.13 2023/02/18 14:32:02 dv Exp $ */
+/* $OpenBSD: acpipwrres.c,v 1.14 2024/07/14 10:48:55 kettenis Exp $ */
 
 /*
  * Copyright (c) 2013 Martin Pieuchot <mpi@openbsd.org>
@@ -33,6 +33,7 @@
 
 int	acpipwrres_match(struct device *, void *, void *);
 void	acpipwrres_attach(struct device *, struct device *, void *);
+int	acpipwrres_activate(struct device *, int);
 
 #ifdef ACPIPWRRES_DEBUG
 #define DPRINTF(x)	printf x
@@ -66,7 +67,8 @@ struct acpipwrres_consumer {
 };
 
 const struct cfattach acpipwrres_ca = {
-	sizeof(struct acpipwrres_softc), acpipwrres_match, acpipwrres_attach
+	sizeof(struct acpipwrres_softc), acpipwrres_match, acpipwrres_attach,
+	NULL, acpipwrres_activate
 };
 
 struct cfdriver acpipwrres_cd = {
@@ -140,6 +142,23 @@ acpipwrres_attach(struct device *parent, struct device *self, void *aux)
 	printf("\n");
 }
 
+int
+acpipwrres_activate(struct device *self, int act)
+{
+	struct acpipwrres_softc *sc = (struct acpipwrres_softc *)self;
+
+	switch (act) {
+	case DVACT_SUSPEND:
+		if (sc->sc_cons_ref == 0 && sc->sc_state != ACPIPWRRES_OFF) {
+			aml_evalname(sc->sc_acpi, sc->sc_devnode, "_OFF", 0,
+			    NULL, NULL);
+			sc->sc_state = ACPIPWRRES_OFF;
+		}
+		break;
+	}
+	return 0;
+}
+
 int
 acpipwrres_ref_incr(struct acpipwrres_softc *sc, struct aml_node *node)
 {
@@ -149,9 +168,11 @@ acpipwrres_ref_incr(struct acpipwrres_softc *sc, struct aml_node *node)
 	DPRINTF(("%s: dev %s ON %d\n", DEVNAME(sc), node->name,
 	    sc->sc_cons_ref));
 
-	if (sc->sc_cons_ref++ == 0)
+	if (sc->sc_cons_ref++ == 0) {
 		aml_evalname(sc->sc_acpi, sc->sc_devnode, "_ON", 0,
 		    NULL, NULL);
+		sc->sc_state = ACPIPWRRES_ON;
+	}
 
 	return (0);
 }
@@ -165,9 +186,11 @@ acpipwrres_ref_decr(struct acpipwrres_softc *sc, struct aml_node *node)
 	DPRINTF(("%s: dev %s OFF %d\n", DEVNAME(sc), node->name,
 	    sc->sc_cons_ref));
 
-	if (--sc->sc_cons_ref == 0)
+	if (--sc->sc_cons_ref == 0) {
 		aml_evalname(sc->sc_acpi, sc->sc_devnode, "_OFF", 0,
 		    NULL, NULL);
+		sc->sc_state = ACPIPWRRES_OFF;
+	}
 
 	return (0);
 }