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

[Oleksandr Dmytryshyn <oleksandr.dmytryshyn@xxxxxxxxxxxxxxx>]

On Tue, Oct 7, 2014 at 5:35 PM, Jan Beulich <JBeulich@xxxxxxxx> wrote:
>>>> 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/.
I'll do it in the next patch-set.

> Jan
>

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