From beca41dfb55f0b83612fe8a2aec508944c4b2138 Mon Sep 17 00:00:00 2001 From: Olivier Certner Date: Mon, 4 May 2026 13:56:52 +0200 Subject: [PATCH] acpi_spmc(4): Remove XXX from a comment about retrieving constraints There is really nothing we can do about a DSM function that works on first call but fails on subsequent calls, except calling it only once, which we already do. While here, soften the comment message, as failure was observed with some specific machines only. No functional change. Reviewed by: emaste, obiwac Sponsored by: The FreeBSD Foundation Differential Revision: https://reviews.freebsd.org/D56799 --- sys/dev/acpica/acpi_spmc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sys/dev/acpica/acpi_spmc.c b/sys/dev/acpica/acpi_spmc.c index d8fa5617c8b..36a2c20e6be 100644 --- a/sys/dev/acpica/acpi_spmc.c +++ b/sys/dev/acpica/acpi_spmc.c @@ -451,7 +451,7 @@ acpi_spmc_get_constraints(device_t dev) dsm_index.regular = DSM_GET_DEVICE_CONSTRAINTS; } - /* XXX It seems like this DSM fails if called more than once. */ + /* It seems like this DSM can fail if called more than once. */ status = acpi_EvaluateDSMTyped(sc->handle, (uint8_t *)&dsm_set->uuid, dsm_set->revision, dsm_index.i, NULL, &result, ACPI_TYPE_PACKAGE);