Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37991 )
Change subject: Doc/acpi: Add rfkill device specification ......................................................................
Patch Set 4:
(1 comment)
https://review.coreboot.org/c/coreboot/+/37991/4/Documentation/acpi/rfkill.m... File Documentation/acpi/rfkill.md:
https://review.coreboot.org/c/coreboot/+/37991/4/Documentation/acpi/rfkill.m... PS4, Line 41: If a bit is not set, then that type of device is not supported on the : platform and the meaning of that bit in the return value or argument of any : other method is undefined. Nit: This language doesn't jive with the below paragraph:
"If the platform provides more than one radio of a given type, it should implement an additional rfkill device. Each radio should only be represented in a single device."
There could be >1 rfkill devices covering different radios so it's not like a given radio isn't supported -- it might just not be covered by this specific device.