diff mbox

[5/5] acpi : remove time from acpi_processor_cx structure

Message ID 1342127026-1526-5-git-send-email-daniel.lezcano@linaro.org (mailing list archive)
State Accepted, archived
Headers show

Commit Message

Daniel Lezcano July 12, 2012, 9:03 p.m. UTC
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(-)

Comments

Rafael Wysocki July 12, 2012, 9:46 p.m. UTC | #1
On Thursday, July 12, 2012, Daniel Lezcano wrote:
> Remove the time field as it is not used.
> 
> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>

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 99ba58f..bc1ce58 100644
> --- a/drivers/acpi/processor_idle.c
> +++ b/drivers/acpi/processor_idle.c
> @@ -861,7 +861,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 87bb9d7..3998334 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];
>  };
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-pm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Rafael Wysocki July 17, 2012, 8:35 p.m. UTC | #2
On Thursday, July 12, 2012, Daniel Lezcano wrote:
> Remove the time 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 99ba58f..bc1ce58 100644
> --- a/drivers/acpi/processor_idle.c
> +++ b/drivers/acpi/processor_idle.c
> @@ -861,7 +861,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 87bb9d7..3998334 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];
>  };
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-pm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/acpi/processor_idle.c b/drivers/acpi/processor_idle.c
index 99ba58f..bc1ce58 100644
--- a/drivers/acpi/processor_idle.c
+++ b/drivers/acpi/processor_idle.c
@@ -861,7 +861,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 87bb9d7..3998334 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];
 };