<feed xmlns='http://www.w3.org/2005/Atom'>
<title>src/sys/dev/acpica, branch main</title>
<subtitle>FreeBSD source tree</subtitle>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/'/>
<entry>
<title>acpi_spmc(4): Remove redundant setting of 'sc-&gt;dev' on attach</title>
<updated>2026-04-17T21:04:50+00:00</updated>
<author>
<name>Olivier Certner</name>
<email>olce@FreeBSD.org</email>
</author>
<published>2026-04-17T21:02:06+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=0a764beb6215f067787fba1723f211b7ef0eeeb0'/>
<id>0a764beb6215f067787fba1723f211b7ef0eeeb0</id>
<content type='text'>
Should have been part of the previous commit (but PEBCAK).

Reviewed by:    obiwac
Fixes:          bd05b47fbd8b ("acpi_spmc(4): Small probe improvements/fixes")
Sponsored by:   The FreeBSD Foundation
Differential Revision:  https://reviews.freebsd.org/D56483
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Should have been part of the previous commit (but PEBCAK).

Reviewed by:    obiwac
Fixes:          bd05b47fbd8b ("acpi_spmc(4): Small probe improvements/fixes")
Sponsored by:   The FreeBSD Foundation
Differential Revision:  https://reviews.freebsd.org/D56483
</pre>
</div>
</content>
</entry>
<entry>
<title>acpi_spmc(4): Small probe improvements/fixes</title>
<updated>2026-04-17T20:27:51+00:00</updated>
<author>
<name>Olivier Certner</name>
<email>olce@FreeBSD.org</email>
</author>
<published>2026-04-10T14:15:09+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=bd05b47fbd8b7301983dc4ceaf0498fb8d5ca9f2'/>
<id>bd05b47fbd8b7301983dc4ceaf0498fb8d5ca9f2</id>
<content type='text'>
Remove the test on presence of an ACPI handle, this is implied by
ACPI_ID_PROBE() succeeding.

Set 'sc-&gt;dev' early, so that acpi_spmc_check_dsm_set() using
device_printf() will print the driver name.

Add a missing newline after printing that more DSM functions are
implemented then expected.

Reviewed by:    obiwac
Sponsored by:   The FreeBSD Foundation
Differential Revision:  https://reviews.freebsd.org/D56483
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Remove the test on presence of an ACPI handle, this is implied by
ACPI_ID_PROBE() succeeding.

Set 'sc-&gt;dev' early, so that acpi_spmc_check_dsm_set() using
device_printf() will print the driver name.

Add a missing newline after printing that more DSM functions are
implemented then expected.

Reviewed by:    obiwac
Sponsored by:   The FreeBSD Foundation
Differential Revision:  https://reviews.freebsd.org/D56483
</pre>
</div>
</content>
</entry>
<entry>
<title>acpi: Return "unknown D-state" in acpi_d_state_to_str() if unknown</title>
<updated>2026-04-16T07:17:07+00:00</updated>
<author>
<name>Aymeric Wibo</name>
<email>obiwac@FreeBSD.org</email>
</author>
<published>2026-04-16T07:12:33+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=dc9ff9e5d078fd923adc3dc5426b5f219156ea43'/>
<id>dc9ff9e5d078fd923adc3dc5426b5f219156ea43</id>
<content type='text'>
Some ACPI debugging prints call acpi_d_state_to_str() on unset D-states
(i.e. ACPI_STATE_UNKNOWN), so return a string explicitly saying "unknown
D-state" instead of just panicking.

Fixes:		84bbfc32a3f4 ("acpi_powerres: D3cold support")
Sponsored by:	The FreeBSD Foundation
MFC after:	3 days
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Some ACPI debugging prints call acpi_d_state_to_str() on unset D-states
(i.e. ACPI_STATE_UNKNOWN), so return a string explicitly saying "unknown
D-state" instead of just panicking.

Fixes:		84bbfc32a3f4 ("acpi_powerres: D3cold support")
Sponsored by:	The FreeBSD Foundation
MFC after:	3 days
</pre>
</div>
</content>
</entry>
<entry>
<title>acpi: Parse _S3D in s2idle instead of _S255D</title>
<updated>2026-04-08T12:11:23+00:00</updated>
<author>
<name>Aymeric Wibo</name>
<email>obiwac@FreeBSD.org</email>
</author>
<published>2026-03-04T13:09:14+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=19ab46cc66045ea936475f1a1e08763eb2fd663e'/>
<id>19ab46cc66045ea936475f1a1e08763eb2fd663e</id>
<content type='text'>
Previously, when entering s2idle, we were parsing the _S255D object to
get the shallowest D-state supported by device, as
acpi_stype_to_sstate() returns -1 for s2idle.

Instead, we should read _S3D.

Relevant document:
https://uefi.org/htmlspecs/ACPI_Spec_6_4_html/07_Power_and_Performance_Mgmt/device-power-management-objects.html#s3d-s3-device-state

Reviewed by:	olce
Tested by:	emaste, olce
Approved by:	olce
Sponsored by:	The FreeBSD Foundation
Differential Revision:	https://reviews.freebsd.org/D55118
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Previously, when entering s2idle, we were parsing the _S255D object to
get the shallowest D-state supported by device, as
acpi_stype_to_sstate() returns -1 for s2idle.

Instead, we should read _S3D.

Relevant document:
https://uefi.org/htmlspecs/ACPI_Spec_6_4_html/07_Power_and_Performance_Mgmt/device-power-management-objects.html#s3d-s3-device-state

Reviewed by:	olce
Tested by:	emaste, olce
Approved by:	olce
Sponsored by:	The FreeBSD Foundation
Differential Revision:	https://reviews.freebsd.org/D55118
</pre>
</div>
</content>
</entry>
<entry>
<title>acpi_spmc: Call new MS turn on display DSM</title>
<updated>2026-04-01T11:56:04+00:00</updated>
<author>
<name>Aymeric Wibo</name>
<email>obiwac@FreeBSD.org</email>
</author>
<published>2026-04-01T11:56:03+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=683f7a8374ff67417ac45949bd56a12c23781232'/>
<id>683f7a8374ff67417ac45949bd56a12c23781232</id>
<content type='text'>
Microsoft added a new function index (turn on display, 9) to their DSM
set. This revision calls this, which fixes S0ix on certain machines,
such as the Lenovo Yoga Slim 7i Aura, who's ECs use this method as a
trigger to restore power to certain devices.

See commit 229ecbaac6b3 ("ACPI: x86: s2idle: Invoke Microsoft _DSM
Function 9 (Turn On Display)") on Linux.

Also see the following:
https://learn.microsoft.com/en-us/windows-hardware/design/device-experiences/modern-standby-firmware-notifications#turn-on-display-notification-function-9

Reviewed by:	olce
Approved by:	olce
Sponsored by:	The FreeBSD Foundation
Differential Revision:	https://reviews.freebsd.org/D56062
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Microsoft added a new function index (turn on display, 9) to their DSM
set. This revision calls this, which fixes S0ix on certain machines,
such as the Lenovo Yoga Slim 7i Aura, who's ECs use this method as a
trigger to restore power to certain devices.

See commit 229ecbaac6b3 ("ACPI: x86: s2idle: Invoke Microsoft _DSM
Function 9 (Turn On Display)") on Linux.

Also see the following:
https://learn.microsoft.com/en-us/windows-hardware/design/device-experiences/modern-standby-firmware-notifications#turn-on-display-notification-function-9

Reviewed by:	olce
Approved by:	olce
Sponsored by:	The FreeBSD Foundation
Differential Revision:	https://reviews.freebsd.org/D56062
</pre>
</div>
</content>
</entry>
<entry>
<title>acpi_spmc: Fail probe if acpi_spmc device already attached</title>
<updated>2026-04-01T11:54:58+00:00</updated>
<author>
<name>Aymeric Wibo</name>
<email>obiwac@FreeBSD.org</email>
</author>
<published>2026-04-01T11:54:57+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=0307db7744d4cfe0d610ebece226dd20fdcc4ed5'/>
<id>0307db7744d4cfe0d610ebece226dd20fdcc4ed5</id>
<content type='text'>
We cannot have more than one SPMC device.

Reviewed by:	olce
Approved by:	olce
Sponsored by:	The FreeBSD Foundation
Differential Revision:	https://reviews.freebsd.org/D56062
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
We cannot have more than one SPMC device.

Reviewed by:	olce
Approved by:	olce
Sponsored by:	The FreeBSD Foundation
Differential Revision:	https://reviews.freebsd.org/D56062
</pre>
</div>
</content>
</entry>
<entry>
<title>acpi_spmc: Improve device description</title>
<updated>2026-03-31T10:01:10+00:00</updated>
<author>
<name>Aymeric Wibo</name>
<email>obiwac@FreeBSD.org</email>
</author>
<published>2026-03-31T10:01:08+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=33e095ee097d6a18dca95b8fae4edbda405bbb20'/>
<id>33e095ee097d6a18dca95b8fae4edbda405bbb20</id>
<content type='text'>
"Low Power S0 Idle" doesn't mean that much. "System Power Management
Controller" is what "SPMC" stands for in the first place.

Sponsored by:	The FreeBSD Foundation
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
"Low Power S0 Idle" doesn't mean that much. "System Power Management
Controller" is what "SPMC" stands for in the first place.

Sponsored by:	The FreeBSD Foundation
</pre>
</div>
</content>
</entry>
<entry>
<title>acpi_spmc: Fix checking expected DSMs</title>
<updated>2026-03-24T08:00:58+00:00</updated>
<author>
<name>Aymeric Wibo</name>
<email>obiwac@FreeBSD.org</email>
</author>
<published>2026-03-24T07:25:50+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=d8a8c890a1a3a003dbd12ec0235304db7fbe9e6e'/>
<id>d8a8c890a1a3a003dbd12ec0235304db7fbe9e6e</id>
<content type='text'>
Previously was or'ing the indices of the DSMs directly, not their
corresponding bits. This commit rectifies this.

Reviewed by:	olce
Approved by:	olce
Fixes:	c5daa5a4c32c ("acpi_spmc: Add system power management controller driver")
Sponsored by:	The FreeBSD Foundation
Differential Revision:	https://reviews.freebsd.org/https://reviews.freebsd.org/D56062
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Previously was or'ing the indices of the DSMs directly, not their
corresponding bits. This commit rectifies this.

Reviewed by:	olce
Approved by:	olce
Fixes:	c5daa5a4c32c ("acpi_spmc: Add system power management controller driver")
Sponsored by:	The FreeBSD Foundation
Differential Revision:	https://reviews.freebsd.org/https://reviews.freebsd.org/D56062
</pre>
</div>
</content>
</entry>
<entry>
<title>acpi_spmc: Simplify constraint freeing</title>
<updated>2026-03-21T05:20:46+00:00</updated>
<author>
<name>Aymeric Wibo</name>
<email>obiwac@FreeBSD.org</email>
</author>
<published>2026-03-21T05:19:21+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=a87bde85232e60771691649fd5718a6de5453e59'/>
<id>a87bde85232e60771691649fd5718a6de5453e59</id>
<content type='text'>
We don't need to check for NULL before calling free().

Sponsored by:	The FreeBSD Foundation
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
We don't need to check for NULL before calling free().

Sponsored by:	The FreeBSD Foundation
</pre>
</div>
</content>
</entry>
<entry>
<title>acpi: Print sysctl name in deprecated sleep type warning</title>
<updated>2026-03-21T03:21:45+00:00</updated>
<author>
<name>Aymeric Wibo</name>
<email>obiwac@FreeBSD.org</email>
</author>
<published>2026-03-21T03:15:03+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=66447abae8bb5d2e7b09a72bdea95033d9b1867a'/>
<id>66447abae8bb5d2e7b09a72bdea95033d9b1867a</id>
<content type='text'>
Reported by:	markj
Fixes:		97d152698f48 ("acpi: Use sleep types defined in sys/power.h")
Event:		AsiaBSDCon 2026
Sponsored by:	The FreeBSD Foundation
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Reported by:	markj
Fixes:		97d152698f48 ("acpi: Use sleep types defined in sys/power.h")
Event:		AsiaBSDCon 2026
Sponsored by:	The FreeBSD Foundation
</pre>
</div>
</content>
</entry>
</feed>
