From: Krzysztof Kozlowski <krzk@kernel.org> To: Marek Szyprowski <m.szyprowski@samsung.com> Cc: Anand Moon <linux.amoon@gmail.com>, Kukjin Kim <kgene@kernel.org>, devicetree <devicetree@vger.kernel.org>, linux-arm-kernel <linux-arm-kernel@lists.infradead.org>, "linux-samsung-soc@vger.kernel.org" <linux-samsung-soc@vger.kernel.org>, Linux Kernel <linux-kernel@vger.kernel.org> Subject: Re: [PATCH v2 3/4] ARM: dts: exynos: Add regulator suspend configuration to Odroid XU3/XU4/HC1 family Date: Mon, 24 Jun 2019 09:41:27 +0200 [thread overview] Message-ID: <CAJKOXPeXELUjsuW9eyGuRj56067qnL-J2Lt4U42mYiyx=X+F_Q@mail.gmail.com> (raw) In-Reply-To: <d94a2f99-fb99-c238-7011-9bbb4c0cd90f@samsung.com> On Mon, 24 Jun 2019 at 09:20, Marek Szyprowski <m.szyprowski@samsung.com> wrote: > > Hi Anand, > > On 2019-06-23 18:02, Anand Moon wrote: > > Thanks for this patch. Please add my > > > > Tested-by: Anand Moon <linux.amoon@gmail.com> > > > > [snip] > > > > Could you integrate below small changes into this patch. > > with these below changes suspend and resume work correctly at my end. > > > > [1] XU4_suspendresume.patch > > > > As per S2MPS11B PMIC 1.2.1 Regulator (Features) > > Fix the min max value for *Buck7* and *Buck8* > > > > -- Buck7 (VDD_1.0V_LDO) 1.5 A (1.2 V to 1.5 V, 12.5 mV step, default on 1.35 V) > > -- Buck8 (VDD_1.8V_LDO) 2.5 A (1.8 V to 2.1 V, 12.5 mV step, default on 2.0 V) > > Could you elaborate why such change for Buck7 and Buck8 is needed? Anand has here valid point - the constraints in DTS do not match hardware manual. This leads to question whether voltage table in driver is proper... Another point is the voltage itself. The schematics describes them as at specific voltage (1.35 V and 2.0 V) but after boot they are 1.2 V and 1.85 V. Maybe this shift comes from the problem above. > > > Also add suspend-off for *Buck9* > > Buck9 internally controls the power of USB hub. > > Adding suspend the this node help proper reset of USB hub on Odroid > > XU4 / HC1/ XU3 > > during suspend and resume. Below it the logs from my testing. > > Disabling Buck9 in suspend indeed reduces the power consumed by the > board during suspend-to-ram from about 80mA to as little as 7-10mA, what > matches the results of OdroidXU3. Thanks for the hint! Although I did not get what is the difference in the logs (Anand pasted two logs but they look the same) but the power consumption is reason is good enough. I would be happy to put in the changelog entire consumption difference. I can measure it on XU3-Lite but can you give me the XU4 (before and after)? Best regards, Krzysztof
WARNING: multiple messages have this Message-ID (diff)
From: Krzysztof Kozlowski <krzk@kernel.org> To: Marek Szyprowski <m.szyprowski@samsung.com> Cc: devicetree <devicetree@vger.kernel.org>, "linux-samsung-soc@vger.kernel.org" <linux-samsung-soc@vger.kernel.org>, Linux Kernel <linux-kernel@vger.kernel.org>, Kukjin Kim <kgene@kernel.org>, linux-arm-kernel <linux-arm-kernel@lists.infradead.org> Subject: Re: [PATCH v2 3/4] ARM: dts: exynos: Add regulator suspend configuration to Odroid XU3/XU4/HC1 family Date: Mon, 24 Jun 2019 09:41:27 +0200 [thread overview] Message-ID: <CAJKOXPeXELUjsuW9eyGuRj56067qnL-J2Lt4U42mYiyx=X+F_Q@mail.gmail.com> (raw) In-Reply-To: <d94a2f99-fb99-c238-7011-9bbb4c0cd90f@samsung.com> On Mon, 24 Jun 2019 at 09:20, Marek Szyprowski <m.szyprowski@samsung.com> wrote: > > Hi Anand, > > On 2019-06-23 18:02, Anand Moon wrote: > > Thanks for this patch. Please add my > > > > Tested-by: Anand Moon <linux.amoon@gmail.com> > > > > [snip] > > > > Could you integrate below small changes into this patch. > > with these below changes suspend and resume work correctly at my end. > > > > [1] XU4_suspendresume.patch > > > > As per S2MPS11B PMIC 1.2.1 Regulator (Features) > > Fix the min max value for *Buck7* and *Buck8* > > > > -- Buck7 (VDD_1.0V_LDO) 1.5 A (1.2 V to 1.5 V, 12.5 mV step, default on 1.35 V) > > -- Buck8 (VDD_1.8V_LDO) 2.5 A (1.8 V to 2.1 V, 12.5 mV step, default on 2.0 V) > > Could you elaborate why such change for Buck7 and Buck8 is needed? Anand has here valid point - the constraints in DTS do not match hardware manual. This leads to question whether voltage table in driver is proper... Another point is the voltage itself. The schematics describes them as at specific voltage (1.35 V and 2.0 V) but after boot they are 1.2 V and 1.85 V. Maybe this shift comes from the problem above. > > > Also add suspend-off for *Buck9* > > Buck9 internally controls the power of USB hub. > > Adding suspend the this node help proper reset of USB hub on Odroid > > XU4 / HC1/ XU3 > > during suspend and resume. Below it the logs from my testing. > > Disabling Buck9 in suspend indeed reduces the power consumed by the > board during suspend-to-ram from about 80mA to as little as 7-10mA, what > matches the results of OdroidXU3. Thanks for the hint! Although I did not get what is the difference in the logs (Anand pasted two logs but they look the same) but the power consumption is reason is good enough. I would be happy to put in the changelog entire consumption difference. I can measure it on XU3-Lite but can you give me the XU4 (before and after)? Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2019-06-24 7:41 UTC|newest] Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-06-21 15:58 [PATCH v2 1/4] ARM: dts: exynos: Disable unused buck10 regulator on Odroid HC1 board Krzysztof Kozlowski 2019-06-21 15:58 ` Krzysztof Kozlowski 2019-06-21 15:58 ` [PATCH v2 2/4] ARM: dts: exynos: Add regulator suspend configuration to Arndale Octa board Krzysztof Kozlowski 2019-06-21 15:58 ` Krzysztof Kozlowski 2019-06-21 15:58 ` [PATCH v2 3/4] ARM: dts: exynos: Add regulator suspend configuration to Odroid XU3/XU4/HC1 family Krzysztof Kozlowski 2019-06-21 15:58 ` Krzysztof Kozlowski 2019-06-23 16:02 ` Anand Moon 2019-06-23 16:02 ` Anand Moon 2019-06-24 7:20 ` Marek Szyprowski 2019-06-24 7:20 ` Marek Szyprowski 2019-06-24 7:41 ` Krzysztof Kozlowski [this message] 2019-06-24 7:41 ` Krzysztof Kozlowski 2019-06-24 7:41 ` Krzysztof Kozlowski 2019-06-24 9:01 ` Marek Szyprowski 2019-06-24 9:01 ` Marek Szyprowski 2019-06-24 9:01 ` Marek Szyprowski 2019-06-24 13:42 ` Anand Moon 2019-06-24 13:42 ` Anand Moon 2019-06-24 13:42 ` Anand Moon 2019-06-21 15:58 ` [PATCH v2 4/4] ARM: dts: exynos: Use proper regulator for eMMC memory on Arndale Octa Krzysztof Kozlowski 2019-06-21 15:58 ` 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='CAJKOXPeXELUjsuW9eyGuRj56067qnL-J2Lt4U42mYiyx=X+F_Q@mail.gmail.com' \ --to=krzk@kernel.org \ --cc=devicetree@vger.kernel.org \ --cc=kgene@kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-samsung-soc@vger.kernel.org \ --cc=linux.amoon@gmail.com \ --cc=m.szyprowski@samsung.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: linkBe 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.