Nishanth Menon <[email protected]> writes:

> Since we already know the state of the autocomp enablement, we can
> see if the requested state is different from the current state and
> enable/disable SR only on the need basis.
>
> Signed-off-by: Nishanth Menon <[email protected]>

Thanks, queuing for 2.6.40.

Kevin

> ---
>  arch/arm/mach-omap2/smartreflex.c |   11 +++++++----
>  1 files changed, 7 insertions(+), 4 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/smartreflex.c 
> b/arch/arm/mach-omap2/smartreflex.c
> index f9057ad..2b6fdc7 100644
> --- a/arch/arm/mach-omap2/smartreflex.c
> +++ b/arch/arm/mach-omap2/smartreflex.c
> @@ -806,10 +806,13 @@ static int omap_sr_autocomp_store(void *data, u64 val)
>               return -EINVAL;
>       }
>  
> -     if (!val)
> -             sr_stop_vddautocomp(sr_info);
> -     else
> -             sr_start_vddautocomp(sr_info);
> +     /* control enable/disable only if there is a delta in value */
> +     if (sr_info->autocomp_active != val) {
> +             if (!val)
> +                     sr_stop_vddautocomp(sr_info);
> +             else
> +                     sr_start_vddautocomp(sr_info);
> +     }
>  
>       return 0;
>  }
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to