Skip to contentSkip to conversation

drivers/idle/intel_idle.c: Fix confusing code identation

PATCH Superseded Marcos Paulo de Souza 2 participants 2 messages Feb 17, 2012 Intel idle driver

v1 of 2 Superseded Latest is v2 →

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 drivers/idle/intel_idle.c: Fix confusing code identation

---
 drivers/idle/intel_idle.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)
drivers/idle/intel_idle.c +4 -4
index 54ab97b..5e71b98 100644
@@ -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)) {
Signature · 1 line
1.7.7.6
Reviewed-by on v1 1/1
#1 lore raw
Quoted 3 lines · Marcos Paulo de Souza

---

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

Regards,
Srivatsa S. Bhat

Quoted 29 lines
 drivers/idle/intel_idle.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)
drivers/idle/intel_idle.c +4 -4
index 54ab97b..5e71b98 100644
@@ -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)) {