linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] drivers/idle/intel_idle.c: Fix confusing code identation
@ 2012-02-17  8:43 Marcos Paulo de Souza
  2012-02-17  8:57 ` Srivatsa S. Bhat
  0 siblings, 1 reply; 2+ messages in thread
From: Marcos Paulo de Souza @ 2012-02-17  8:43 UTC (permalink / raw)
  To: lenb; +Cc: linux-pm, linux-kernel, srivatsa.bhat, Marcos Paulo de Souza

Suggested-by: Srivatsa S. Bhat <srivatsa.bhat@linux.vnet.ibm.com>
Signed-off-by: Marcos Paulo de Souza <marcos.mage@gmail.com>
---
 drivers/idle/intel_idle.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/idle/intel_idle.c b/drivers/idle/intel_idle.c
index 54ab97b..5e71b98 100644
--- a/drivers/idle/intel_idle.c
+++ b/drivers/idle/intel_idle.c
@@ -495,8 +495,7 @@ int intel_idle_cpu_init(int cpu)
 		int num_substates;
 
 		if (cstate > max_cstate) {
-			printk(PREFIX "max_cstate %d reached\n",
-			       max_cstate);
+			printk(PREFIX "max_cstate %d reached\n", max_cstate);
 			break;
 		}
 
@@ -512,8 +511,9 @@ int intel_idle_cpu_init(int cpu)
 		dev->states_usage[dev->state_count].driver_data =
 			(void *)get_driver_data(cstate);
 
-			dev->state_count += 1;
-		}
+		dev->state_count += 1;
+	}
+
 	dev->cpu = cpu;
 
 	if (cpuidle_register_device(dev)) {
-- 
1.7.7.6


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] drivers/idle/intel_idle.c: Fix confusing code identation
  2012-02-17  8:43 [PATCH] drivers/idle/intel_idle.c: Fix confusing code identation Marcos Paulo de Souza
@ 2012-02-17  8:57 ` Srivatsa S. Bhat
  0 siblings, 0 replies; 2+ messages in thread
From: Srivatsa S. Bhat @ 2012-02-17  8:57 UTC (permalink / raw)
  To: Marcos Paulo de Souza; +Cc: lenb, linux-pm, linux-kernel

On 02/17/2012 02:13 PM, Marcos Paulo de Souza wrote:

> Suggested-by: Srivatsa S. Bhat <srivatsa.bhat@linux.vnet.ibm.com>
> Signed-off-by: Marcos Paulo de Souza <marcos.mage@gmail.com>
> ---


Reviewed-by: Srivatsa S. Bhat <srivatsa.bhat@linux.vnet.ibm.com>

[ A small commit message wouldn't hurt though :-) ]

Regards,
Srivatsa S. Bhat

>  drivers/idle/intel_idle.c |    8 ++++----
>  1 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/idle/intel_idle.c b/drivers/idle/intel_idle.c
> index 54ab97b..5e71b98 100644
> --- a/drivers/idle/intel_idle.c
> +++ b/drivers/idle/intel_idle.c
> @@ -495,8 +495,7 @@ int intel_idle_cpu_init(int cpu)
>  		int num_substates;
> 
>  		if (cstate > max_cstate) {
> -			printk(PREFIX "max_cstate %d reached\n",
> -			       max_cstate);
> +			printk(PREFIX "max_cstate %d reached\n", max_cstate);
>  			break;
>  		}
> 
> @@ -512,8 +511,9 @@ int intel_idle_cpu_init(int cpu)
>  		dev->states_usage[dev->state_count].driver_data =
>  			(void *)get_driver_data(cstate);
> 
> -			dev->state_count += 1;
> -		}
> +		dev->state_count += 1;
> +	}
> +
>  	dev->cpu = cpu;
> 
>  	if (cpuidle_register_device(dev)) {


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2012-02-17  8:58 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-02-17  8:43 [PATCH] drivers/idle/intel_idle.c: Fix confusing code identation Marcos Paulo de Souza
2012-02-17  8:57 ` Srivatsa S. Bhat

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).