Messages in this thread | | | From | Matthew Garrett <> | Subject | Re: [PATCH v2] platform: x86: asus-wmi: add fan control | Date | Thu, 10 Oct 2013 16:06:40 +0000 |
| |
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?
-- Matthew Garrett <matthew.garrett@nebula.com>
| |