Skip to contentSkip to conversation

Remove unnecessary braces

PATCH Applied Evgeny Kapaev 3 participants 3 messages Oct 15, 2013 – Oct 16, 2013 CPU frequency scaling framework

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 Remove unnecessary braces

As per coding style, braces {} are not necessary for single statement block

---
 drivers/cpufreq/speedstep-centrino.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
drivers/cpufreq/speedstep-centrino.c +1 -2
index f897d51..98797a6 100644
@@ -373,9 +373,8 @@ static int centrino_cpu_init(struct cpufreq_policy *policy)
 		return -ENODEV;
 	}
 
-	if (centrino_cpu_init_table(policy)) {
+	if (centrino_cpu_init_table(policy))
 		return -ENODEV;
-	}
 
 	/* Check to see if Enhanced SpeedStep is enabled, and try to
 	   enable it if not. */
Signature · 1 line
1.8.1.2
maintainer Acked-by on v1 1/1
#1 lore raw
Quoted 23 lines · Evgeny Kapaev

As per coding style, braces {} are not necessary for single statement block

---
 drivers/cpufreq/speedstep-centrino.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
drivers/cpufreq/speedstep-centrino.c +1 -2
index f897d51..98797a6 100644
@@ -373,9 +373,8 @@ static int centrino_cpu_init(struct cpufreq_policy
 *policy)
 		return -ENODEV;
 	}
 
-	if (centrino_cpu_init_table(policy)) {
+	if (centrino_cpu_init_table(policy))
 		return -ENODEV;
-	}
 
 	/* Check to see if Enhanced SpeedStep is enabled, and try to
Quoted 1 line

enable it if not. */

#2 lore raw
Quoted 4 lines · Viresh Kumar
Quoted 3 lines · Evgeny Kapaev

As per coding style, braces {} are not necessary for single statement block

Queued up for 3.13, thanks!

Quoted 27 lines
Quoted 20 lines
---
 drivers/cpufreq/speedstep-centrino.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
drivers/cpufreq/speedstep-centrino.c +1 -2
index f897d51..98797a6 100644
@@ -373,9 +373,8 @@ static int centrino_cpu_init(struct cpufreq_policy
 *policy)
 		return -ENODEV;
 	}
 
-	if (centrino_cpu_init_table(policy)) {
+	if (centrino_cpu_init_table(policy))
 		return -ENODEV;
-	}
 
 	/* Check to see if Enhanced SpeedStep is enabled, and try to
Quoted 1 line

enable it if not. */

--
To unsubscribe from this list: send the line "unsubscribe linux-pm" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html

Signature · 2 lines
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.