All of lore.kernel.org
 help / color / mirror / Atom feed
From: Nishanth Menon <nm@ti.com>
To: Kishon Vijay Abraham I <kishon@ti.com>
Cc: mark.rutland@arm.com, devicetree@vger.kernel.org,
	linux@arm.linux.org.uk, pawel.moll@arm.com,
	ijc+devicetree@hellion.org.uk, tony@atomide.com,
	swarren@wwwdotorg.org, linux-kernel@vger.kernel.org,
	rob.herring@calxeda.com, bcousson@baylibre.com,
	linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 1/2] ARM: dts: omap5-uevm: remove always_on, boot_on from smps10_out1
Date: Thu, 10 Oct 2013 09:19:50 -0500	[thread overview]
Message-ID: <20131010141949.GA13277@kahuna> (raw)
In-Reply-To: <1381402195-29257-1-git-send-email-kishon@ti.com>

On 16:19-20131010, Kishon Vijay Abraham I wrote:
> smps10 should be enabled only in the case of host mode. So stop
> doing always_on, boot_on from smps10_out1. The driver will enable it in host
> mode.
> 
> Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
> ---
>  arch/arm/boot/dts/omap5-uevm.dts |    2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/omap5-uevm.dts b/arch/arm/boot/dts/omap5-uevm.dts
> index d784b3a..e06a04a 100644
> --- a/arch/arm/boot/dts/omap5-uevm.dts
> +++ b/arch/arm/boot/dts/omap5-uevm.dts
> @@ -334,8 +334,6 @@
>  					regulator-name = "smps10_out1";
>  					regulator-min-microvolt = <5000000>;
>  					regulator-max-microvolt = <5000000>;
> -					regulator-always-on;
> -					regulator-boot-on;
>  				};
>  
>  				ldo1_reg: ldo1 {
> -- 
Always good to get rid of regulator-always-on. Thanks for it.

regulator-boot-on indicates that PMIC enables it by default as part of
OTP or some internal behavior -> Looking at the measurements done on
uEVM and OTP information -> regulator-boot-on should be kept here.

Hence Nak.
-- 
Regards,
Nishanth Menon

WARNING: multiple messages have this Message-ID (diff)
From: nm@ti.com (Nishanth Menon)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/2] ARM: dts: omap5-uevm: remove always_on, boot_on from smps10_out1
Date: Thu, 10 Oct 2013 09:19:50 -0500	[thread overview]
Message-ID: <20131010141949.GA13277@kahuna> (raw)
In-Reply-To: <1381402195-29257-1-git-send-email-kishon@ti.com>

On 16:19-20131010, Kishon Vijay Abraham I wrote:
> smps10 should be enabled only in the case of host mode. So stop
> doing always_on, boot_on from smps10_out1. The driver will enable it in host
> mode.
> 
> Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
> ---
>  arch/arm/boot/dts/omap5-uevm.dts |    2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/omap5-uevm.dts b/arch/arm/boot/dts/omap5-uevm.dts
> index d784b3a..e06a04a 100644
> --- a/arch/arm/boot/dts/omap5-uevm.dts
> +++ b/arch/arm/boot/dts/omap5-uevm.dts
> @@ -334,8 +334,6 @@
>  					regulator-name = "smps10_out1";
>  					regulator-min-microvolt = <5000000>;
>  					regulator-max-microvolt = <5000000>;
> -					regulator-always-on;
> -					regulator-boot-on;
>  				};
>  
>  				ldo1_reg: ldo1 {
> -- 
Always good to get rid of regulator-always-on. Thanks for it.

regulator-boot-on indicates that PMIC enables it by default as part of
OTP or some internal behavior -> Looking at the measurements done on
uEVM and OTP information -> regulator-boot-on should be kept here.

Hence Nak.
-- 
Regards,
Nishanth Menon

WARNING: multiple messages have this Message-ID (diff)
From: Nishanth Menon <nm@ti.com>
To: Kishon Vijay Abraham I <kishon@ti.com>
Cc: <bcousson@baylibre.com>, <mark.rutland@arm.com>,
	<devicetree@vger.kernel.org>, <linux@arm.linux.org.uk>,
	<pawel.moll@arm.com>, <ijc+devicetree@hellion.org.uk>,
	<tony@atomide.com>, <swarren@wwwdotorg.org>,
	<linux-kernel@vger.kernel.org>, <rob.herring@calxeda.com>,
	<linux-omap@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 1/2] ARM: dts: omap5-uevm: remove always_on, boot_on from smps10_out1
Date: Thu, 10 Oct 2013 09:19:50 -0500	[thread overview]
Message-ID: <20131010141949.GA13277@kahuna> (raw)
In-Reply-To: <1381402195-29257-1-git-send-email-kishon@ti.com>

On 16:19-20131010, Kishon Vijay Abraham I wrote:
> smps10 should be enabled only in the case of host mode. So stop
> doing always_on, boot_on from smps10_out1. The driver will enable it in host
> mode.
> 
> Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
> ---
>  arch/arm/boot/dts/omap5-uevm.dts |    2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/omap5-uevm.dts b/arch/arm/boot/dts/omap5-uevm.dts
> index d784b3a..e06a04a 100644
> --- a/arch/arm/boot/dts/omap5-uevm.dts
> +++ b/arch/arm/boot/dts/omap5-uevm.dts
> @@ -334,8 +334,6 @@
>  					regulator-name = "smps10_out1";
>  					regulator-min-microvolt = <5000000>;
>  					regulator-max-microvolt = <5000000>;
> -					regulator-always-on;
> -					regulator-boot-on;
>  				};
>  
>  				ldo1_reg: ldo1 {
> -- 
Always good to get rid of regulator-always-on. Thanks for it.

regulator-boot-on indicates that PMIC enables it by default as part of
OTP or some internal behavior -> Looking at the measurements done on
uEVM and OTP information -> regulator-boot-on should be kept here.

Hence Nak.
-- 
Regards,
Nishanth Menon

  parent reply	other threads:[~2013-10-10 14:19 UTC|newest]

Thread overview: 72+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-10-10 10:49 [PATCH 1/2] ARM: dts: omap5-uevm: remove always_on, boot_on from smps10_out1 Kishon Vijay Abraham I
2013-10-10 10:49 ` Kishon Vijay Abraham I
2013-10-10 10:49 ` Kishon Vijay Abraham I
     [not found] ` <1381402195-29257-1-git-send-email-kishon-l0cyMroinI0@public.gmane.org>
2013-10-10 10:49   ` [PATCH 2/2] ARM: dts: omap5: Add dr_mode for dwc3 Kishon Vijay Abraham I
2013-10-10 10:49     ` Kishon Vijay Abraham I
2013-10-10 10:49     ` Kishon Vijay Abraham I
2013-10-11 14:46     ` Roger Quadros
2013-10-11 14:46       ` Roger Quadros
2013-10-11 14:46       ` Roger Quadros
2013-10-15  5:49       ` [PATCH v2 " Kishon Vijay Abraham I
2013-10-15  5:49         ` Kishon Vijay Abraham I
2013-10-15  5:49         ` Kishon Vijay Abraham I
2013-10-16 13:17         ` Kishon Vijay Abraham I
2013-10-16 13:17           ` Kishon Vijay Abraham I
2013-10-16 13:17           ` Kishon Vijay Abraham I
2013-10-17 20:08           ` Benoit Cousson
2013-10-17 20:08             ` Benoit Cousson
     [not found]             ` <526043BA.5040100-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
2013-10-21 10:03               ` Kishon Vijay Abraham I
2013-10-21 10:03                 ` Kishon Vijay Abraham I
2013-10-21 10:03                 ` Kishon Vijay Abraham I
2014-04-16 16:16       ` [PATCH " Felipe Balbi
2014-04-16 16:16         ` Felipe Balbi
2014-04-16 16:16         ` Felipe Balbi
2014-04-16 16:22         ` Felipe Balbi
2014-04-16 16:22           ` Felipe Balbi
2014-04-16 16:22           ` Felipe Balbi
2014-09-08 11:34         ` Dmitry Lifshitz
2014-09-08 11:34           ` Dmitry Lifshitz
     [not found]           ` <540D9449.3090008-UTxiZqZC01RS1MOuV/RT9w@public.gmane.org>
2014-09-08 11:39             ` Roger Quadros
2014-09-08 11:39               ` Roger Quadros
2014-09-08 11:39               ` Roger Quadros
2014-09-08 12:32               ` Dmitry Lifshitz
2014-09-08 12:32                 ` Dmitry Lifshitz
     [not found]                 ` <540DA1E5.1060701-UTxiZqZC01RS1MOuV/RT9w@public.gmane.org>
2014-09-08 13:25                   ` Felipe Balbi
2014-09-08 13:25                     ` Felipe Balbi
2014-09-08 13:25                     ` Felipe Balbi
2014-09-08 13:24           ` Felipe Balbi
2014-09-08 13:24             ` Felipe Balbi
2014-09-08 13:24             ` Felipe Balbi
     [not found]             ` <20140908132403.GA22409-HgARHv6XitL9zxVx7UNMDg@public.gmane.org>
2014-09-10 12:03               ` Dmitry Lifshitz
2014-09-10 12:03                 ` Dmitry Lifshitz
2014-09-10 12:03                 ` Dmitry Lifshitz
2014-09-11 17:14                 ` Felipe Balbi
2014-09-11 17:14                   ` Felipe Balbi
2014-09-11 17:14                   ` Felipe Balbi
2013-10-10 14:19 ` Nishanth Menon [this message]
2013-10-10 14:19   ` [PATCH 1/2] ARM: dts: omap5-uevm: remove always_on, boot_on from smps10_out1 Nishanth Menon
2013-10-10 14:19   ` Nishanth Menon
2013-10-11  6:13   ` Kishon Vijay Abraham I
2013-10-11  6:13     ` Kishon Vijay Abraham I
2013-10-11  6:13     ` Kishon Vijay Abraham I
2013-10-11  6:30     ` Nishanth Menon
2013-10-11  6:30       ` Nishanth Menon
2013-10-11  6:53       ` Kishon Vijay Abraham I
2013-10-11  6:53         ` Kishon Vijay Abraham I
2013-10-11  6:54         ` Kishon Vijay Abraham I
2013-10-11  6:54           ` Kishon Vijay Abraham I
     [not found]           ` <5257A0C0.8000701-l0cyMroinI0@public.gmane.org>
2013-10-11  7:27             ` Nishanth Menon
2013-10-11  7:27               ` Nishanth Menon
2013-10-11  7:27               ` Nishanth Menon
2013-10-16 13:17 ` Kishon Vijay Abraham I
2013-10-16 13:17   ` Kishon Vijay Abraham I
2013-10-16 13:17   ` Kishon Vijay Abraham I
     [not found]   ` <525E9206.9010501-l0cyMroinI0@public.gmane.org>
2013-10-16 13:27     ` Nishanth Menon
2013-10-16 13:27       ` Nishanth Menon
2013-10-16 13:27       ` Nishanth Menon
     [not found]       ` <525E9434.1090006-l0cyMroinI0@public.gmane.org>
2013-10-17 20:09         ` Benoit Cousson
2013-10-17 20:09           ` Benoit Cousson
2013-10-17 20:09           ` Benoit Cousson
2013-10-21 10:03           ` Kishon Vijay Abraham I
2013-10-21 10:03             ` Kishon Vijay Abraham I
2013-10-21 10:03             ` Kishon Vijay Abraham I

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20131010141949.GA13277@kahuna \
    --to=nm@ti.com \
    --cc=bcousson@baylibre.com \
    --cc=devicetree@vger.kernel.org \
    --cc=ijc+devicetree@hellion.org.uk \
    --cc=kishon@ti.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=mark.rutland@arm.com \
    --cc=pawel.moll@arm.com \
    --cc=rob.herring@calxeda.com \
    --cc=swarren@wwwdotorg.org \
    --cc=tony@atomide.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.