* [PATCH 1/5] acpi : remove latency_ticks from acpi_processor_cx structure
@ 2012-07-12 15:29 Daniel Lezcano
2012-07-12 15:30 ` [PATCH 2/5] acpi : remove index " Daniel Lezcano
` (4 more replies)
0 siblings, 5 replies; 10+ messages in thread
From: Daniel Lezcano @ 2012-07-12 15:29 UTC (permalink / raw)
To: rjw; +Cc: linux-pm
Remove the latency_ticks field as it is not used.
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
---
drivers/acpi/processor_idle.c | 2 --
include/acpi/processor.h | 1 -
2 files changed, 0 insertions(+), 3 deletions(-)
diff --git a/drivers/acpi/processor_idle.c b/drivers/acpi/processor_idle.c
index b894627..9ef007d 100644
--- a/drivers/acpi/processor_idle.c
+++ b/drivers/acpi/processor_idle.c
@@ -583,7 +583,6 @@ static void acpi_processor_power_verify_c3(struct acpi_processor *pr,
*/
cx->valid = 1;
- cx->latency_ticks = cx->latency;
/*
* On older chipsets, BM_RLD needs to be set
* in order for Bus Master activity to wake the
@@ -616,7 +615,6 @@ static int acpi_processor_power_verify(struct acpi_processor *pr)
if (!cx->address)
break;
cx->valid = 1;
- cx->latency_ticks = cx->latency; /* Normalize latency */
break;
case ACPI_STATE_C3:
diff --git a/include/acpi/processor.h b/include/acpi/processor.h
index ac3bff6..19423c3 100644
--- a/include/acpi/processor.h
+++ b/include/acpi/processor.h
@@ -59,7 +59,6 @@ struct acpi_processor_cx {
u8 entry_method;
u8 index;
u32 latency;
- u32 latency_ticks;
u32 power;
u32 usage;
u64 time;
--
1.7.5.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 2/5] acpi : remove index from acpi_processor_cx structure
2012-07-12 15:29 [PATCH 1/5] acpi : remove latency_ticks from acpi_processor_cx structure Daniel Lezcano
@ 2012-07-12 15:30 ` Daniel Lezcano
2012-07-12 15:30 ` [PATCH 3/5] acpi : remove usage " Daniel Lezcano
` (3 subsequent siblings)
4 siblings, 0 replies; 10+ messages in thread
From: Daniel Lezcano @ 2012-07-12 15:30 UTC (permalink / raw)
To: rjw; +Cc: linux-pm
Remove the index field. It could be given without adding extra
information in the cx structure.
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
---
arch/x86/kernel/acpi/cstate.c | 18 +++++++++---------
drivers/acpi/processor_idle.c | 15 +++++++--------
include/acpi/processor.h | 11 +++++------
3 files changed, 21 insertions(+), 23 deletions(-)
diff --git a/arch/x86/kernel/acpi/cstate.c b/arch/x86/kernel/acpi/cstate.c
index d2b7f27..faa7f50 100644
--- a/arch/x86/kernel/acpi/cstate.c
+++ b/arch/x86/kernel/acpi/cstate.c
@@ -112,8 +112,8 @@ out:
return retval;
}
-int acpi_processor_ffh_cstate_probe(unsigned int cpu,
- struct acpi_processor_cx *cx, struct acpi_power_register *reg)
+int acpi_processor_ffh_cstate_probe(unsigned int cpu, struct acpi_processor_cx *cx,
+ int index, struct acpi_power_register *reg)
{
struct cstate_entry *percpu_entry;
struct cpuinfo_x86 *c = &cpu_data(cpu);
@@ -126,16 +126,16 @@ int acpi_processor_ffh_cstate_probe(unsigned int cpu,
return -1;
percpu_entry = per_cpu_ptr(cpu_cstate_entry, cpu);
- percpu_entry->states[cx->index].eax = 0;
- percpu_entry->states[cx->index].ecx = 0;
+ percpu_entry->states[index].eax = 0;
+ percpu_entry->states[index].ecx = 0;
/* Make sure we are running on right CPU */
retval = work_on_cpu(cpu, acpi_processor_ffh_cstate_probe_cpu, cx);
if (retval == 0) {
/* Use the hint in CST */
- percpu_entry->states[cx->index].eax = cx->address;
- percpu_entry->states[cx->index].ecx = MWAIT_ECX_INTERRUPT_BREAK;
+ percpu_entry->states[index].eax = cx->address;
+ percpu_entry->states[index].ecx = MWAIT_ECX_INTERRUPT_BREAK;
}
/*
@@ -173,14 +173,14 @@ void mwait_idle_with_hints(unsigned long ax, unsigned long cx)
}
}
-void acpi_processor_ffh_cstate_enter(struct acpi_processor_cx *cx)
+void acpi_processor_ffh_cstate_enter(int index)
{
unsigned int cpu = smp_processor_id();
struct cstate_entry *percpu_entry;
percpu_entry = per_cpu_ptr(cpu_cstate_entry, cpu);
- mwait_idle_with_hints(percpu_entry->states[cx->index].eax,
- percpu_entry->states[cx->index].ecx);
+ mwait_idle_with_hints(percpu_entry->states[index].eax,
+ percpu_entry->states[index].ecx);
}
EXPORT_SYMBOL_GPL(acpi_processor_ffh_cstate_enter);
diff --git a/drivers/acpi/processor_idle.c b/drivers/acpi/processor_idle.c
index 9ef007d..0922143 100644
--- a/drivers/acpi/processor_idle.c
+++ b/drivers/acpi/processor_idle.c
@@ -430,12 +430,11 @@ static int acpi_processor_get_power_info_cst(struct acpi_processor *pr)
current_count++;
cx.address = reg->address;
- cx.index = current_count + 1;
cx.entry_method = ACPI_CSTATE_SYSTEMIO;
if (reg->space_id == ACPI_ADR_SPACE_FIXED_HARDWARE) {
- if (acpi_processor_ffh_cstate_probe
- (pr->id, &cx, reg) == 0) {
+ if (acpi_processor_ffh_cstate_probe(
+ pr->id, &cx, current_count + 1, reg) == 0) {
cx.entry_method = ACPI_CSTATE_FFH;
} else if (cx.type == ACPI_STATE_C1) {
/*
@@ -704,13 +703,13 @@ static int acpi_idle_bm_check(void)
*
* Caller disables interrupt before call and enables interrupt after return.
*/
-static inline void acpi_idle_do_entry(struct acpi_processor_cx *cx)
+static inline void acpi_idle_do_entry(struct acpi_processor_cx *cx, int index)
{
/* Don't trace irqs off for idle */
stop_critical_timings();
if (cx->entry_method == ACPI_CSTATE_FFH) {
/* Call into architectural FFH based C-state */
- acpi_processor_ffh_cstate_enter(cx);
+ acpi_processor_ffh_cstate_enter(index);
} else if (cx->entry_method == ACPI_CSTATE_HALT) {
acpi_safe_halt();
} else {
@@ -752,7 +751,7 @@ static int acpi_idle_enter_c1(struct cpuidle_device *dev,
lapic_timer_state_broadcast(pr, cx, 1);
kt1 = ktime_get_real();
- acpi_idle_do_entry(cx);
+ acpi_idle_do_entry(cx, index);
kt2 = ktime_get_real();
idle_time = ktime_to_us(ktime_sub(kt2, kt1));
@@ -847,7 +846,7 @@ static int acpi_idle_enter_simple(struct cpuidle_device *dev,
kt1 = ktime_get_real();
/* Tell the scheduler that we are going deep-idle: */
sched_clock_idle_sleep_event();
- acpi_idle_do_entry(cx);
+ acpi_idle_do_entry(cx, index);
kt2 = ktime_get_real();
idle_time_ns = ktime_to_ns(ktime_sub(kt2, kt1));
idle_time = idle_time_ns;
@@ -960,7 +959,7 @@ static int acpi_idle_enter_bm(struct cpuidle_device *dev,
ACPI_FLUSH_CPU_CACHE();
}
- acpi_idle_do_entry(cx);
+ acpi_idle_do_entry(cx, index);
/* Re-enable bus master arbitration */
if (pr->flags.bm_check && pr->flags.bm_control) {
diff --git a/include/acpi/processor.h b/include/acpi/processor.h
index 19423c3..d2016a1 100644
--- a/include/acpi/processor.h
+++ b/include/acpi/processor.h
@@ -57,7 +57,6 @@ struct acpi_processor_cx {
u8 type;
u32 address;
u8 entry_method;
- u8 index;
u32 latency;
u32 power;
u32 usage;
@@ -249,8 +248,9 @@ void acpi_processor_power_init_bm_check(struct acpi_processor_flags *flags,
unsigned int cpu);
int acpi_processor_ffh_cstate_probe(unsigned int cpu,
struct acpi_processor_cx *cx,
+ int index,
struct acpi_power_register *reg);
-void acpi_processor_ffh_cstate_enter(struct acpi_processor_cx *cstate);
+void acpi_processor_ffh_cstate_enter(int index);
#else
static inline void acpi_processor_power_init_bm_check(struct
acpi_processor_flags
@@ -261,13 +261,12 @@ static inline void acpi_processor_power_init_bm_check(struct
}
static inline int acpi_processor_ffh_cstate_probe(unsigned int cpu,
struct acpi_processor_cx *cx,
- struct acpi_power_register
- *reg)
+ int index,
+ struct acpi_power_register *reg)
{
return -1;
}
-static inline void acpi_processor_ffh_cstate_enter(struct acpi_processor_cx
- *cstate)
+static inline void acpi_processor_ffh_cstate_enter(int index)
{
return;
}
--
1.7.5.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 3/5] acpi : remove usage from acpi_processor_cx structure
2012-07-12 15:29 [PATCH 1/5] acpi : remove latency_ticks from acpi_processor_cx structure Daniel Lezcano
2012-07-12 15:30 ` [PATCH 2/5] acpi : remove index " Daniel Lezcano
@ 2012-07-12 15:30 ` Daniel Lezcano
2012-07-12 15:30 ` [PATCH 4/5] acpi : remove power " Daniel Lezcano
` (2 subsequent siblings)
4 siblings, 0 replies; 10+ messages in thread
From: Daniel Lezcano @ 2012-07-12 15:30 UTC (permalink / raw)
To: rjw; +Cc: linux-pm
Remove the usage field as it is not used.
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
---
drivers/acpi/processor_idle.c | 5 -----
include/acpi/processor.h | 1 -
2 files changed, 0 insertions(+), 6 deletions(-)
diff --git a/drivers/acpi/processor_idle.c b/drivers/acpi/processor_idle.c
index 0922143..4c06939 100644
--- a/drivers/acpi/processor_idle.c
+++ b/drivers/acpi/processor_idle.c
@@ -759,7 +759,6 @@ static int acpi_idle_enter_c1(struct cpuidle_device *dev,
dev->last_residency = (int)idle_time;
local_irq_enable();
- cx->usage++;
lapic_timer_state_broadcast(pr, cx, 0);
return index;
@@ -862,8 +861,6 @@ static int acpi_idle_enter_simple(struct cpuidle_device *dev,
if (cx->entry_method != ACPI_CSTATE_FFH)
current_thread_info()->status |= TS_POLLING;
- cx->usage++;
-
lapic_timer_state_broadcast(pr, cx, 0);
cx->time += idle_time;
return index;
@@ -983,8 +980,6 @@ static int acpi_idle_enter_bm(struct cpuidle_device *dev,
if (cx->entry_method != ACPI_CSTATE_FFH)
current_thread_info()->status |= TS_POLLING;
- cx->usage++;
-
lapic_timer_state_broadcast(pr, cx, 0);
cx->time += idle_time;
return index;
diff --git a/include/acpi/processor.h b/include/acpi/processor.h
index d2016a1..6ee4c06 100644
--- a/include/acpi/processor.h
+++ b/include/acpi/processor.h
@@ -59,7 +59,6 @@ struct acpi_processor_cx {
u8 entry_method;
u32 latency;
u32 power;
- u32 usage;
u64 time;
u8 bm_sts_skip;
char desc[ACPI_CX_DESC_LEN];
--
1.7.5.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 4/5] acpi : remove power from acpi_processor_cx structure
2012-07-12 15:29 [PATCH 1/5] acpi : remove latency_ticks from acpi_processor_cx structure Daniel Lezcano
2012-07-12 15:30 ` [PATCH 2/5] acpi : remove index " Daniel Lezcano
2012-07-12 15:30 ` [PATCH 3/5] acpi : remove usage " Daniel Lezcano
@ 2012-07-12 15:30 ` Daniel Lezcano
2012-07-12 15:30 ` [PATCH 5/5] acpi : remove time " Daniel Lezcano
2012-07-12 20:34 ` [PATCH 1/5] acpi : remove latency_ticks " Rafael J. Wysocki
4 siblings, 0 replies; 10+ messages in thread
From: Daniel Lezcano @ 2012-07-12 15:30 UTC (permalink / raw)
To: rjw; +Cc: linux-pm
Remove the power field as it is not used.
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
---
drivers/acpi/processor_idle.c | 2 --
include/acpi/processor.h | 1 -
2 files changed, 0 insertions(+), 3 deletions(-)
diff --git a/drivers/acpi/processor_idle.c b/drivers/acpi/processor_idle.c
index 4c06939..9845739 100644
--- a/drivers/acpi/processor_idle.c
+++ b/drivers/acpi/processor_idle.c
@@ -482,8 +482,6 @@ static int acpi_processor_get_power_info_cst(struct acpi_processor *pr)
if (obj->type != ACPI_TYPE_INTEGER)
continue;
- cx.power = obj->integer.value;
-
current_count++;
memcpy(&(pr->power.states[current_count]), &cx, sizeof(cx));
diff --git a/include/acpi/processor.h b/include/acpi/processor.h
index 6ee4c06..3a675f4 100644
--- a/include/acpi/processor.h
+++ b/include/acpi/processor.h
@@ -58,7 +58,6 @@ struct acpi_processor_cx {
u32 address;
u8 entry_method;
u32 latency;
- u32 power;
u64 time;
u8 bm_sts_skip;
char desc[ACPI_CX_DESC_LEN];
--
1.7.5.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 5/5] acpi : remove time from acpi_processor_cx structure
2012-07-12 15:29 [PATCH 1/5] acpi : remove latency_ticks from acpi_processor_cx structure Daniel Lezcano
` (2 preceding siblings ...)
2012-07-12 15:30 ` [PATCH 4/5] acpi : remove power " Daniel Lezcano
@ 2012-07-12 15:30 ` Daniel Lezcano
2012-07-12 20:34 ` [PATCH 1/5] acpi : remove latency_ticks " Rafael J. Wysocki
4 siblings, 0 replies; 10+ messages in thread
From: Daniel Lezcano @ 2012-07-12 15:30 UTC (permalink / raw)
To: rjw; +Cc: linux-pm
Remove the time field as it is not used.
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
---
drivers/acpi/processor_idle.c | 2 --
include/acpi/processor.h | 1 -
2 files changed, 0 insertions(+), 3 deletions(-)
diff --git a/drivers/acpi/processor_idle.c b/drivers/acpi/processor_idle.c
index 9845739..16c6ebb 100644
--- a/drivers/acpi/processor_idle.c
+++ b/drivers/acpi/processor_idle.c
@@ -860,7 +860,6 @@ static int acpi_idle_enter_simple(struct cpuidle_device *dev,
current_thread_info()->status |= TS_POLLING;
lapic_timer_state_broadcast(pr, cx, 0);
- cx->time += idle_time;
return index;
}
@@ -979,7 +978,6 @@ static int acpi_idle_enter_bm(struct cpuidle_device *dev,
current_thread_info()->status |= TS_POLLING;
lapic_timer_state_broadcast(pr, cx, 0);
- cx->time += idle_time;
return index;
}
diff --git a/include/acpi/processor.h b/include/acpi/processor.h
index 3a675f4..abc89b0 100644
--- a/include/acpi/processor.h
+++ b/include/acpi/processor.h
@@ -58,7 +58,6 @@ struct acpi_processor_cx {
u32 address;
u8 entry_method;
u32 latency;
- u64 time;
u8 bm_sts_skip;
char desc[ACPI_CX_DESC_LEN];
};
--
1.7.5.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 1/5] acpi : remove latency_ticks from acpi_processor_cx structure
2012-07-12 15:29 [PATCH 1/5] acpi : remove latency_ticks from acpi_processor_cx structure Daniel Lezcano
` (3 preceding siblings ...)
2012-07-12 15:30 ` [PATCH 5/5] acpi : remove time " Daniel Lezcano
@ 2012-07-12 20:34 ` Rafael J. Wysocki
2012-07-12 20:47 ` Daniel Lezcano
4 siblings, 1 reply; 10+ messages in thread
From: Rafael J. Wysocki @ 2012-07-12 20:34 UTC (permalink / raw)
To: Daniel Lezcano; +Cc: Linux PM list, ACPI Devel Mailing List
On Thursday, July 12, 2012, Daniel Lezcano wrote:
> Remove the latency_ticks field as it is not used.
>
> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Thanks for the patches, but can you please repost them to
linux-pm@vger.kernel.org so that they are included into the kernel.org
Patchwork? They will be much easier to review and handle then.
And please CC linux-acpi@vger.kernel.org for all ACPI-related patches in the
future.
Thanks,
Rafael
> ---
> drivers/acpi/processor_idle.c | 2 --
> include/acpi/processor.h | 1 -
> 2 files changed, 0 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/acpi/processor_idle.c b/drivers/acpi/processor_idle.c
> index b894627..9ef007d 100644
> --- a/drivers/acpi/processor_idle.c
> +++ b/drivers/acpi/processor_idle.c
> @@ -583,7 +583,6 @@ static void acpi_processor_power_verify_c3(struct acpi_processor *pr,
> */
> cx->valid = 1;
>
> - cx->latency_ticks = cx->latency;
> /*
> * On older chipsets, BM_RLD needs to be set
> * in order for Bus Master activity to wake the
> @@ -616,7 +615,6 @@ static int acpi_processor_power_verify(struct acpi_processor *pr)
> if (!cx->address)
> break;
> cx->valid = 1;
> - cx->latency_ticks = cx->latency; /* Normalize latency */
> break;
>
> case ACPI_STATE_C3:
> diff --git a/include/acpi/processor.h b/include/acpi/processor.h
> index ac3bff6..19423c3 100644
> --- a/include/acpi/processor.h
> +++ b/include/acpi/processor.h
> @@ -59,7 +59,6 @@ struct acpi_processor_cx {
> u8 entry_method;
> u8 index;
> u32 latency;
> - u32 latency_ticks;
> u32 power;
> u32 usage;
> u64 time;
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/5] acpi : remove latency_ticks from acpi_processor_cx structure
2012-07-12 20:34 ` [PATCH 1/5] acpi : remove latency_ticks " Rafael J. Wysocki
@ 2012-07-12 20:47 ` Daniel Lezcano
0 siblings, 0 replies; 10+ messages in thread
From: Daniel Lezcano @ 2012-07-12 20:47 UTC (permalink / raw)
To: Rafael J. Wysocki; +Cc: Linux PM list, ACPI Devel Mailing List
On 07/12/2012 10:34 PM, Rafael J. Wysocki wrote:
> On Thursday, July 12, 2012, Daniel Lezcano wrote:
>> Remove the latency_ticks field as it is not used.
>>
>> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
>
> Thanks for the patches, but can you please repost them to
> linux-pm@vger.kernel.org so that they are included into the kernel.org
> Patchwork? They will be much easier to review and handle then.
>
> And please CC linux-acpi@vger.kernel.org for all ACPI-related patches in the
> future.
Yes, sure. Sorry for that.
Thanks
-- Daniel
>> ---
>> drivers/acpi/processor_idle.c | 2 --
>> include/acpi/processor.h | 1 -
>> 2 files changed, 0 insertions(+), 3 deletions(-)
>>
>> diff --git a/drivers/acpi/processor_idle.c b/drivers/acpi/processor_idle.c
>> index b894627..9ef007d 100644
>> --- a/drivers/acpi/processor_idle.c
>> +++ b/drivers/acpi/processor_idle.c
>> @@ -583,7 +583,6 @@ static void acpi_processor_power_verify_c3(struct acpi_processor *pr,
>> */
>> cx->valid = 1;
>>
>> - cx->latency_ticks = cx->latency;
>> /*
>> * On older chipsets, BM_RLD needs to be set
>> * in order for Bus Master activity to wake the
>> @@ -616,7 +615,6 @@ static int acpi_processor_power_verify(struct acpi_processor *pr)
>> if (!cx->address)
>> break;
>> cx->valid = 1;
>> - cx->latency_ticks = cx->latency; /* Normalize latency */
>> break;
>>
>> case ACPI_STATE_C3:
>> diff --git a/include/acpi/processor.h b/include/acpi/processor.h
>> index ac3bff6..19423c3 100644
>> --- a/include/acpi/processor.h
>> +++ b/include/acpi/processor.h
>> @@ -59,7 +59,6 @@ struct acpi_processor_cx {
>> u8 entry_method;
>> u8 index;
>> u32 latency;
>> - u32 latency_ticks;
>> u32 power;
>> u32 usage;
>> u64 time;
>>
>
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 1/5] acpi : remove latency_ticks from acpi_processor_cx structure
@ 2012-07-12 21:03 Daniel Lezcano
2012-07-12 21:40 ` Rafael J. Wysocki
2012-07-17 20:32 ` Rafael J. Wysocki
0 siblings, 2 replies; 10+ messages in thread
From: Daniel Lezcano @ 2012-07-12 21:03 UTC (permalink / raw)
To: rjw, lenb; +Cc: linux-pm, linux-acpi
Remove the latency_ticks field as it is not used.
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
---
drivers/acpi/processor_idle.c | 2 --
include/acpi/processor.h | 1 -
2 files changed, 0 insertions(+), 3 deletions(-)
diff --git a/drivers/acpi/processor_idle.c b/drivers/acpi/processor_idle.c
index f3decb3..da92c65 100644
--- a/drivers/acpi/processor_idle.c
+++ b/drivers/acpi/processor_idle.c
@@ -586,7 +586,6 @@ static void acpi_processor_power_verify_c3(struct acpi_processor *pr,
*/
cx->valid = 1;
- cx->latency_ticks = cx->latency;
/*
* On older chipsets, BM_RLD needs to be set
* in order for Bus Master activity to wake the
@@ -619,7 +618,6 @@ static int acpi_processor_power_verify(struct acpi_processor *pr)
if (!cx->address)
break;
cx->valid = 1;
- cx->latency_ticks = cx->latency; /* Normalize latency */
break;
case ACPI_STATE_C3:
diff --git a/include/acpi/processor.h b/include/acpi/processor.h
index 9d65047..a70cbcc 100644
--- a/include/acpi/processor.h
+++ b/include/acpi/processor.h
@@ -59,7 +59,6 @@ struct acpi_processor_cx {
u8 entry_method;
u8 index;
u32 latency;
- u32 latency_ticks;
u32 power;
u32 usage;
u64 time;
--
1.7.5.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 1/5] acpi : remove latency_ticks from acpi_processor_cx structure
2012-07-12 21:03 Daniel Lezcano
@ 2012-07-12 21:40 ` Rafael J. Wysocki
2012-07-17 20:32 ` Rafael J. Wysocki
1 sibling, 0 replies; 10+ messages in thread
From: Rafael J. Wysocki @ 2012-07-12 21:40 UTC (permalink / raw)
To: Daniel Lezcano; +Cc: lenb, linux-pm, linux-acpi
On Thursday, July 12, 2012, Daniel Lezcano wrote:
> Remove the latency_ticks field as it is not used.
>
> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
This one looks good to me.
Thanks,
Rafael
> ---
> drivers/acpi/processor_idle.c | 2 --
> include/acpi/processor.h | 1 -
> 2 files changed, 0 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/acpi/processor_idle.c b/drivers/acpi/processor_idle.c
> index f3decb3..da92c65 100644
> --- a/drivers/acpi/processor_idle.c
> +++ b/drivers/acpi/processor_idle.c
> @@ -586,7 +586,6 @@ static void acpi_processor_power_verify_c3(struct acpi_processor *pr,
> */
> cx->valid = 1;
>
> - cx->latency_ticks = cx->latency;
> /*
> * On older chipsets, BM_RLD needs to be set
> * in order for Bus Master activity to wake the
> @@ -619,7 +618,6 @@ static int acpi_processor_power_verify(struct acpi_processor *pr)
> if (!cx->address)
> break;
> cx->valid = 1;
> - cx->latency_ticks = cx->latency; /* Normalize latency */
> break;
>
> case ACPI_STATE_C3:
> diff --git a/include/acpi/processor.h b/include/acpi/processor.h
> index 9d65047..a70cbcc 100644
> --- a/include/acpi/processor.h
> +++ b/include/acpi/processor.h
> @@ -59,7 +59,6 @@ struct acpi_processor_cx {
> u8 entry_method;
> u8 index;
> u32 latency;
> - u32 latency_ticks;
> u32 power;
> u32 usage;
> u64 time;
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/5] acpi : remove latency_ticks from acpi_processor_cx structure
2012-07-12 21:03 Daniel Lezcano
2012-07-12 21:40 ` Rafael J. Wysocki
@ 2012-07-17 20:32 ` Rafael J. Wysocki
1 sibling, 0 replies; 10+ messages in thread
From: Rafael J. Wysocki @ 2012-07-17 20:32 UTC (permalink / raw)
To: Daniel Lezcano; +Cc: lenb, linux-pm, linux-acpi
On Thursday, July 12, 2012, Daniel Lezcano wrote:
> Remove the latency_ticks field as it is not used.
>
> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Applied to the 'pm-cpuidle' branch of the linux-pm.git tree.
Thanks,
Rafael
> ---
> drivers/acpi/processor_idle.c | 2 --
> include/acpi/processor.h | 1 -
> 2 files changed, 0 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/acpi/processor_idle.c b/drivers/acpi/processor_idle.c
> index f3decb3..da92c65 100644
> --- a/drivers/acpi/processor_idle.c
> +++ b/drivers/acpi/processor_idle.c
> @@ -586,7 +586,6 @@ static void acpi_processor_power_verify_c3(struct acpi_processor *pr,
> */
> cx->valid = 1;
>
> - cx->latency_ticks = cx->latency;
> /*
> * On older chipsets, BM_RLD needs to be set
> * in order for Bus Master activity to wake the
> @@ -619,7 +618,6 @@ static int acpi_processor_power_verify(struct acpi_processor *pr)
> if (!cx->address)
> break;
> cx->valid = 1;
> - cx->latency_ticks = cx->latency; /* Normalize latency */
> break;
>
> case ACPI_STATE_C3:
> diff --git a/include/acpi/processor.h b/include/acpi/processor.h
> index 9d65047..a70cbcc 100644
> --- a/include/acpi/processor.h
> +++ b/include/acpi/processor.h
> @@ -59,7 +59,6 @@ struct acpi_processor_cx {
> u8 entry_method;
> u8 index;
> u32 latency;
> - u32 latency_ticks;
> u32 power;
> u32 usage;
> u64 time;
>
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2012-07-17 20:27 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-07-12 15:29 [PATCH 1/5] acpi : remove latency_ticks from acpi_processor_cx structure Daniel Lezcano
2012-07-12 15:30 ` [PATCH 2/5] acpi : remove index " Daniel Lezcano
2012-07-12 15:30 ` [PATCH 3/5] acpi : remove usage " Daniel Lezcano
2012-07-12 15:30 ` [PATCH 4/5] acpi : remove power " Daniel Lezcano
2012-07-12 15:30 ` [PATCH 5/5] acpi : remove time " Daniel Lezcano
2012-07-12 20:34 ` [PATCH 1/5] acpi : remove latency_ticks " Rafael J. Wysocki
2012-07-12 20:47 ` Daniel Lezcano
-- strict thread matches above, loose matches on Subject: below --
2012-07-12 21:03 Daniel Lezcano
2012-07-12 21:40 ` Rafael J. Wysocki
2012-07-17 20:32 ` Rafael J. Wysocki
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.