Re: [resend PATCH 1/3] pinctrl: dra7: dt-bindings: add pin off modes for dra7 SoC

2014-09-09 Thread Mugunthan V N
On Tuesday 09 September 2014 01:02 AM, Nishanth Menon wrote:
> A) There is no OFF mode on DRA7. in sleep state use mode 15 (gated).
> B) when using this for wakeup - use pinctrl wakeup handling to do the
> wakeup.

Will drop this patch and use mode 15 in sleep mode and submit v2 patch
series

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


Re: [resend PATCH 1/3] pinctrl: dra7: dt-bindings: add pin off modes for dra7 SoC

2014-09-09 Thread Mugunthan V N
On Tuesday 09 September 2014 01:02 AM, Nishanth Menon wrote:
 A) There is no OFF mode on DRA7. in sleep state use mode 15 (gated).
 B) when using this for wakeup - use pinctrl wakeup handling to do the
 wakeup.

Will drop this patch and use mode 15 in sleep mode and submit v2 patch
series

Regards
Mugunthan V N
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [resend PATCH 1/3] pinctrl: dra7: dt-bindings: add pin off modes for dra7 SoC

2014-09-08 Thread Nishanth Menon
On 00:49-20140909, Mugunthan V N wrote:
> Add pin off modes for dra7 SoC so that during module disable or suspend
> state it can help saving power
> 
> Signed-off-by: Mugunthan V N 
> ---
>  include/dt-bindings/pinctrl/dra.h | 8 
>  1 file changed, 8 insertions(+)
> 
> diff --git a/include/dt-bindings/pinctrl/dra.h 
> b/include/dt-bindings/pinctrl/dra.h
> index 002a285..a0ff2d0 100644
> --- a/include/dt-bindings/pinctrl/dra.h
> +++ b/include/dt-bindings/pinctrl/dra.h
> @@ -46,5 +46,13 @@
>  #define PIN_INPUT_PULLUP (PULL_ENA | INPUT_EN | PULL_UP)
>  #define PIN_INPUT_PULLDOWN   (PULL_ENA | INPUT_EN)
>  
> +/* Off mode states */
> +#define PIN_OFF_NONE 0
> +#define PIN_OFF_OUTPUT_HIGH  (OFF_EN | OFFOUT_EN | OFFOUT_VAL)
> +#define PIN_OFF_OUTPUT_LOW   (OFF_EN | OFFOUT_EN)
> +#define PIN_OFF_INPUT_PULLUP (OFF_EN | OFF_PULL_EN | OFF_PULL_UP)
> +#define PIN_OFF_INPUT_PULLDOWN   (OFF_EN | OFF_PULL_EN)
> +#define PIN_OFF_WAKEUPENABLE WAKEUP_EN
> +
>  #endif

NAK for OFF_NONE, OFF_xyz etc.

A) There is no OFF mode on DRA7. in sleep state use mode 15 (gated).
B) when using this for wakeup - use pinctrl wakeup handling to do the
wakeup.

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


Re: [resend PATCH 1/3] pinctrl: dra7: dt-bindings: add pin off modes for dra7 SoC

2014-09-08 Thread Nishanth Menon
On 00:49-20140909, Mugunthan V N wrote:
 Add pin off modes for dra7 SoC so that during module disable or suspend
 state it can help saving power
 
 Signed-off-by: Mugunthan V N mugunthan...@ti.com
 ---
  include/dt-bindings/pinctrl/dra.h | 8 
  1 file changed, 8 insertions(+)
 
 diff --git a/include/dt-bindings/pinctrl/dra.h 
 b/include/dt-bindings/pinctrl/dra.h
 index 002a285..a0ff2d0 100644
 --- a/include/dt-bindings/pinctrl/dra.h
 +++ b/include/dt-bindings/pinctrl/dra.h
 @@ -46,5 +46,13 @@
  #define PIN_INPUT_PULLUP (PULL_ENA | INPUT_EN | PULL_UP)
  #define PIN_INPUT_PULLDOWN   (PULL_ENA | INPUT_EN)
  
 +/* Off mode states */
 +#define PIN_OFF_NONE 0
 +#define PIN_OFF_OUTPUT_HIGH  (OFF_EN | OFFOUT_EN | OFFOUT_VAL)
 +#define PIN_OFF_OUTPUT_LOW   (OFF_EN | OFFOUT_EN)
 +#define PIN_OFF_INPUT_PULLUP (OFF_EN | OFF_PULL_EN | OFF_PULL_UP)
 +#define PIN_OFF_INPUT_PULLDOWN   (OFF_EN | OFF_PULL_EN)
 +#define PIN_OFF_WAKEUPENABLE WAKEUP_EN
 +
  #endif

NAK for OFF_NONE, OFF_xyz etc.

A) There is no OFF mode on DRA7. in sleep state use mode 15 (gated).
B) when using this for wakeup - use pinctrl wakeup handling to do the
wakeup.

Regards,
Nishanth Menon
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/