Re: [PATCH 3/5] acpi : remove usage from acpi_processor_cx structure

["Rafael J. Wysocki" <rjw@sisk.pl>]

On Thursday, July 12, 2012, Daniel Lezcano wrote:
> Remove the usage 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 |    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 9808e60..d044588 100644
> --- a/drivers/acpi/processor_idle.c
> +++ b/drivers/acpi/processor_idle.c
> @@ -761,7 +761,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;
> @@ -863,8 +862,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 d09c94f..0957457 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];
>