[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [RFC PATCH 01/13] cpufreq: move cpufreq.h file to the xen/include/cpufreq location



>>> On 07.10.14 at 16:19, <oleksandr.dmytryshyn@xxxxxxxxxxxxxxx> wrote:
> Cpufreq driver should be more generalizable (not ACPI-specific).
> Thus this file should be placed to the proper location.

For an arbitrary definition of "proper", because ...

> Signed-off-by: Oleksandr Dmytryshyn <oleksandr.dmytryshyn@xxxxxxxxxxxxxxx>
> ---
>  xen/arch/x86/acpi/cpu_idle.c                 |   2 +-
>  xen/arch/x86/acpi/cpufreq/cpufreq.c          |   2 +-
>  xen/arch/x86/acpi/cpufreq/powernow.c         |   2 +-
>  xen/arch/x86/acpi/power.c                    |   2 +-
>  xen/arch/x86/cpu/mwait-idle.c                |   2 +-
>  xen/drivers/acpi/pmstat.c                    |   2 +-
>  xen/drivers/cpufreq/cpufreq.c                |   2 +-
>  xen/drivers/cpufreq/cpufreq_misc_governors.c |   2 +-
>  xen/drivers/cpufreq/cpufreq_ondemand.c       |   4 +-
>  xen/drivers/cpufreq/utility.c                |   2 +-
>  xen/include/acpi/cpufreq/cpufreq.h           | 258 --------------------------
>  xen/include/cpufreq/cpufreq.h                | 261 
> +++++++++++++++++++++++++++

I have a hard time seeing why we'd need a cpufreq/ subdir here.
Just put this in include/xen/.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.