linux-kernel - RE: [PATCH 3/3] power: Add LP8727 charger driver

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]

Date:	Wed, 12 Oct 2011 23:06:10 -0700
From:	"Kim, Milo" <Milo.Kim@...com>
To:	"Anton Vorontsov" <cbouatmailru@...il.com>
cc:	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"dwmw2@...radead.org" <dwmw2@...radead.org>
Subject: RE: [PATCH 3/3] power: Add LP8727 charger driver

Hi, Anton Vorontsov

I send the revised patch.

Signed-off-by: Woogyom Kim <milo.kim@...com>

>From 809577ee23f2d27df449b84310eb6ebf0ea98332 Mon Sep 17 00:00:00 2001
From: Milo(Woogyom) Kim <milo.kim@...com>
Date: Thu, 13 Oct 2011 14:56:08 +0900
Subject: [PATCH] power: lp8727 coding style changes

Pointer coding style changes
: add space between return type and function pointer
  ex) u8(*get_batt_present) (void)
   -> u8 (*get_batt_present) (void)
---
 include/linux/lp8727.h |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/include/linux/lp8727.h b/include/linux/lp8727.h
index 3ec5589..ef30c7c 100755
--- a/include/linux/lp8727.h
+++ b/include/linux/lp8727.h
@@ -43,10 +43,10 @@ struct lp8727_chg_param {
 };
 
 struct lp8727_platform_data {
-	u8(*get_batt_present) (void);
-	u16(*get_batt_level) (void);
-	u8(*get_batt_capacity) (void);
-	u8(*get_batt_temp) (void);
+	u8 (*get_batt_present) (void);
+	u16 (*get_batt_level) (void);
+	u8 (*get_batt_capacity) (void);
+	u8 (*get_batt_temp) (void);
 	struct lp8727_chg_param ac;
 	struct lp8727_chg_param usb;
 };
-- 
1.7.4.1

Thanks & BR
Milo (Woogyom) Kim
milo.kim@...com

-----Original Message-----
From: Kim, Milo 
Sent: Wednesday, September 07, 2011 5:56 PM
To: 'cbouatmailru@...il.com'
Cc: 'linux-kernel@...r.kernel.org'; 'dwmw2@...radead.org'; 'cbou@...l.ru'
Subject: [PATCH 3/3] power: Add LP8727 charger driver


diff --git a/include/linux/lp8727.h b/include/linux/lp8727.h
new file mode 100755
index 0000000..3ec5589
--- /dev/null
+++ b/include/linux/lp8727.h
@@ -0,0 +1,54 @@
+/*
+ * lp8727.h - Driver for LP8727 Micro/Mini USB IC with intergrated charger
+ *
+ *			Copyright (C) 2011 National Semiconductor
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ *
+ */
+
+#ifndef _LP8727_H
+#define _LP8727_H
+
+enum lp8727_eoc_level {
+	EOC_5P,
+	EOC_10P,
+	EOC_16P,
+	EOC_20P,
+	EOC_25P,
+	EOC_33P,
+	EOC_50P,
+};
+
+enum lp8727_ichg {
+	ICHG_90mA,
+	ICHG_100mA,
+	ICHG_400mA,
+	ICHG_450mA,
+	ICHG_500mA,
+	ICHG_600mA,
+	ICHG_700mA,
+	ICHG_800mA,
+	ICHG_900mA,
+	ICHG_1000mA,
+};
+
+struct lp8727_chg_param {
+	/* end of charge level setting */
+	enum lp8727_eoc_level eoc_level;
+	/* charging current */
+	enum lp8727_ichg ichg;
+};
+
+struct lp8727_platform_data {
+	u8(*get_batt_present) (void);
+	u16(*get_batt_level) (void);
+	u8(*get_batt_capacity) (void);
+	u8(*get_batt_temp) (void);
+	struct lp8727_chg_param ac;
+	struct lp8727_chg_param usb;
+};
+
+#endif
-- 
1.7.1


Best Regards
------------------------------------------------------------------------------------------------------------
Milo (Woo Gyom) KIM
National Semiconductor
milo.kim@....com
------------------------------------------------------------------------------------------------------------



--
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/