Re: [PATCH 2/2] mfd: twl4030-power: use 'ti,system-power-controller' as alternative way to support system power off

2014-09-17 Thread Nishanth Menon
On 16:05-20140916, Lee Jones wrote:
 On Mon, 08 Sep 2014, Tony Lindgren wrote:
 
  * Nishanth Menon n...@ti.com [140903 12:07]:
   ti,system-power-controller is more or less the standard way of
   indicating that the PMIC is the system wide power controller and hence
   may be used to switch off the system. Almost ALL TI PMIC drivers and
   many Maxim PMIC drivers follow the same style.
   
   So support 'ti,system-power-controller' in addition to the usual
   'ti,use_poweroff' to indicate that the PMIC instance has control for
   switching off the system.
   
   Signed-off-by: Nishanth Menon n...@ti.com
  
  Acked-by: Tony Lindgren t...@atomide.com
 
 I assume you're going to resend this with the document modifications?
 When you do, don't forget to apply Tony's Ack, as it will ensure a
 faster merge.
 

Thanks for the reminder, This did indeed slip through the cracks.
Posting updated rev in a few mins..
-- 
Regards,
Nishanth Menon
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 2/2] mfd: twl4030-power: use 'ti,system-power-controller' as alternative way to support system power off

2014-09-16 Thread Lee Jones
On Mon, 08 Sep 2014, Tony Lindgren wrote:

 * Nishanth Menon n...@ti.com [140903 12:07]:
  ti,system-power-controller is more or less the standard way of
  indicating that the PMIC is the system wide power controller and hence
  may be used to switch off the system. Almost ALL TI PMIC drivers and
  many Maxim PMIC drivers follow the same style.
  
  So support 'ti,system-power-controller' in addition to the usual
  'ti,use_poweroff' to indicate that the PMIC instance has control for
  switching off the system.
  
  Signed-off-by: Nishanth Menon n...@ti.com
 
 Acked-by: Tony Lindgren t...@atomide.com

I assume you're going to resend this with the document modifications?
When you do, don't forget to apply Tony's Ack, as it will ensure a
faster merge.

  ---
   drivers/mfd/twl4030-power.c |3 +++
   1 file changed, 3 insertions(+)
  
  diff --git a/drivers/mfd/twl4030-power.c b/drivers/mfd/twl4030-power.c
  index 3bc969a..1c129ba 100644
  --- a/drivers/mfd/twl4030-power.c
  +++ b/drivers/mfd/twl4030-power.c
  @@ -627,6 +627,9 @@ static bool twl4030_power_use_poweroff(const struct 
  twl4030_power_data *pdata,
  if (pdata  pdata-use_poweroff)
  return true;
   
  +   if (of_property_read_bool(node, ti,system-power-controller))
  +   return true;
  +
  if (of_property_read_bool(node, ti,use_poweroff))
  return true;
   

-- 
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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 2/2] mfd: twl4030-power: use 'ti,system-power-controller' as alternative way to support system power off

2014-09-08 Thread Tony Lindgren
* Nishanth Menon n...@ti.com [140903 12:07]:
 ti,system-power-controller is more or less the standard way of
 indicating that the PMIC is the system wide power controller and hence
 may be used to switch off the system. Almost ALL TI PMIC drivers and
 many Maxim PMIC drivers follow the same style.
 
 So support 'ti,system-power-controller' in addition to the usual
 'ti,use_poweroff' to indicate that the PMIC instance has control for
 switching off the system.
 
 Signed-off-by: Nishanth Menon n...@ti.com

Acked-by: Tony Lindgren t...@atomide.com

 ---
  drivers/mfd/twl4030-power.c |3 +++
  1 file changed, 3 insertions(+)
 
 diff --git a/drivers/mfd/twl4030-power.c b/drivers/mfd/twl4030-power.c
 index 3bc969a..1c129ba 100644
 --- a/drivers/mfd/twl4030-power.c
 +++ b/drivers/mfd/twl4030-power.c
 @@ -627,6 +627,9 @@ static bool twl4030_power_use_poweroff(const struct 
 twl4030_power_data *pdata,
   if (pdata  pdata-use_poweroff)
   return true;
  
 + if (of_property_read_bool(node, ti,system-power-controller))
 + return true;
 +
   if (of_property_read_bool(node, ti,use_poweroff))
   return true;
  
 -- 
 1.7.9.5
 
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH 2/2] mfd: twl4030-power: use 'ti,system-power-controller' as alternative way to support system power off

2014-09-03 Thread Nishanth Menon
ti,system-power-controller is more or less the standard way of
indicating that the PMIC is the system wide power controller and hence
may be used to switch off the system. Almost ALL TI PMIC drivers and
many Maxim PMIC drivers follow the same style.

So support 'ti,system-power-controller' in addition to the usual
'ti,use_poweroff' to indicate that the PMIC instance has control for
switching off the system.

Signed-off-by: Nishanth Menon n...@ti.com
---
 drivers/mfd/twl4030-power.c |3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/mfd/twl4030-power.c b/drivers/mfd/twl4030-power.c
index 3bc969a..1c129ba 100644
--- a/drivers/mfd/twl4030-power.c
+++ b/drivers/mfd/twl4030-power.c
@@ -627,6 +627,9 @@ static bool twl4030_power_use_poweroff(const struct 
twl4030_power_data *pdata,
if (pdata  pdata-use_poweroff)
return true;
 
+   if (of_property_read_bool(node, ti,system-power-controller))
+   return true;
+
if (of_property_read_bool(node, ti,use_poweroff))
return true;
 
-- 
1.7.9.5

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html