linux-kernel - Re: [PATCH] arm: omap5: dts: split SMPS10 dt node

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <520A32CF.1010701@baylibre.com>
Date:	Tue, 13 Aug 2013 15:21:19 +0200
From:	Benoit Cousson <bcousson@...libre.com>
To:	Kishon Vijay Abraham I <kishon@...com>
CC:	benoit.cousson@...aro.org, tony@...mide.com,
	rob.herring@...xeda.com, pawel.moll@....com, mark.rutland@....com,
	swarren@...dotorg.org, ian.campbell@...rix.com,
	linux-omap@...r.kernel.org, devicetree@...r.kernel.org,
	linux-arm-kernel@...ts.infradead.org, linux@....linux.org.uk,
	linux-kernel@...r.kernel.org, balbi@...com
Subject: Re: [PATCH] arm: omap5: dts: split SMPS10 dt node

Hi Kishon,

On 12/08/2013 11:37, Kishon Vijay Abraham I wrote:
> SMPS10 has two outputs OUT1 and OUT2. Hence SMPS10 is modeled as
> two regulators. The dt node is split to reflect it.

Mmm, I'm curious. How is it supposed to work?

Do you have dedicated control on each output?

Otherwise, it should be seen as one output with two potential consumers.

Regards,
Benoit

>
> Signed-off-by: Kishon Vijay Abraham I <kishon@...com>
> ---
>   arch/arm/boot/dts/omap5-uevm.dts |   13 +++++++++++--
>   1 file changed, 11 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/boot/dts/omap5-uevm.dts b/arch/arm/boot/dts/omap5-uevm.dts
> index 65d7b60..05b9b12 100644
> --- a/arch/arm/boot/dts/omap5-uevm.dts
> +++ b/arch/arm/boot/dts/omap5-uevm.dts
> @@ -334,9 +334,18 @@
>   					ti,smps-range = <0x80>;
>   				};
>
> -				smps10_reg: smps10 {
> +				smps10_out2_reg: smps10_out2 {
>   					/* VBUS_5V_OTG */
> -					regulator-name = "smps10";
> +					regulator-name = "smps10_out2";
> +					regulator-min-microvolt = <5000000>;
> +					regulator-max-microvolt = <5000000>;
> +					regulator-always-on;
> +					regulator-boot-on;
> +				};
> +
> +				smps10_out1_reg: smps10_out1 {
> +					/* VBUS_5V_OTG */
> +					regulator-name = "smps10_out1";
>   					regulator-min-microvolt = <5000000>;
>   					regulator-max-microvolt = <5000000>;
>   					regulator-always-on;
>

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