devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Krzysztof Kozlowski <k.kozlowski@samsung.com>
To: Anand Moon <linux.amoon@gmail.com>
Cc: Krzysztof Kozlowski <k.kozlowski@samsung.com>,
	Dongjin Kim <tobetter@gmail.com>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"linux-samsung-soc@vger.kernel.org"
	<linux-samsung-soc@vger.kernel.org>,
	Russell King <linux@arm.linux.org.uk>,
	Lukasz Majewski <l.majewski@samsung.com>,
	Eduardo Valentin <edubezval@gmail.com>,
	Sjoerd Simons <sjoerd.simons@collabora.co.uk>,
	Kukjin Kim <kgene@kernel.org>, Guenter Roeck <linux@roeck-us.net>,
	Markus Reichl <m.reichl@fivetechno.de>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCHv5 2/5] ARM: dts: Odroid-XU3 Enable TMU at Exynos5422 base
Date: Sun, 24 May 2015 13:33:38 +0900	[thread overview]
Message-ID: <CAJKOXPcQBVaaS+32GYpyQHQ3m_EZYqu0noXtpp_FVBEaGyUgGw@mail.gmail.com> (raw)
In-Reply-To: <CANAwSgQzYvmquo9fcUPLK5XHqo1G=HYHc-YuhUvtw2yYaDi9GA@mail.gmail.com>

2015-05-24 2:35 GMT+09:00 Anand Moon <linux.amoon@gmail.com>:
> Hi Krzysztof,
>
> I did some debugging on the this using powerdebug utility.
> Using  LDO7 to control TMU seams to me correct option compared to BUCK3.
>
> Setting LDO7 to control TMU I observed  following output on powerdebug.
> I observed that the power drawn by the board on Odroid show is much
> less compare to BUCK3 on the same setup.
> I can see the HDMI output on Odroid-V2 display screen.
> I have done some stress testing on this configuration and It worked correctly.
>
> Powerdebug output using LDO7
> ---------------------------------------------------------------------------------------------------------------
> PowerDebug 0.7.3     Clocks  Regulators  Sensors  Gpio
> Name        Status      State       Type        Users       Microvolts
>  Min u-volts Max u-volts
> phy                                             0           0
>  0           0
> phy                                             0           0
>  0           0
> vdd_ldo1                enabled     voltage     0           1000000
>  1000000     1000000
> LDO1                                            0           0
>  0           0
> LDO2                    enabled     voltage     0           1800000
>  0           0
> vdd_ldo3                enabled     voltage     0           1800000
>  1800000     1800000
> LDO3                                            0           0
>  0           0
> LDO4                    enabled     voltage     0           1800000
>  0           0
> vdd_ldo5                enabled     voltage     0           1800000
>  1800000     1800000
> LDO5                                            0           0
>  0           0
> vdd_ldo6                enabled     voltage     0           1000000
>  1000000     1000000
> hdmi        okay                                0           0
>  0           0
> hdmi        okay                                0           0
>  0           0
> LDO6                                            0           0
>  0           0
> vdd_ldo7                enabled     voltage     0           1800000
>  1800000     1800000
> hdmi        okay                                0           0
>  0           0
> tmu         okay                                0           0
>  0           0
> tmu         okay                                0           0
>  0           0
> tmu         okay                                0           0
>  0           0
> tmu         okay                                0           0
>  0           0
> tmu         okay                                0           0
>  0           0
> LDO7                                            0           0
>  0           0
> vdd_ldo8                enabled     voltage     0           1800000
>  1800000     1800000
> LDO8                                            0           0
>  0           0
> vdd_ldo9                enabled     voltage     0           3000000
>  3000000     3000000
> LDO9                                            0           0
>  0           0
>
> Powerdebug output using BUCK3.
> --------------------------------------------------------------------------------------------------------------------------
> PowerDebug 0.7.3     Clocks  Regulators  Sensors  Gpio
> Name        Status      State       Type        Users       Microvolts
>  Min u-volts Max u-volts
> LDO21                   disabled    voltage     0           1800000
>  0           0
> LDO22                   disabled    voltage     0           1200000
>  0           0
> LDO23                   enabled     voltage     0           1100000
>  0           0
> tsp_io                  enabled     voltage     0           2800000
>  2800000     2800000
> LDO24                                           0           0
>  0           0
> LDO25                   disabled    voltage     0           1800000
>  0           0
> vdd_ldo26               enabled     voltage     0           3000000
>  3000000     3000000
> LDO26                                           0           0
>  0           0
> LDO27                   enabled     voltage     0           1000000
>  0           0
> LDO28                   disabled    voltage     0           3300000
>  0           0
> LDO29                   disabled    voltage     0           1800000
>  0           0
> LDO30                   disabled    voltage     0           1800000
>  0           0
> LDO31                   disabled    voltage     0           1800000
>  0           0
> LDO32                   disabled    voltage     0           1800000
>  0           0
> LDO33                   disabled    voltage     0           1800000
>  0           0
> LDO34                   disabled    voltage     0           3000000
>  0           0
> LDO35                   disabled    voltage     0           1600000
>  0           0
> LDO36                   disabled    voltage     0           1800000
>  0           0
> LDO37                   disabled    voltage     0           1800000
>  0           0
> LDO38                   disabled    voltage     0           2800000
>  0           0
> vdd_mif                 enabled     voltage     0           1100000
>  800000      1300000
> BUCK1                                           0           0
>  0           0
> vdd_arm                 enabled     voltage     0           1000000
>  800000      1500000
> BUCK2                                           0           0
>  0           0
> vdd_int                 enabled     voltage     0           1000000
>  800000      1400000
> tmu         okay                                0           0
>  0           0
> tmu         okay                                0           0
>  0           0
> tmu         okay                                0           0
>  0           0
> tmu         okay                                0           0
>  0           0
> tmu         okay                                0           0
>  0           0
> BUCK3                                           0           0
>  0           0
> vdd_g3d                 enabled     voltage     0           1000000
>  800000      1400000
> BUCK4                                           0           0
>  0           0
> vdd_mem                 enabled     voltage     0           1200000
>  800000      1400000
> BUCK5                                           0           0
>  0           0
> vdd_kfc                 enabled     voltage     0           1025000
>  800000      1500000
>
> So I would like to go with LDO7.
>
> Please share your thoughts.

I don't understand how grepping through /sys/class/regulator helps in
making this decision. I just don't get it. Can you please explain it
to me?

Best regards,
Krzysztof

  reply	other threads:[~2015-05-24  4:33 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-05-13  6:36 v5 Exynos5422 odroidxu3 pwm-fan control using thermal sensors Anand Moon
2015-05-13  6:36 ` [PATCHv5 1/5] ARM: dts: exynos5422-odroidxu3 Add pwm-fan node to the Odroid-XU3 board Anand Moon
2015-05-13  7:11   ` Krzysztof Kozlowski
     [not found]     ` <CAJKOXPefJwqvfZPDovQ6EjEOBfuJkidgeYDF_=KC9O+=2=_KoQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-05-13  7:46       ` Anand Moon
2015-06-14  5:31         ` Krzysztof Kozlowski
2015-06-14  5:35           ` Anand Moon
2015-05-13  8:17     ` [PATCHv5 1/5] ARM: dts: exynos5422-odroidxu3 Add pwm-fan node tothe " Kukjin Kim
2015-05-13  6:36 ` [PATCHv5 2/5] ARM: dts: Odroid-XU3 Enable TMU at Exynos5422 base Anand Moon
     [not found]   ` <1431499020-3912-3-git-send-email-linux.amoon-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2015-05-13  7:21     ` Krzysztof Kozlowski
2015-05-13  8:21       ` Anand Moon
2015-05-13  8:32         ` Krzysztof Kozlowski
     [not found]           ` <CAJKOXPfC9nGEsYJ3qJ6m_iMsB=D8_6sOEpLQzEvmLry5zS5j3g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-05-13  8:45             ` Anand Moon
2015-05-14 16:16             ` Anand Moon
2015-05-15  0:12               ` Krzysztof Kozlowski
     [not found]                 ` <CAJKOXPd_1rTZZCz66nQKidUOOEBmgUD+05fHUBvXN-VuD5mTCw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-05-15  1:36                   ` Anand Moon
2015-05-19  7:28                   ` Anand Moon
     [not found]                     ` <CANAwSgTbjmSxhJj4o2pnD5pR2V9JbQ3az+TSSCDwgwNnvF7P9w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-05-19  7:42                       ` Krzysztof Kozlowski
     [not found]                         ` <CADoNuNfuDCn_vvACc0=xj0MjdKr4pYcj_-Xh0--Q8_qciuW5YA@mail.gmail.com>
2015-05-20 14:59                           ` Anand Moon
2015-05-21  0:03                             ` Krzysztof Kozlowski
2015-05-23 17:35                               ` Anand Moon
2015-05-24  4:33                                 ` Krzysztof Kozlowski [this message]
2015-05-24  5:19                                   ` Anand Moon
2015-05-13  6:36 ` [PATCHv5 3/5] ARM: dts: define default thermal-zones for exynos5422 Anand Moon
2015-05-13  7:23   ` Krzysztof Kozlowski
     [not found]     ` <CAJKOXPerF9xgGzFuU+HC_9QaOAAMkoQo2DxH0Y=NwLJ7TMWH0A-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-05-13  8:02       ` Krzysztof Kozlowski
2015-05-13 13:16         ` Lukasz Majewski
2015-05-13  6:36 ` [PATCHv5 4/5] ARM: dts: exynos5420 update the cooling cells for core cpu0 Anand Moon
2015-05-13  7:31   ` Krzysztof Kozlowski
2015-05-13  7:58     ` Anand Moon
2015-05-13  8:05       ` Krzysztof Kozlowski
2015-05-13 10:07     ` Anand Moon
     [not found]       ` <CANAwSgTdQk6BGD4xxB6k=p=YOV+CHv_+Q_bSyrxiPsdXVps92g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-05-13 10:12         ` Krzysztof Kozlowski
2015-05-13 11:01           ` Anand Moon
2015-05-13  6:37 ` [PATCHv5 5/5] ARM: dts: Enable thermal_zone for exynos5422-odroidxu3 Anand Moon
2015-05-13  7:40   ` Krzysztof Kozlowski
2015-05-13  8:39     ` Anand Moon
2015-05-13 10:05       ` Krzysztof Kozlowski

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=CAJKOXPcQBVaaS+32GYpyQHQ3m_EZYqu0noXtpp_FVBEaGyUgGw@mail.gmail.com \
    --to=k.kozlowski@samsung.com \
    --cc=devicetree@vger.kernel.org \
    --cc=edubezval@gmail.com \
    --cc=kgene@kernel.org \
    --cc=l.majewski@samsung.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=linux.amoon@gmail.com \
    --cc=linux@arm.linux.org.uk \
    --cc=linux@roeck-us.net \
    --cc=m.reichl@fivetechno.de \
    --cc=sjoerd.simons@collabora.co.uk \
    --cc=tobetter@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).