Doug / smarc-fsl-linux-kernel

Download as
Browse Code ยป

Commit 58c3667d83376c2f3016465f59f22c19c8f846e1

Authored by Guenter Roeck
1 parent 28a33cbc24
Exists in smarc-l5.0.0_1.0.0-ga and in 5 other branches imx_3.10.17_1.0.1_ga, imx_3.10.53_1.1.0_ga, imx_3.14.28_1.0.0_ga, smarc-imx_3.10.53_1.1.0_ga, smarc-imx_3.14.28_1.0.0_ga

hwmon: (w83627ehf) Add missing break statement 

A case statement in nct6775_write_fan_div() is missing a break. Fix it.

This patch addresses Coverity #141439: Missing break in switch.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Acked-by: Jean Delvare <khali@linux-fr.org>

Showing 1 changed file with 1 additions and 0 deletions Side-by-side Diff

drivers/hwmon/w83627ehf.c
Diff comments   View file @ 58c3667
... ... @@ -599,6 +599,7 @@
599 599 reg = (w83627ehf_read_value(data, NCT6775_REG_FANDIV1) & 0x7)
600 600 | ((data->fan_div[1] << 4) & 0x70);
601 601 w83627ehf_write_value(data, NCT6775_REG_FANDIV1, reg);
  602 + break;
602 603 case 2:
603 604 reg = (w83627ehf_read_value(data, NCT6775_REG_FANDIV2) & 0x70)
604 605 | (data->fan_div[2] & 0x7);