diff mbox

[3/3] mfd: Add battery charger as subdevice to the tps65217.

Message ID 1441699779-22189-4-git-send-email-enric.balletbo@collabora.com (mailing list archive)
State New, archived
Headers show

Commit Message

Enric Balletbo Serra Sept. 8, 2015, 8:09 a.m. UTC
Add tps65217 battery charger subdevice.

Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
---
 drivers/mfd/tps65217.c | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Lee Jones Sept. 20, 2015, 4:19 a.m. UTC | #1
On Tue, 08 Sep 2015, Enric Balletbo i Serra wrote:

> Add tps65217 battery charger subdevice.
> 
> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
> ---
>  drivers/mfd/tps65217.c | 4 ++++
>  1 file changed, 4 insertions(+)

Applied, thanks.

> diff --git a/drivers/mfd/tps65217.c b/drivers/mfd/tps65217.c
> index 55add04..d32b5442 100644
> --- a/drivers/mfd/tps65217.c
> +++ b/drivers/mfd/tps65217.c
> @@ -39,6 +39,10 @@ static const struct mfd_cell tps65217s[] = {
>  		.name = "tps65217-bl",
>  		.of_compatible = "ti,tps65217-bl",
>  	},
> +	{
> +		.name = "tps65217-charger",
> +		.of_compatible = "ti,tps65217-charger",
> +	},
>  };
>  
>  /**
Enric Balletbo Serra Sept. 22, 2015, 7:43 a.m. UTC | #2
2015-09-20 6:19 GMT+02:00 Lee Jones <lee.jones@linaro.org>:
> On Tue, 08 Sep 2015, Enric Balletbo i Serra wrote:
>
>> Add tps65217 battery charger subdevice.
>>
>> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
>> ---
>>  drivers/mfd/tps65217.c | 4 ++++
>>  1 file changed, 4 insertions(+)
>
> Applied, thanks.
>

Many thanks, any feedback about ?

[PATCH 1/3] devicetree: Add TPS65217 charger binding
[PATCH 2/3] power_supply: Add support for tps65217-charger.

Or should go through another tree ?



>> diff --git a/drivers/mfd/tps65217.c b/drivers/mfd/tps65217.c
>> index 55add04..d32b5442 100644
>> --- a/drivers/mfd/tps65217.c
>> +++ b/drivers/mfd/tps65217.c
>> @@ -39,6 +39,10 @@ static const struct mfd_cell tps65217s[] = {
>>               .name = "tps65217-bl",
>>               .of_compatible = "ti,tps65217-bl",
>>       },
>> +     {
>> +             .name = "tps65217-charger",
>> +             .of_compatible = "ti,tps65217-charger",
>> +     },
>>  };
>>
>>  /**
>
> --
> Lee Jones
> Linaro STMicroelectronics Landing Team Lead
> Linaro.org ? Open source software for ARM SoCs
> Follow Linaro: Facebook | Twitter | Blog
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Sebastian Reichel Sept. 22, 2015, 11:37 a.m. UTC | #3
Hi,

On Tue, Sep 22, 2015 at 09:43:50AM +0200, Enric Balletbo Serra wrote:
> 2015-09-20 6:19 GMT+02:00 Lee Jones <lee.jones@linaro.org>:
> > On Tue, 08 Sep 2015, Enric Balletbo i Serra wrote:
> >
> >> Add tps65217 battery charger subdevice.
> >>
> >> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
> >> ---
> >>  drivers/mfd/tps65217.c | 4 ++++
> >>  1 file changed, 4 insertions(+)
> >
> > Applied, thanks.
> >
> 
> Many thanks, any feedback about ?
> 
> [PATCH 1/3] devicetree: Add TPS65217 charger binding
> [PATCH 2/3] power_supply: Add support for tps65217-charger.
> 
> Or should go through another tree ?

Those go through my tree. I'm currently catching up with my backlog.
I will have a look at these later.

-- Sebastian
diff mbox

Patch

diff --git a/drivers/mfd/tps65217.c b/drivers/mfd/tps65217.c
index 55add04..d32b5442 100644
--- a/drivers/mfd/tps65217.c
+++ b/drivers/mfd/tps65217.c
@@ -39,6 +39,10 @@  static const struct mfd_cell tps65217s[] = {
 		.name = "tps65217-bl",
 		.of_compatible = "ti,tps65217-bl",
 	},
+	{
+		.name = "tps65217-charger",
+		.of_compatible = "ti,tps65217-charger",
+	},
 };
 
 /**