RE: [GIT PULL 2/2] Samsung defconfig updates for v3.18

2014-10-15 Thread Kukjin Kim
Sjoerd Simons wrote: > > On Tue, 2014-10-14 at 23:32 -0700, Olof Johansson wrote: > > On Tue, Oct 14, 2014 at 4:27 PM, Kukjin Kim wrote: > > > git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git > > > tags/samsung-defconfig > > > > > > for you to fetch changes up to 508423bebcd

Re: [PATCH 1/1] ARM: EXYNOS: Call regulator core suspend prepare and finish functions

2014-10-15 Thread Javier Martinez Canillas
Hello Doug, Thanks a lot for your feedback. On 10/15/2014 06:19 PM, Doug Anderson wrote: > Javier, > > On Wed, Oct 15, 2014 at 5:01 AM, Javier Martinez Canillas > wrote: >> The regulator framework has a set of helpers functions to be used when >> the system is entering and leaving from suspend

[PATCH 2/9] regulator: max77802: Add .{get,set}_mode callbacks

2014-10-15 Thread Javier Martinez Canillas
Some max77802 LDOs (1, 3, 20 and 21) support to be configured in Low Power Mode during system normal operation. Add function handlers for the .get_mode and .set_mode operations to set the mode on these LDOs. Signed-off-by: Javier Martinez Canillas --- drivers/regulator/max77802.c | 44 ++

[PATCH 3/9] regulator: max77802: Split regulator operations for BUCKs

2014-10-15 Thread Javier Martinez Canillas
Not all the max77802 BUCKs regulators have the same functionality, for example BUCKs 2-4 support the output to be configured as normal or Low Power Mode by the PWRREQ enable pin while the other BUCKs only support their output to be set ON or OFF by PWRREQ. As a preparation for adding a set_suspend_

[PATCH 0/9] Add max77802 regulator operating mode support

2014-10-15 Thread Javier Martinez Canillas
Hello Mark, This is the second attempt to add operating modes support for the regulators in the max77802 PMIC. Instead of trying to come up with a generic solution like in the previous attempt [0], I added DT properties that are specific to the max77802 PMIC so is documented clearly what are the o

[PATCH 1/9] regulator: max77802: Add .set_suspend_{enable,disable} callbacks

2014-10-15 Thread Javier Martinez Canillas
The max77802 PMIC has an enable pin (PWRREQ) that can be used to switch regulators ON and OFF automatically by the Application Processor when the system is leaving and entering sleep mode. Only the BUCKs regulators had a .set_suspend_disable function handler that used the enable pin to turn OFF th

[PATCH 5/9] regulator: max77802: Don't treat OFF as an operating mode

2014-10-15 Thread Javier Martinez Canillas
The only operating modes that are supported by the regulators in the max77802 PMIC are Output ON (normal) and Output On in Low Power Mode. OFF was wrongly counted as an operating mode while is only a regulator status. Make clear in the code that OFF is not an operating mode. Signed-off-by: Javier

[PATCH 4/9] regulator: max77802: Add set suspend mode for BUCKs and simplify code

2014-10-15 Thread Javier Martinez Canillas
The max77802 PMIC has a special enable pin (PWRREQ) that can be used by the Application Processor (AP) to power down and up voltage rails. The max77802 PMIC regulators have 3 different enable control logics. Some regulators support to be configured on different operational mode during normal opera

[PATCH 6/9] regulator: max77802: Add header for operating modes

2014-10-15 Thread Javier Martinez Canillas
Add a header file for the max77802 constants that could be shared between the regulator driver and Device Tree source files. Also, remove standby and off opmodes since only normal and low power are valid operating modes. Signed-off-by: Javier Martinez Canillas --- drivers/regulator/max77802.c

[PATCH 7/9] regulator: max77802: Document regulator opmode DT properties

2014-10-15 Thread Javier Martinez Canillas
Some regulators from the max77802 PMIC support to be configured in one of two operating mode: Output ON (normal) and Output On Low Power Mode. Not all regulators support these two modes and for some of them, the mode can be changed while the system is running in normal operation while others only s

[PATCH 9/9] ARM: dts: Configure regulators for suspend on exynos Peach boards

2014-10-15 Thread Javier Martinez Canillas
The regulator core now has support to choose if a regulator has to be enabled or disabled during system suspend and also the max77802 driver supports changing the regulator operating mode during runtime and when the system enters into sleep mode. To lower power during suspend, configure the regula

[PATCH 8/9] regulator: max77802: Parse regulator operating mode properties

2014-10-15 Thread Javier Martinez Canillas
The max77802 PMIC regulators output can be configured in one of two modes: Output ON (normal) and Output ON in Low Power Mode. Some of the regulators support their operating mode to be changed on startup or by consumers when the system is running while others only support their operating mode to be

Re: [PATCH 1/1] ARM: EXYNOS: Call regulator core suspend prepare and finish functions

2014-10-15 Thread Doug Anderson
Javier, On Wed, Oct 15, 2014 at 5:01 AM, Javier Martinez Canillas wrote: > The regulator framework has a set of helpers functions to be used when > the system is entering and leaving from suspend but these are not called > on Exynos platforms. This means that the .set_suspend_* function handlers

Re: [PATCH v2 1/4] dwc3: exynos: Add support for SCLK present on Exynos7

2014-10-15 Thread Felipe Balbi
Hi, On Tue, Oct 14, 2014 at 10:25:00AM +0530, Vivek Gautam wrote: > Hi Felipe, > > > On Tue, Oct 14, 2014 at 4:14 AM, Felipe Balbi wrote: > > Hi, > > > > On Mon, Oct 13, 2014 at 01:54:59PM +0900, Anton Tikhomirov wrote: > >> Hi Vivek, > >> > >> > Exynos7 also has a separate special gate clock g

Re: [PATCH] arm64: dts: exynos7: add support for cpuidle core power down

2014-10-15 Thread Lorenzo Pieralisi
On Wed, Oct 15, 2014 at 02:02:18PM +0100, Mark Rutland wrote: > > > + CPU_SLEEP: cpu-sleep { > > > + compatible = "arm,idle-state"; > > > + local-timer-stop; > > > + arm,psci-suspend-param = <0x001>; > > > +

Re: [GIT PULL 2/2] Samsung defconfig updates for v3.18

2014-10-15 Thread Sjoerd Simons
On Tue, 2014-10-14 at 23:32 -0700, Olof Johansson wrote: > On Tue, Oct 14, 2014 at 4:27 PM, Kukjin Kim wrote: > > git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git > > tags/samsung-defconfig > > > > for you to fetch changes up to 508423bebcda29eb0ba7c627f895387dad7cdcd6: > >

Re: [PATCH] arm64: dts: exynos7: add support for cpuidle core power down

2014-10-15 Thread Mark Rutland
> > + CPU_SLEEP: cpu-sleep { > > + compatible = "arm,idle-state"; > > + local-timer-stop; > > + arm,psci-suspend-param = <0x001>; > > + entry-latency-us = <20>; > > +

[PATCH 1/1] ARM: EXYNOS: Call regulator core suspend prepare and finish functions

2014-10-15 Thread Javier Martinez Canillas
The regulator framework has a set of helpers functions to be used when the system is entering and leaving from suspend but these are not called on Exynos platforms. This means that the .set_suspend_* function handlers defined in regulator drivers are never called when the system is suspended. Sugg

Re: exynos5422-odroid-xu3: MCPM: only 6 of 8 CPUs coming online

2014-10-15 Thread Joonyoung Shim
Hi, Kevin Hilman kernel.org> writes: > > Hi Abhilash, > > Abhilash Kesavan samsung.com> writes: > > > I had noticed a similar issue where only 6 cores would come up during > > development on 5420. This was because there was a check in our u-boot > > secondary boot-up path which would put a c

Re: [PATCH] arm64: dts: exynos7: add support for cpuidle core power down

2014-10-15 Thread Lorenzo Pieralisi
On Wed, Oct 15, 2014 at 07:35:20AM +0100, Chander Kashyap wrote: > Exynos7 has core power down state where cores can be powered off > independently. > This patch adds support for this state. Please tell us more about the idle-state values you are adding, in particular entry, exit latencies and mi