Mailing List Archive: Linux: Kernel

[PATCH v2 0/2] mfd: dt: add power off support for Tegra20/Tegra30

Index | Next | Previous | View Threaded

bilhuang at nvidia

Aug 19, 2012, 6:07 PM

Post #1 of 8 (95 views)
Permalink

[PATCH v2 0/2] mfd: dt: add power off support for Tegra20/Tegra30

This patch series add new property into regulator DT for telling whether or not
to hook pmic's power off routine to system call "pm_power_off".

Patch 1 add power off support for Tegra20 boards using TPS6586x
Patch 2 add power off support for Tegra30 boards using TPS65910

Verified on Seaboard (Tegra20) and Cardhu (Tegra30)

V2:
* Take multiple pmic instances into consideration while assigning global variables
as per suggestion from Thierry Reding <thierry.reding [at] avionic-design>

V1:
* Based on master branch of sameo/mfd-2.6.git

Bill Huang (2):
mfd: dt: tps6586x: Add power off control
mfd: dt: tps65910: add power off control

Documentation/devicetree/bindings/mfd/tps65910.txt | 4 +++
.../devicetree/bindings/regulator/tps6586x.txt | 6 +++++
drivers/mfd/tps6586x.c | 19 +++++++++++++++++
drivers/mfd/tps65910.c | 22 ++++++++++++++++++++
include/linux/mfd/tps6586x.h | 1 +
include/linux/mfd/tps65910.h | 3 ++
6 files changed, 55 insertions(+), 0 deletions(-)

--
1.7.4.1

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

bilhuang at nvidia

Sep 11, 2012, 3:25 AM

Post #2 of 8 (71 views)
Permalink

RE: [PATCH v2 0/2] mfd: dt: add power off support for Tegra20/Tegra30 [In reply to]

Hi all,

Could somebody review this?

Thanks,
Bill

>
> This patch series add new property into regulator DT for telling whether or not to hook pmic's power off routine to system call
> "pm_power_off".
>
> Patch 1 add power off support for Tegra20 boards using TPS6586x Patch 2 add power off support for Tegra30 boards using TPS65910
>
> Verified on Seaboard (Tegra20) and Cardhu (Tegra30)
>
> V2:
> * Take multiple pmic instances into consideration while assigning global variables
> as per suggestion from Thierry Reding <thierry.reding [at] avionic-design>
>
> V1:
> * Based on master branch of sameo/mfd-2.6.git
>
> Bill Huang (2):
> mfd: dt: tps6586x: Add power off control
> mfd: dt: tps65910: add power off control
>
> Documentation/devicetree/bindings/mfd/tps65910.txt | 4 +++
> .../devicetree/bindings/regulator/tps6586x.txt | 6 +++++
> drivers/mfd/tps6586x.c | 19 +++++++++++++++++
> drivers/mfd/tps65910.c | 22 ++++++++++++++++++++
> include/linux/mfd/tps6586x.h | 1 +
> include/linux/mfd/tps65910.h | 3 ++
> 6 files changed, 55 insertions(+), 0 deletions(-)
>
> --
> 1.7.4.1
nvpublic
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

thierry.reding at avionic-design

Sep 11, 2012, 3:46 AM

Post #3 of 8 (73 views)
Permalink

Re: [PATCH v2 0/2] mfd: dt: add power off support for Tegra20/Tegra30 [In reply to]

On Tue, Sep 11, 2012 at 06:25:14PM +0800, Bill Huang wrote:
> Hi all,
>
> Could somebody review this?

Given that I haven't been able to test yet (due to time constraints)
with PM_SLEEP_SMP enabled, I don't want to give you a Tested-by, but the
code looks okay to me, so for both patches:

Reviewed-by: Thierry Reding <thierry.reding [at] avionic-design>

swarren at wwwdotorg

Sep 11, 2012, 8:15 AM

Post #4 of 8 (71 views)
Permalink

Re: [PATCH v2 0/2] mfd: dt: add power off support for Tegra20/Tegra30 [In reply to]

On 09/11/2012 04:46 AM, Thierry Reding wrote:
> On Tue, Sep 11, 2012 at 06:25:14PM +0800, Bill Huang wrote:
>> Hi all,
>>
>> Could somebody review this?
>
> Given that I haven't been able to test yet (due to time
> constraints) with PM_SLEEP_SMP enabled, I don't want to give you a
> Tested-by, but the code looks okay to me, so for both patches:
>
> Reviewed-by: Thierry Reding <thierry.reding [at] avionic-design>

I have tested this with PM_SLEEP_SMP enabled, and it solved the
problem. I think I already gave my Tested-by, but if not:

Tested-by: Stephen Warren <swarren [at] wwwdotorg>

I hope that this gets applied to the MFD tree early enough (i.e.
within the next 2-3 days or so) that I can rely on the binding be
accepted, and hence apply patches to Tegra's device tree to enable
this functionality for 3.7.

Bill, given this was posted about 3 weeks ago, perhaps repost the
series in case Samuel doesn't have it any more, and hence can't apply it.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

sameo at linux

Sep 11, 2012, 9:07 AM

Post #5 of 8 (67 views)
Permalink

Re: [PATCH v2 0/2] mfd: dt: add power off support for Tegra20/Tegra30 [In reply to]

Hi Bill,

On Sun, Aug 19, 2012 at 06:07:54PM -0700, Bill Huang wrote:
> This patch series add new property into regulator DT for telling whether or not
> to hook pmic's power off routine to system call "pm_power_off".
>
> Patch 1 add power off support for Tegra20 boards using TPS6586x
> Patch 2 add power off support for Tegra30 boards using TPS65910
>
> Verified on Seaboard (Tegra20) and Cardhu (Tegra30)
>
> V2:
> * Take multiple pmic instances into consideration while assigning global variables
> as per suggestion from Thierry Reding <thierry.reding [at] avionic-design>
>
> V1:
> * Based on master branch of sameo/mfd-2.6.git
>
> Bill Huang (2):
> mfd: dt: tps6586x: Add power off control
> mfd: dt: tps65910: add power off control
>
> Documentation/devicetree/bindings/mfd/tps65910.txt | 4 +++
> .../devicetree/bindings/regulator/tps6586x.txt | 6 +++++
> drivers/mfd/tps6586x.c | 19 +++++++++++++++++
> drivers/mfd/tps65910.c | 22 ++++++++++++++++++++
> include/linux/mfd/tps6586x.h | 1 +
> include/linux/mfd/tps65910.h | 3 ++
> 6 files changed, 55 insertions(+), 0 deletions(-)
I applied those 2 patches to my for-next branch, thanks.

Cheers,
Samuel.

--
Intel Open Source Technology Centre
http://oss.intel.com/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

sameo at linux

Sep 11, 2012, 9:08 AM

Post #6 of 8 (65 views)
Permalink

Re: [PATCH v2 0/2] mfd: dt: add power off support for Tegra20/Tegra30 [In reply to]

Hi Stephen,

On Tue, Sep 11, 2012 at 09:15:07AM -0600, Stephen Warren wrote:
> On 09/11/2012 04:46 AM, Thierry Reding wrote:
> > On Tue, Sep 11, 2012 at 06:25:14PM +0800, Bill Huang wrote:
> >> Hi all,
> >>
> >> Could somebody review this?
> >
> > Given that I haven't been able to test yet (due to time
> > constraints) with PM_SLEEP_SMP enabled, I don't want to give you a
> > Tested-by, but the code looks okay to me, so for both patches:
> >
> > Reviewed-by: Thierry Reding <thierry.reding [at] avionic-design>
>
> I have tested this with PM_SLEEP_SMP enabled, and it solved the
> problem. I think I already gave my Tested-by, but if not:
>
> Tested-by: Stephen Warren <swarren [at] wwwdotorg>
>
> I hope that this gets applied to the MFD tree early enough (i.e.
> within the next 2-3 days or so) that I can rely on the binding be
> accepted, and hence apply patches to Tegra's device tree to enable
> this functionality for 3.7.
>
> Bill, given this was posted about 3 weeks ago, perhaps repost the
> series in case Samuel doesn't have it any more, and hence can't apply it.
No need for that. Sorry for the delay, I just applied and pushed those 2
patches.

Cheers,
Samuel.

--
Intel Open Source Technology Centre
http://oss.intel.com/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

swarren at wwwdotorg

Sep 11, 2012, 9:26 AM

Post #7 of 8 (66 views)
Permalink

Re: [PATCH v2 0/2] mfd: dt: add power off support for Tegra20/Tegra30 [In reply to]

On 09/11/2012 10:08 AM, Samuel Ortiz wrote:
> Hi Stephen,
>
> On Tue, Sep 11, 2012 at 09:15:07AM -0600, Stephen Warren wrote:
>> On 09/11/2012 04:46 AM, Thierry Reding wrote:
>>> On Tue, Sep 11, 2012 at 06:25:14PM +0800, Bill Huang wrote:
>>>> Hi all,
>>>>
>>>> Could somebody review this?
>>>
>>> Given that I haven't been able to test yet (due to time
>>> constraints) with PM_SLEEP_SMP enabled, I don't want to give you a
>>> Tested-by, but the code looks okay to me, so for both patches:
>>>
>>> Reviewed-by: Thierry Reding <thierry.reding [at] avionic-design>
>>
>> I have tested this with PM_SLEEP_SMP enabled, and it solved the
>> problem. I think I already gave my Tested-by, but if not:
>>
>> Tested-by: Stephen Warren <swarren [at] wwwdotorg>
>>
>> I hope that this gets applied to the MFD tree early enough (i.e.
>> within the next 2-3 days or so) that I can rely on the binding be
>> accepted, and hence apply patches to Tegra's device tree to enable
>> this functionality for 3.7.
>>
>> Bill, given this was posted about 3 weeks ago, perhaps repost the
>> series in case Samuel doesn't have it any more, and hence can't apply it.
>
> No need for that. Sorry for the delay, I just applied and pushed those 2
> patches.

Great! Thanks.

Do you have "[PATCH V4 REPOST] mfd: add MAX8907 core driver" on your
list too?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

sameo at linux

Sep 11, 2012, 11:56 AM

Post #8 of 8 (65 views)
Permalink

Re: [PATCH v2 0/2] mfd: dt: add power off support for Tegra20/Tegra30 [In reply to]

On Tue, Sep 11, 2012 at 10:26:55AM -0600, Stephen Warren wrote:
> On 09/11/2012 10:08 AM, Samuel Ortiz wrote:
> > Hi Stephen,
> >
> > On Tue, Sep 11, 2012 at 09:15:07AM -0600, Stephen Warren wrote:
> >> On 09/11/2012 04:46 AM, Thierry Reding wrote:
> >>> On Tue, Sep 11, 2012 at 06:25:14PM +0800, Bill Huang wrote:
> >>>> Hi all,
> >>>>
> >>>> Could somebody review this?
> >>>
> >>> Given that I haven't been able to test yet (due to time
> >>> constraints) with PM_SLEEP_SMP enabled, I don't want to give you a
> >>> Tested-by, but the code looks okay to me, so for both patches:
> >>>
> >>> Reviewed-by: Thierry Reding <thierry.reding [at] avionic-design>
> >>
> >> I have tested this with PM_SLEEP_SMP enabled, and it solved the
> >> problem. I think I already gave my Tested-by, but if not:
> >>
> >> Tested-by: Stephen Warren <swarren [at] wwwdotorg>
> >>
> >> I hope that this gets applied to the MFD tree early enough (i.e.
> >> within the next 2-3 days or so) that I can rely on the binding be
> >> accepted, and hence apply patches to Tegra's device tree to enable
> >> this functionality for 3.7.
> >>
> >> Bill, given this was posted about 3 weeks ago, perhaps repost the
> >> series in case Samuel doesn't have it any more, and hence can't apply it.
> >
> > No need for that. Sorry for the delay, I just applied and pushed those 2
> > patches.
>
> Great! Thanks.
>
> Do you have "[PATCH V4 REPOST] mfd: add MAX8907 core driver" on your
> list too?
Yes, I'll be chewing my MFD pending patches list in the next days, and this
one is on the list.

Cheers,
Samuel.

--
Intel Open Source Technology Centre
http://oss.intel.com/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

Index | Next | Previous | View Threaded

Interested in having your list archived? Contact Gossamer Threads