]> nv-tegra.nvidia Code Review - linux-2.6.git/commitdiff
lp8727_charger: Fix wrong code style
authorKim, Milo <Milo.Kim@ti.com>
Fri, 27 Jan 2012 06:58:51 +0000 (22:58 -0800)
committerAnton Vorontsov <anton.vorontsov@linaro.org>
Mon, 26 Mar 2012 16:40:56 +0000 (20:40 +0400)
Definition of STATUS2
: remove space before tabs.

Return code of lp8727_is_dedicated_charger(), lp8727_is_usb_charger()
: remove parentheses when return is not a function.

MODULE_AUTHOR section
: remove space at the start of a line.

Signed-off-by: Milo(Woogyom) Kim <milo.kim@ti.com>
Signed-off-by: Anton Vorontsov <anton.vorontsov@linaro.org>
drivers/power/lp8727_charger.c

index 1e8a98b29d80d34644f58c1a58c6edc275907626..d5aab54eda11a7c72d846c9d8f1fa83a47608829 100644 (file)
@@ -26,7 +26,7 @@
 #define INT1           0x4
 #define INT2           0x5
 #define STATUS1                0x6
-#define STATUS2        0x7
+#define STATUS2                0x7
 #define CHGCTRL2       0x9
 
 /* CTRL1 register */
@@ -153,14 +153,14 @@ static int lp8727_is_dedicated_charger(struct lp8727_chg *pchg)
 {
        u8 val;
        lp8727_read_byte(pchg, STATUS1, &val);
-       return (val & DCPORT);
+       return val & DCPORT;
 }
 
 static int lp8727_is_usb_charger(struct lp8727_chg *pchg)
 {
        u8 val;
        lp8727_read_byte(pchg, STATUS1, &val);
-       return (val & CHPORT);
+       return val & CHPORT;
 }
 
 static void lp8727_ctrl_switch(struct lp8727_chg *pchg, u8 sw)
@@ -504,6 +504,6 @@ module_init(lp8727_init);
 module_exit(lp8727_exit);
 
 MODULE_DESCRIPTION("TI/National Semiconductor LP8727 charger driver");
-MODULE_AUTHOR
-    ("Woogyom Kim <milo.kim@ti.com>, Daniel Jeong <daniel.jeong@ti.com>");
+MODULE_AUTHOR("Woogyom Kim <milo.kim@ti.com>, "
+             "Daniel Jeong <daniel.jeong@ti.com>");
 MODULE_LICENSE("GPL");