]> Pileus Git - ~andy/linux/commitdiff
regulator: pfuze100-regulator: Fix some checkpatch complaints
authorFabio Estevam <fabio.estevam@freescale.com>
Wed, 15 Jan 2014 02:52:45 +0000 (00:52 -0200)
committerMark Brown <broonie@linaro.org>
Wed, 15 Jan 2014 11:46:56 +0000 (11:46 +0000)
Fix the following checkpatch error and warning:

ERROR: switch and case should be at the same indent
#311: FILE: drivers/regulator/pfuze100-regulator.c:311:
+ switch (value & 0x0f) {
[...]
+ case 0x8:
[...]
+ case 0x0:
[...]
+ default:

WARNING: line over 80 characters
#312: FILE: drivers/regulator/pfuze100-regulator.c:312:
+ /* Freescale misprogrammed 1-3% of parts prior to week 8 of 2013 as ID=8 */

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
Signed-off-by: Mark Brown <broonie@linaro.org>
drivers/regulator/pfuze100-regulator.c

index 0c511ae52c4228be93bcd52cc48d24977d78c647..f68e5d5a011e3b532d10bea19c64dff1b00907d2 100644 (file)
@@ -309,14 +309,17 @@ static int pfuze_identify(struct pfuze_chip *pfuze_chip)
                return ret;
 
        switch (value & 0x0f) {
-               /* Freescale misprogrammed 1-3% of parts prior to week 8 of 2013 as ID=8 */
-               case 0x8:
-                       dev_info(pfuze_chip->dev, "Assuming misprogrammed ID=0x8");
-               case 0x0:
-                       break;
-               default:
-                       dev_warn(pfuze_chip->dev, "Illegal ID: %x\n", value);
-                       return -ENODEV;
+       /*
+        * Freescale misprogrammed 1-3% of parts prior to week 8 of 2013
+        * as ID=8
+        */
+       case 0x8:
+               dev_info(pfuze_chip->dev, "Assuming misprogrammed ID=0x8");
+       case 0x0:
+               break;
+       default:
+               dev_warn(pfuze_chip->dev, "Illegal ID: %x\n", value);
+               return -ENODEV;
        }
 
        ret = regmap_read(pfuze_chip->regmap, PFUZE100_REVID, &value);