lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [day] [month] [year] [list]
Message-ID: <B567DBAB974C0544994013492B949F8E3812D6EB09@EXMAIL03.scwf.nsc.com>
Date:	Thu, 26 Jan 2012 22:58:51 -0800
From:	"Kim, Milo" <Milo.Kim@...com>
To:	"Anton Vorontsov" <cbouatmailru@...il.com>
cc:	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: [PATCH 4/6] lp8727_charger: fix wrong code style

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@...com>
---
 drivers/power/lp8727_charger.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/power/lp8727_charger.c b/drivers/power/lp8727_charger.c
index b333cdd..b531725 100755
--- a/drivers/power/lp8727_charger.c
+++ b/drivers/power/lp8727_charger.c
@@ -25,7 +25,7 @@
 #define INT1		0x4
 #define INT2		0x5
 #define STATUS1		0x6
-#define STATUS2 	0x7
+#define STATUS2		0x7
 #define CHGCTRL2	0x9
 
 /* CTRL1 register */
@@ -152,14 +152,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)
@@ -503,5 +503,5 @@ module_exit(lp8727_exit);
 
 MODULE_DESCRIPTION("Texas Instruments LP8727 charger driver");
 MODULE_AUTHOR
-    ("Woogyom Kim <milo.kim@...com>, Daniel Jeong <daniel.jeong@...com>");
+("Woogyom Kim <milo.kim@...com>, Daniel Jeong <daniel.jeong@...com>");
 MODULE_LICENSE("GPL");
-- 
1.7.4.1


Best Regards,
Milo (Woogyom) Kim
Texas Instruments Incorporated




--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ