x86, mce, therm_throt.c: Fix missing curly braces in error handling logic
authorJin Dongming <[email protected]>
Thu, 26 Aug 2010 08:29:05 +0000 (17:29 +0900)
committerIngo Molnar <[email protected]>
Fri, 8 Oct 2010 08:29:20 +0000 (10:29 +0200)
When the feature PTS is not supported by CPU, the sysfile
package_power_limit_count for package should not be
generated.

This patch is used for fixing missing { and }.

The patch is not complete as there are other error handling
problems in this function - but that can wait until the
merge window.

Signed-off-by: Jin Dongming <[email protected]>
Reviewed-by: Fenghua Yu <[email protected]>
Acked-by: Jean Delvare <[email protected]>
Cc: Brown Len <[email protected]>
Cc: Guenter Roeck <[email protected]>
Cc: Hidetoshi Seto <[email protected]>
Cc: [email protected] <[email protected]>
LKML-Reference: <4C7625D1.4060201@np.css.fujitsu.com>
Signed-off-by: Ingo Molnar <[email protected]>
arch/x86/kernel/cpu/mcheck/therm_throt.c

index d9368eeda3090eb9f53482704e000e600b6237e3..169d8804a9f8eed37faf06503e1b8243a566cb37 100644 (file)
@@ -216,7 +216,7 @@ static __cpuinit int thermal_throttle_add_dev(struct sys_device *sys_dev,
                err = sysfs_add_file_to_group(&sys_dev->kobj,
                                              &attr_core_power_limit_count.attr,
                                              thermal_attr_group.name);
-       if (cpu_has(c, X86_FEATURE_PTS))
+       if (cpu_has(c, X86_FEATURE_PTS)) {
                err = sysfs_add_file_to_group(&sys_dev->kobj,
                                              &attr_package_throttle_count.attr,
                                              thermal_attr_group.name);
@@ -224,6 +224,7 @@ static __cpuinit int thermal_throttle_add_dev(struct sys_device *sys_dev,
                        err = sysfs_add_file_to_group(&sys_dev->kobj,
                                        &attr_package_power_limit_count.attr,
                                        thermal_attr_group.name);
+       }
 
        return err;
 }