diff options
| author | Olivier Certner <olce@FreeBSD.org> | 2026-05-04 19:42:40 +0000 |
|---|---|---|
| committer | Olivier Certner <olce@FreeBSD.org> | 2026-05-13 12:38:23 +0000 |
| commit | ee13d62417f28ee22331ab81ecac2ed3457ea1e1 (patch) | |
| tree | 45b2bb3175bcff85c1ca17dec8e8e436107463d4 | |
| parent | 0813dc0325476599841268f405d4bce8b341b2db (diff) | |
acpi_spmc(4): Constraint parsing: Clearer error messages
Reviewed by: imp, obiwac
Sponsored by: The FreeBSD Foundation
Differential Revision: https://reviews.freebsd.org/D56815
| -rw-r--r-- | sys/dev/acpica/acpi_spmc.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/sys/dev/acpica/acpi_spmc.c b/sys/dev/acpica/acpi_spmc.c index 95d802f7688d..5aed9ad3e5e4 100644 --- a/sys/dev/acpica/acpi_spmc.c +++ b/sys/dev/acpica/acpi_spmc.c @@ -516,8 +516,8 @@ acpi_spmc_parse_constraints_intel(struct acpi_spmc_softc *sc, ACPI_OBJECT *objec */ revision = detail->Package.Elements[0].Integer.Value; if (revision != 0) { - device_printf(sc->dev, "Unknown revision %d for " - "device constraint detail package\n", revision); + device_printf(sc->dev, "Intel: Unknown revision %d for " + "constraint %zu's detail package\n", revision, i); sc->constraint_count--; continue; } @@ -553,7 +553,8 @@ acpi_spmc_parse_constraints_amd(struct acpi_spmc_softc *sc, ACPI_OBJECT *object) constraints = &object->Package.Elements[2]; if (constraints->Package.Count != constraint_count) { - device_printf(sc->dev, "constraint count mismatch (%d to %zu)\n", + device_printf(sc->dev, + "AMD: Constraints: Count mismatch (%d to %zu)\n", constraints->Package.Count, constraint_count); return (ENXIO); } @@ -566,7 +567,8 @@ acpi_spmc_parse_constraints_amd(struct acpi_spmc_softc *sc, ACPI_OBJECT *object) /* Parse the constraint package. */ constraint_obj = &constraints->Package.Elements[i]; if (constraint_obj->Package.Count != 4) { - device_printf(sc->dev, "constraint %zu has %d elements\n", + device_printf(sc->dev, + "AMD: Constraint %zu has %d elements, not 4\n", i, constraint_obj->Package.Count); acpi_spmc_free_constraints(sc); return (ENXIO); |