RE: [PATCH] cpufreq: governor: Change confusing struct field and variable names

2016-04-28 Thread Chen, Yu C
> -Original Message- > From: Rafael J. Wysocki [mailto:r...@rjwysocki.net] > Sent: Thursday, April 28, 2016 7:19 AM > To: Linux PM list > Cc: Linux Kernel Mailing List; Viresh Kumar; Chen, Yu C > Subject: [PATCH] cpufreq: governor: Change confusing struct field an

RE: [PATCH] cpufreq: governor: Change confusing struct field and variable names

2016-04-28 Thread Chen, Yu C
> -Original Message- > From: Rafael J. Wysocki [mailto:r...@rjwysocki.net] > Sent: Thursday, April 28, 2016 7:19 AM > To: Linux PM list > Cc: Linux Kernel Mailing List; Viresh Kumar; Chen, Yu C > Subject: [PATCH] cpufreq: governor: Change confusing struct field an

Re: [PATCH] cpufreq: governor: Change confusing struct field and variable names

2016-04-27 Thread Viresh Kumar
On 28-04-16, 01:19, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > The name of the prev_cpu_wall field in struct cpu_dbs_info is > confusing, because it doesn't represent wall time, but the previous > update time as returned by get_cpu_idle_time() (that may be

Re: [PATCH] cpufreq: governor: Change confusing struct field and variable names

2016-04-27 Thread Viresh Kumar
On 28-04-16, 01:19, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > The name of the prev_cpu_wall field in struct cpu_dbs_info is > confusing, because it doesn't represent wall time, but the previous > update time as returned by get_cpu_idle_time() (that may be the > current value of

[PATCH] cpufreq: governor: Change confusing struct field and variable names

2016-04-27 Thread Rafael J. Wysocki
From: Rafael J. Wysocki The name of the prev_cpu_wall field in struct cpu_dbs_info is confusing, because it doesn't represent wall time, but the previous update time as returned by get_cpu_idle_time() (that may be the current value of jiffies_64 in some cases, for

[PATCH] cpufreq: governor: Change confusing struct field and variable names

2016-04-27 Thread Rafael J. Wysocki
From: Rafael J. Wysocki The name of the prev_cpu_wall field in struct cpu_dbs_info is confusing, because it doesn't represent wall time, but the previous update time as returned by get_cpu_idle_time() (that may be the current value of jiffies_64 in some cases, for example). Moreover, the names