Page MenuHomeFreeBSD

acpi_spmc(4): Rename functions parsing constraints
ClosedPublic

Authored by olce on May 4 2026, 8:32 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Jun 4, 8:47 PM
Unknown Object (File)
Thu, Jun 4, 6:52 PM
Unknown Object (File)
Wed, Jun 3, 7:19 PM
Unknown Object (File)
Sun, May 31, 7:17 AM
Unknown Object (File)
Thu, May 28, 5:28 PM
Unknown Object (File)
Thu, May 28, 5:28 PM
Unknown Object (File)
Tue, May 26, 10:46 PM
Unknown Object (File)
Tue, May 26, 1:18 PM
Subscribers

Details

Summary

Rename acpi_spmc_get_constraints_intel() and
acpi_spmc_get_constraints_amd() ("get" => "parse") to reflect that they
are actually just parsing the constraints passed by
acpi_spmc_get_constraints().

No functional change (intended).

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

olce requested review of this revision.May 4 2026, 8:32 PM
This revision is now accepted and ready to land.May 7 2026, 2:35 AM

(Context impact, nothing really changed.)

This revision now requires review to proceed.May 7 2026, 7:57 PM

I mean, technically these functions aren't just parsing the constraints, they're also getting them... I don't know if I agree with this change

I mean, technically these functions aren't just parsing the constraints, they're also getting them... I don't know if I agree with this change

Maybe it's a question of words, but these functions do not "get" them in the sense they query the firmware, that is done only at start of acpi_spmc_get_constraints(), they just interpret/parse the result, building our own internal structures.

I mean, technically these functions aren't just parsing the constraints, they're also getting them... I don't know if I agree with this change

Maybe it's a question of words, but these functions do not "get" them in the sense they query the firmware, that is done only at start of acpi_spmc_get_constraints(), they just interpret/parse the result, building our own internal structures.

Ah no you are totally right. We are passing the object. I have no idea why I thought this function was the one getting the object too.

This revision is now accepted and ready to land.May 11 2026, 11:27 AM

(Context impacts, no real changes.)

This revision now requires review to proceed.May 12 2026, 1:29 PM
This revision is now accepted and ready to land.May 12 2026, 2:05 PM