diff mbox

[1/3] ARM: pxa: add gpio-charger configuration on poodle

Message ID 1427711971-8164-1-git-send-email-dbaryshkov@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Dmitry Baryshkov March 30, 2015, 10:39 a.m. UTC
Add declration of gpio-charger device for Sharp SL-5600 poodle device.

Signed-off-by: Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>
---
 arch/arm/mach-pxa/poodle.c | 20 ++++++++++++++++++++
 1 file changed, 20 insertions(+)

Comments

Robert Jarzmik April 3, 2015, 7:16 p.m. UTC | #1
Dmitry Eremin-Solenikov <dbaryshkov@gmail.com> writes:

> Add declration of gpio-charger device for Sharp SL-5600 poodle device.
Typo here ^.

And I'd like in the commit message to know :
 - is it the main battery charger (I could look into the code, I know, but I'd
   like it to be in the commit message anyway) ?
 - was it successfully tested on Sharp SL-5600 ?

Cheers.

--
Robert
Dmitry Baryshkov April 3, 2015, 7:23 p.m. UTC | #2
2015-04-03 22:16 GMT+03:00 Robert Jarzmik <robert.jarzmik@free.fr>:
> Dmitry Eremin-Solenikov <dbaryshkov@gmail.com> writes:
>
>> Add declration of gpio-charger device for Sharp SL-5600 poodle device.
> Typo here ^.
>
> And I'd like in the commit message to know :
>  - is it the main battery charger (I could look into the code, I know, but I'd
>    like it to be in the commit message anyway) ?

I'll add it to v2.

>  - was it successfully tested on Sharp SL-5600 ?

:)) I wouldn't dare to submit a code that wasn't tested on hw w/o
special notice.
Robert Jarzmik June 19, 2015, 6:43 p.m. UTC | #3
Dmitry Eremin-Solenikov <dbaryshkov@gmail.com> writes:

> 2015-04-03 22:16 GMT+03:00 Robert Jarzmik <robert.jarzmik@free.fr>:
>> Dmitry Eremin-Solenikov <dbaryshkov@gmail.com> writes:
>>
>>> Add declration of gpio-charger device for Sharp SL-5600 poodle device.
>> Typo here ^.
>>
>> And I'd like in the commit message to know :
>>  - is it the main battery charger (I could look into the code, I know, but I'd
>>    like it to be in the commit message anyway) ?
>
> I'll add it to v2.
>
>>  - was it successfully tested on Sharp SL-5600 ?
>
> :)) I wouldn't dare to submit a code that wasn't tested on hw w/o
> special notice.

Hi Dmitry,

I'll drop that from my radar, I didn't see any update lately. Once the poodle
battery driver is accepted and once you provide an update on these patches, I'll
come back to these patches.

Cheers.
diff mbox

Patch

diff --git a/arch/arm/mach-pxa/poodle.c b/arch/arm/mach-pxa/poodle.c
index 6002112..4dd29a7 100644
--- a/arch/arm/mach-pxa/poodle.c
+++ b/arch/arm/mach-pxa/poodle.c
@@ -31,6 +31,7 @@ 
 #include <linux/spi/ads7846.h>
 #include <linux/spi/pxa2xx_spi.h>
 #include <linux/mtd/sharpsl.h>
+#include <linux/power/gpio-charger.h>
 #include <linux/memblock.h>
 #include <linux/iio/machine.h>
 #include <linux/mfd/locomo.h>
@@ -413,12 +414,31 @@  static struct platform_device sharpsl_rom_device = {
 	.dev.platform_data = &sharpsl_rom_data,
 };
 
+static char *poodle_ac_supplied_to[] = {
+	"main-battery",
+};
+
+static struct gpio_charger_platform_data poodle_power_data = {
+	.name			= "charger",
+	.type			= POWER_SUPPLY_TYPE_MAINS,
+	.gpio			= POODLE_GPIO_AC_IN,
+	.supplied_to		= poodle_ac_supplied_to,
+	.num_supplicants	= ARRAY_SIZE(poodle_ac_supplied_to),
+};
+
+static struct platform_device poodle_power_device = {
+	.name			= "gpio-charger",
+	.id			= -1,
+	.dev.platform_data	= &poodle_power_data,
+};
+
 static struct platform_device *devices[] __initdata = {
 	&poodle_locomo_device,
 	&poodle_scoop_device,
 	&poodle_audio_device,
 	&sharpsl_nand_device,
 	&sharpsl_rom_device,
+	&poodle_power_device,
 };
 
 static struct i2c_board_info __initdata poodle_i2c_devices[] = {