On 05/30/2012 09:15 AM, kejun.zhou wrote:
> Signed-off-by: kejun.zhou <kejun.z...@linaro.org>
> ---
>  cpuidle/cpuidle_killer.c |    6 ++++--
>  1 files changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/cpuidle/cpuidle_killer.c b/cpuidle/cpuidle_killer.c
> index c8910f4..0e4d666 100644
> --- a/cpuidle/cpuidle_killer.c
> +++ b/cpuidle/cpuidle_killer.c
> @@ -5,8 +5,10 @@
>  #ifndef ANDROID
>  #include <sys/timex.h>
>  #else
> -// As of 4.0.4, Bionic doesn't provide the timex/adjtimex interface
> -// However, the kernel does...
> +/* 
> +* As of 4.0.4, Bionic doesn't provide the timex/adjtimex interface
> +* However, the kernel does...
> +*/
>  #include <linux/timex.h> // for struct timex
>  #include <asm/unistd.h> // for __NR_adjtimex
>  static int adjtimex(struct timex *buf) {

Hmm ... Usually, you should resend your patchset updated with the fixes,
not one on top of your patches.
Also, you did not fix the C-style comments 'everywhere' :)

-- 
 <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


_______________________________________________
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev

Reply via email to