Messages in this thread | | | Date | Sun, 13 Oct 2013 04:29:34 -0500 | Subject | Re: [PATCH v2] platform: x86: asus-wmi: add fan control | From | Felipe Contreras <> |
| |
On Thu, Oct 10, 2013 at 11:06 AM, Matthew Garrett <matthew.garrett@nebula.com> wrote: > On Thu, 2013-10-10 at 16:59 +0100, Corentin Chary wrote: > >> >> This doesn't really seems to be related to wmi, and is likely to be >> available only on a subset of models. Maybe it should a separate >> driver instead ? > > This version seems to be implemented entirely in WMI, and it's using the > same WMI GUID as asus-wmi - implementing it here seems appropriate. I am > concerned about the phys/virt thing, though. The ACPI interpreter is > running in the kernel, not the hardware - are we artificially limiting > SystemMemory opregions to physical addresses?
I don't see anything in acpi_ex_system_memory_space_handler() that takes into consideration virtual addresses.
-- Felipe Contreras
| |