Andreas,
On Thu, Jul 31, 2014 at 9:54 PM, Andreas Färber afaer...@suse.de wrote:
Use the new style for referencing inherited nodes and use symbolic names.
Goal is the alignment of all exynos5250 based device trees for comparison.
Signed-off-by: Andreas Färber afaer...@suse.de
---
v5: New
Andreas,
On Fri, Aug 1, 2014 at 5:52 PM, Andreas Färber afaer...@suse.de wrote:
Allows them to be extended by reference.
Signed-off-by: Andreas Färber afaer...@suse.de
---
v6: Split off from Snow/SMDK cleanups (Doug Anderson)
arch/arm/boot/dts/exynos5250.dtsi | 24
Tomasz,
On Thu, Jul 31, 2014 at 12:40 PM, Tomasz Figa tomasz.f...@gmail.com wrote:
Andreas,
On 31.07.2014 21:20, Andreas Färber wrote:
Am 31.07.2014 21:05, schrieb Tomasz Figa:
On 31.07.2014 18:08, Andreas Färber wrote:
Adds initial support for the HP Chromebook 11.
[snip]
+ gpio-keys
Andreas,
On Fri, Aug 1, 2014 at 5:52 PM, Andreas Färber afaer...@suse.de wrote:
Adds initial support for the HP Chromebook 11.
Cc: Vincent Palatin vpala...@chromium.org
Cc: Doug Anderson diand...@chromium.org
Cc: Stephan van Schaik step...@synkhronix.com
Signed-off-by: Andreas Färber afaer
Andreas,
On Sat, Aug 2, 2014 at 3:25 AM, Andreas Färber afaer...@suse.de wrote:
Hi,
Am 02.08.2014 06:57, schrieb Doug Anderson:
On Fri, Aug 1, 2014 at 7:34 PM, Javier Martinez Canillas
javier.marti...@collabora.co.uk wrote:
On 08/02/2014 02:52 AM, Andreas Färber wrote:
Based
Fabio,
On Wed, Aug 6, 2014 at 6:35 PM, Fabio Estevam feste...@gmail.com wrote:
On Wed, Aug 6, 2014 at 10:08 PM, Javier Martinez Canillas
javier.marti...@collabora.co.uk wrote:
+hsi2c_8 {
+ status = okay;
+ clock-frequency = 333000;
Doesn't it work at the more standard 400kHz
Hi,
On Fri, Aug 8, 2014 at 11:29 AM, Javier Martinez Canillas
javier.marti...@collabora.co.uk wrote:
Hello,
On 08/08/2014 06:38 PM, Javier Martinez Canillas wrote:
It seems as if the first call to exynos_irq_set_type() that is made by OF is
a
no-op while the second call is the one that
Javier,
On Fri, Aug 8, 2014 at 3:26 PM, Javier Martinez Canillas
javier.marti...@collabora.co.uk wrote:
I have some vague recollection that I set interrupts to pin-function
0 by default for some reason (assuming they would go to 0xf when
interrupts were enabled). ...but I can't for the life
is not set
CONFIG_THERMAL=y
I'm good with this, so:
Reviewed-by: Doug Anderson diand...@chromium.org
To address Bartlomiej: =m is what's in ChromeOS for exynos, though I
notice that other platforms in the ChromeOS tree have =y. I'm not
sure why there is a difference, but it's probably just
/boot/dts/exynos5420-peach-pit.dts | 12 ++--
arch/arm/boot/dts/exynos5800-peach-pi.dts | 12 ++--
2 files changed, 12 insertions(+), 12 deletions(-)
Matches my schematics. Kukjin: I think this could be applied to
for-next whenever it's convenient.
Acked-by: Doug Anderson diand
Bartlomiej,
On Mon, Aug 11, 2014 at 8:42 AM, Bartlomiej Zolnierkiewicz
b.zolnier...@samsung.com wrote:
Hi,
On Monday, August 11, 2014 02:52:27 PM Javier Martinez Canillas wrote:
Hello Bartlomiej,
On 08/11/2014 02:40 PM, Bartlomiej Zolnierkiewicz wrote:
index fc7d168..c390bb9 100644
---
Hi,
On Mon, Aug 11, 2014 at 10:28 AM, Bartlomiej Zolnierkiewicz
b.zolnier...@samsung.com wrote:
Hi,
On Monday, August 11, 2014 06:23:01 PM Javier Martinez Canillas wrote:
Hello,
On 08/11/2014 05:59 PM, Doug Anderson wrote:
Bartlomiej,
On Mon, Aug 11, 2014 at 8:42 AM, Bartlomiej
Javier,
On Tue, Aug 12, 2014 at 9:44 AM, Javier Martinez Canillas
javier.marti...@collabora.co.uk wrote:
The tps65090 PMU is a component used in many ChromeOS devices
so instead of having the same device tree definitions in many
files, create a .dtsi fragment that can be included in DTS.
Hi,
On Wed, May 21, 2014 at 2:08 AM, Seungwon Jeon tgih@samsung.com wrote:
On Wed, May 21, 2014, Doug Anderson wrote:
If we happened to get a data error at just the wrong time the dw_mmc
driver could get into a state where it would never complete its
request. That would leave the caller
anything else we end the
request and unblock anyone waiting.
Signed-off-by: Doug Anderson diand...@chromium.org
Signed-off-by: Yuvaraj Kumar C D yuvaraj...@gmail.com
Acked-by: Seungwon Jeon tgih@samsung.com
---
Changes in v2:
- Removed TODO
- Set cmd to NULL before calling dw_mci_request_end
Javier,
On Tue, Aug 19, 2014 at 8:08 AM, Javier Martinez Canillas
javier.marti...@collabora.co.uk wrote:
The VCC/VDD and VCCQ/VDD_IO power supplies for the MMC are
provided by the tps65090 fet4 and max77802 ldo4 regulators
respectively. Add the phandle to the regulators tree nodes
for the the
(-)
Reviewed-by: Doug Anderson diand...@chromium.org
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Anderson diand...@chromium.org
Tested-by: Doug Anderson diand...@chromium.org
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
, 5 insertions(+), 6 deletions(-)
Reviewed-by: Doug Anderson diand...@chromium.org
Tested-by: Doug Anderson diand...@chromium.org
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at http
/kernel/process.c | 1 -
2 files changed, 2 deletions(-)
Reviewed-by: Doug Anderson diand...@chromium.org
Tested-by: Doug Anderson diand...@chromium.org
(FYI: all patches tested by me were tested on rk3288-evb)
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body
linux/watchdog.h.
arch/arm64/kernel/process.c | 2 ++
1 file changed, 2 insertions(+)
Reviewed-by: Doug Anderson diand...@chromium.org
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at http
| 1 +
4 files changed, 30 insertions(+)
This patch doesn't apply cleanly with the in-flight (clk: rockchip:
protect critical clocks from getting disabled) patch from Heiko. It's
trivial to resolve and unclear which will land first, so I think it's
fine...
Reviewed-by: Doug Anderson diand
Guenter,
On Wed, Aug 20, 2014 at 9:42 PM, Guenter Roeck li...@roeck-us.net wrote:
On Wed, Aug 20, 2014 at 09:10:31PM -0700, Doug Anderson wrote:
Guenter,
On Tue, Aug 19, 2014 at 5:45 PM, Guenter Roeck li...@roeck-us.net wrote:
machine_restart is supported on non-ARM platforms
Kukjin,
On Fri, Aug 22, 2014 at 3:49 AM, Kukjin Kim kgene@samsung.com wrote:
Andreas Färber wrote:
Allows them to be extended by reference.
Reviewed-by: Doug Anderson diand...@chromium.org
Signed-off-by: Andreas Färber afaer...@suse.de
---
v6 - v7:
* Dropped uart* labels (Tomasz
Ulf,
On Fri, Aug 22, 2014 at 8:35 AM, Ulf Hansson ulf.hans...@linaro.org wrote:
On 22 August 2014 15:47, Yuvaraj Kumar C D yuvaraj...@gmail.com wrote:
From: Doug Anderson diand...@chromium.org
For UHS cards we need the ability to switch voltages from 3.3V to
1.8V. Add support to the dw_mmc
Hi,
On Fri, Aug 22, 2014 at 3:02 PM, Javier Martinez Canillas
javier.marti...@collabora.co.uk wrote:
Hello Mark,
On 08/22/2014 08:30 PM, Mark Brown wrote:
The problem is that one of these regulators is used as the vqmmc-supply
(VCCQ/VDD_IO) so the mmc host controller driver disables it on
Jaehoon,
On Mon, Aug 25, 2014 at 5:32 AM, Jaehoon Chung jh80.ch...@samsung.com wrote:
On 08/22/2014 10:47 PM, Yuvaraj Kumar C D wrote:
This patch makes use of mmc_regulator_get_supply() to handle
the vmmc and vqmmc regulators.Also it moves the code handling
the these regulators to
Ulf,
On Mon, Aug 25, 2014 at 1:13 AM, Ulf Hansson ulf.hans...@linaro.org wrote:
On 22 August 2014 20:27, Sonny Rao sonny...@chromium.org wrote:
On Fri, Aug 22, 2014 at 8:31 AM, Ulf Hansson ulf.hans...@linaro.org wrote:
On 22 August 2014 15:47, Yuvaraj Kumar C D yuvaraj...@gmail.com wrote:
Jaehoon,
On Mon, Aug 25, 2014 at 1:50 AM, Jaehoon Chung jh80.ch...@samsung.com wrote:
On 08/25/2014 05:13 PM, Ulf Hansson wrote:
On 22 August 2014 20:27, Sonny Rao sonny...@chromium.org wrote:
On Fri, Aug 22, 2014 at 8:31 AM, Ulf Hansson ulf.hans...@linaro.org wrote:
On 22 August 2014 15:47,
Javier,
On Mon, Aug 25, 2014 at 2:07 AM, Javier Martinez Canillas
javier.marti...@collabora.co.uk wrote:
Hello Yuvaraj,
On 08/25/2014 10:22 AM, Yuvaraj Cd wrote:
Good question. I'm not that familiar with the dw_mmc host controller nor
its driver implementation so I'll let Yuvaraj or Doug to
Ulf,
On Mon, Aug 25, 2014 at 1:31 AM, Ulf Hansson ulf.hans...@linaro.org wrote:
On 22 August 2014 22:38, Doug Anderson diand...@google.com wrote:
Ulf,
On Fri, Aug 22, 2014 at 8:35 AM, Ulf Hansson ulf.hans...@linaro.org wrote:
On 22 August 2014 15:47, Yuvaraj Kumar C D yuvaraj...@gmail.com
to operating mode NORMAL in that case.
+*/
+ if (val == MAX77802_OPMODE_OFF)
+ max77802-opmode[id] = MAX77802_OPMODE_NORMAL;
+ else
+ max77802-opmode[id] = val;
Reviewed-by: Doug Anderson diand
Jaehoon,
On Tue, Aug 26, 2014 at 8:48 PM, Jaehoon Chung jh80.ch...@samsung.com wrote:
Hi, Doug,
On 08/26/2014 12:25 AM, Doug Anderson wrote:
Jaehoon,
On Mon, Aug 25, 2014 at 1:50 AM, Jaehoon Chung jh80.ch...@samsung.com
wrote:
On 08/25/2014 05:13 PM, Ulf Hansson wrote:
On 22 August 2014
Jaehoon,
On Tue, Aug 26, 2014 at 9:47 PM, Jaehoon Chung jh80.ch...@samsung.com wrote:
Doug,
On 08/27/2014 01:14 PM, Doug Anderson wrote:
Jaehoon,
On Tue, Aug 26, 2014 at 8:48 PM, Jaehoon Chung jh80.ch...@samsung.com
wrote:
Hi, Doug,
On 08/26/2014 12:25 AM, Doug Anderson wrote:
Jaehoon
Ulf,
On Wed, Aug 27, 2014 at 4:17 AM, Ulf Hansson ulf.hans...@linaro.org wrote:
Now, we've got MMC_CAP_NEEDS_POLL, so dw_mmc will periodically be
called to check the card detect line, but with vmmc and vqmmc off. It
will be unable to return a sensible value without actually turning on
vmmc
Ulf,
On Thu, Aug 28, 2014 at 12:25 AM, Ulf Hansson ulf.hans...@linaro.org wrote:
On 27 August 2014 17:52, Doug Anderson diand...@google.com wrote:
Ulf,
On Wed, Aug 27, 2014 at 4:17 AM, Ulf Hansson ulf.hans...@linaro.org wrote:
Now, we've got MMC_CAP_NEEDS_POLL, so dw_mmc will periodically
Jaehoon,
On Thu, Aug 28, 2014 at 1:43 AM, Jaehoon Chung jh80.ch...@samsung.com wrote:
On 08/28/2014 12:49 AM, Doug Anderson wrote:
Jaehoon,
On Tue, Aug 26, 2014 at 9:47 PM, Jaehoon Chung jh80.ch...@samsung.com
wrote:
Doug,
On 08/27/2014 01:14 PM, Doug Anderson wrote:
Jaehoon,
On Tue
Hi,
On Thu, Aug 28, 2014 at 10:50 AM, Sonny Rao sonny...@chromium.org wrote:
On Thu, Aug 28, 2014 at 8:50 AM, Doug Anderson diand...@google.com wrote:
Ulf,
On Thu, Aug 28, 2014 at 12:25 AM, Ulf Hansson ulf.hans...@linaro.org wrote:
On 27 August 2014 17:52, Doug Anderson diand...@google.com
Will,
On Fri, Sep 5, 2014 at 5:22 AM, Will Deacon will.dea...@arm.com wrote:
[Looks like it's not just Rutland that can't spell the address of the
mailing list today. Fixed here, so please use this post in any replies].
On Fri, Sep 05, 2014 at 12:57:04PM +0100, Will Deacon wrote:
Hi all,
Hi,
On Sun, Sep 7, 2014 at 8:52 AM, Tomasz Figa tomasz.f...@gmail.com wrote:
So I believe we've got a process issue here. If you don't have normal
support for display hardware, but you want to keep the display
operational thanks to bootloader already initializing it, you should not
add
Hi,
On Sun, Sep 7, 2014 at 8:52 AM, Tomasz Figa tomasz.f...@gmail.com wrote:
Now with regulators this is pretty straightforward, but with clocks I
believe it's an open issue. AFAIR we've discussed this on MLs some time
ago (at least I remember Doug commenting on that topic) and kind of
Javier,
On Sun, Sep 7, 2014 at 11:09 PM, Javier Martinez Canillas
javier.marti...@collabora.co.uk wrote:
Hello Doug,
On 09/08/2014 06:36 AM, Doug Anderson wrote:
One (ugly?) solution would be to add a feature to your bootloader to
modify the device tree to mark regulators as always
Grant,
On Mon, Sep 8, 2014 at 5:20 AM, Grant Likely grant.lik...@secretlab.ca wrote:
On Mon, Sep 8, 2014 at 12:21 PM, Will Deacon will.dea...@arm.com wrote:
On Sun, Sep 07, 2014 at 05:19:03PM +0100, Tomasz Figa wrote:
At least for next 3.17-rc I'd suggest fixing this up in respective clock
Will,
On Mon, Sep 8, 2014 at 9:07 AM, Will Deacon will.dea...@arm.com wrote:
On Mon, Sep 08, 2014 at 04:55:31PM +0100, Doug Anderson wrote:
So simple-framebuffer is added to the device tree here:
https://chromium-review.googlesource.com/#/c/49358/2/board/samsung/smdk5250/smdk5250.c
That's
Javier,
On Wed, Aug 20, 2014 at 4:19 AM, Javier Martinez Canillas
javier.marti...@collabora.co.uk wrote:
Exynos5420 based Peach Pit and Exynos5800 based Peach Pi boards
uses a Maxim 77802 power management IC to drive regulators and
its Real Time Clock. This patch adds support for this chip.
Canillas javier.marti...@collabora.co.uk
Signed-off-by: Javier Martinez Canillas javier.marti...@collabora.co.uk
---
v2: 1) Add blank line before hdmi regulators.
arch/arm/boot/dts/exynos5800-peach-pi.dts | 5 +
1 file changed, 5 insertions(+)
Reviewed-by: Doug Anderson diand...@chromium.org
Hi,
On Tue, Sep 9, 2014 at 9:44 PM, Doug Anderson diand...@chromium.org wrote:
Javier / Rahul,
On Tue, Aug 19, 2014 at 8:08 AM, Javier Martinez Canillas
javier.marti...@collabora.co.uk wrote:
From: Rahul Sharma rahul.sha...@samsung.com
Adding regulators for HDMI for Peach-pi board
Canillas javier.marti...@collabora.co.uk
Signed-off-by: Javier Martinez Canillas javier.marti...@collabora.co.uk
---
arch/arm/boot/dts/exynos5800-peach-pi.dts | 5 +
1 file changed, 5 insertions(+)
Reviewed-by: Doug Anderson diand...@chromium.org
--
To unsubscribe from this list: send the line
Canillas javier.marti...@collabora.co.uk
Signed-off-by: Javier Martinez Canillas javier.marti...@collabora.co.uk
---
arch/arm/boot/dts/exynos5420-peach-pit.dts | 5 +
1 file changed, 5 insertions(+)
Reviewed-by: Doug Anderson diand...@chromium.org
--
To unsubscribe from this list: send
/arm/boot/dts/exynos5420-peach-pit.dts | 6
arch/arm/boot/dts/exynos5800-peach-pi.dts | 6
3 files changed, 56 insertions(+)
Reviewed-by: Doug Anderson diand...@chromium.org
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message
changed, 12 insertions(+)
I would note that the downstream dts file has this i2c bus at 400kHz.
...but that's not a problem with your patch. Perhaps you could submit
that as a separate patch?
Reviewed-by: Doug Anderson diand...@chromium.org
--
To unsubscribe from this list: send the line
Hi,
On Mon, Sep 8, 2014 at 8:47 PM, kg...@kernel.org wrote:
Javier Martinez Canillas wrote:
Hello Kukjin,
Hi,
On Wed, Aug 20, 2014 at 1:19 PM, Javier Martinez Canillas
javier.marti...@collabora.co.uk wrote:
This is a second version of the series that adds max77802 support for
the
Will,
On Wed, Sep 10, 2014 at 4:17 AM, Will Deacon will.dea...@arm.com wrote:
On Mon, Sep 08, 2014 at 03:05:40PM +0100, Javier Martinez Canillas wrote:
Hello Will,
Hi Javier,
Since many folks don't agree that hacking different subsystems is the way
forward I'll hold the patches and don't
Grant,
On Wed, Sep 10, 2014 at 9:29 AM, Grant Likely grant.lik...@secretlab.ca wrote:
On Wed, Sep 10, 2014 at 4:39 PM, Mark Brown broo...@kernel.org wrote:
On Wed, Sep 10, 2014 at 03:56:16PM +0100, Grant Likely wrote:
On Wed, Sep 10, 2014 at 3:31 PM, Mark Brown broo...@kernel.org wrote:
As
Mark,
On Wed, Sep 10, 2014 at 12:45 PM, Mark Brown broo...@kernel.org wrote:
On Wed, Sep 10, 2014 at 09:45:21AM -0700, Doug Anderson wrote:
Right now I know that clock disabling is supposed to be inhibited
during the early boot process. I think regulators too?
No, for regulators we'll
Javier,
On Wed, Sep 10, 2014 at 3:19 AM, Javier Martinez Canillas
javier.marti...@collabora.co.uk wrote:
The downstream ChromeOS 3.8 kernel sets the clock frequency
for the I2C bus 7 at 400kHz. Do the same change in mainline.
Suggested-by: Doug Anderson diand...@chromium.org
Signed-off
-by: Doug Anderson diand...@chromium.org
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi,
On Thu, Sep 11, 2014 at 11:03 AM, Mark Brown broo...@kernel.org wrote:
On Thu, Sep 11, 2014 at 10:22:32AM +0100, Grant Likely wrote:
On Wed, 10 Sep 2014 17:57:23 +0100, Mark Brown broo...@kernel.org wrote:
It's not quite as simple as just disabling PM - for example in the
clocks case
Hi,
On Fri, Sep 12, 2014 at 1:17 AM, Javier Martinez Canillas
javier.marti...@collabora.co.uk wrote:
From: Doug Anderson diand...@chromium.org
The max77686 includes an RTC that keeps power during suspend. It's
convenient to be able to use it as a wakeup source.
NOTE: due to wakeup ordering
Hi,
On Fri, Sep 12, 2014 at 8:54 AM, Javier Martinez Canillas
javier.marti...@collabora.co.uk wrote:
You are right, I completely forgot to check if that actually landed and to
remove the note in that case...
Maybe when the set is applied the note can be removed from this patch or do
you
recommended reset procedure
are now merged in upstream.
So you can go ahead and pick this change.
arch/arm/boot/dts/exynos5420-peach-pit.dts |2 +-
arch/arm/boot/dts/exynos5800-peach-pi.dts |2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
Reviewed-by: Doug Anderson diand
Hi,
On Tue, Sep 16, 2014 at 8:20 AM, Javier Martinez Canillas
jav...@dowhile0.org wrote:
Hello Daniel,
On Tue, Sep 16, 2014 at 5:03 PM, Daniel Drake dr...@endlessm.com wrote:
On Tue, Sep 16, 2014 at 7:48 AM, Javier Martinez Canillas
jav...@dowhile0.org wrote:
Clock list for s3c-rtc device:
/exynos5800-peach-pi.dts | 5 -
2 files changed, 8 insertions(+), 2 deletions(-)
Reviewed-by: Doug Anderson diand...@chromium.org
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at http
that it will eventually need to get merged with Andreas's, so:
Reviewed-by: Doug Anderson diand...@chromium.org
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo
Daniel,
On Wed, Sep 17, 2014 at 9:49 AM, Daniel Drake dr...@endlessm.com wrote:
On Tue, Sep 16, 2014 at 4:15 PM, Doug Anderson diand...@chromium.org wrote:
NOTE: I don't think that the builtin RTC is terribly important for any
exynos-based Chromebooks that I'm aware of. We rely on the RTC
taken them (I think).
Doug Anderson (1):
rtc: max77686: Allow the max77686 rtc to wakeup the system
http://ozlabs.org/~akpm/mmots/broken-out/rtc-max77686-allow-the-max77686-rtc-to-wakeup-the-system.patch
Javier Martinez Canillas (5):
rtc: max77686: Remove dead code for SMPL and WTSR
/mfd/syscon.c | 96
++
1 file changed, 74 insertions(+), 22 deletions(-)
You probably already have enough tags, but just in case. ;) On an
rk3288-based system (this patch backported to 3.14):
Tested-by: Doug Anderson diand...@chromium.org
Bartlomiej
On Mon, Sep 29, 2014 at 5:31 AM, Bartlomiej Zolnierkiewicz
b.zolnier...@samsung.com wrote:
Hi,
On Friday, August 29, 2014 01:34:44 PM Ulf Hansson wrote:
On 22 August 2014 15:47, Yuvaraj Kumar C D yuvaraj...@gmail.com wrote:
This patch makes use of mmc_regulator_get_supply() to
Hi,
On Wed, Oct 1, 2014 at 6:06 AM, Bartlomiej Zolnierkiewicz
b.zolnier...@samsung.com wrote:
Hi,
On Tuesday, September 30, 2014 10:22:34 AM Doug Anderson wrote:
Bartlomiej
On Mon, Sep 29, 2014 at 5:31 AM, Bartlomiej Zolnierkiewicz
b.zolnier...@samsung.com wrote:
Hi,
On Friday
Hi,
On Wed, Oct 1, 2014 at 7:00 AM, Bartlomiej Zolnierkiewicz
b.zolnier...@samsung.com wrote:
Hi,
On Wednesday, October 01, 2014 12:47:52 AM YUVARAJ CD wrote:
Since I am out of station, i dont have an access to my work set up.
Can you send me the dts entries of sd crad and their
and vqmmc is
not specified. Fix it.
Without this patch:
- mmc: dw_mmc: use mmc_regulator_get_supply to handle regulators
patch causes a SD card detection to fail
- mmc: dw_mmc: Support voltage changes patch causes a boot hang
This patch fixes both above problems.
Suggested-by: Doug Anderson
Bartiomiej
On Thu, Oct 2, 2014 at 9:39 AM, Bartlomiej Zolnierkiewicz
b.zolnier...@samsung.com wrote:
On Thursday, October 02, 2014 09:19:08 AM Doug Anderson wrote:
Bartiomiej,
On Thu, Oct 2, 2014 at 9:10 AM, Bartlomiej Zolnierkiewicz
b.zolnier...@samsung.com wrote:
Regulators for MMC2 (SD
Bartlomiej,
On Thu, Oct 2, 2014 at 10:24 AM, Bartlomiej Zolnierkiewicz
b.zolnier...@samsung.com wrote:
Hi,
On Thursday, October 02, 2014 09:45:41 AM Doug Anderson wrote:
Bartiomiej
On Thu, Oct 2, 2014 at 9:39 AM, Bartlomiej Zolnierkiewicz
b.zolnier...@samsung.com wrote:
On Thursday
Hi,
On Wed, Oct 8, 2014 at 8:20 AM, Arnd Bergmann a...@arndb.de wrote:
Why does it cause a regression though? Does this mean you are breaking
any boot with an old DT file and a new kernel? That would be very
bad, the driver is supposed to keep working with an existing dtb.
The old dts file
that the .set_suspend_* function handlers
defined in regulator drivers are never called when the system is suspended.
Suggested-by: Doug Anderson diand...@chromium.org
Signed-off-by: Javier Martinez Canillas javier.marti...@collabora.co.uk
Could you also add a patch to your series ripping out the call
that the .set_suspend_* function handlers
defined by regulator drivers are not called when the system is suspended.
Suggested-by: Doug Anderson diand...@chromium.org
Signed-off-by: Javier Martinez Canillas javier.marti...@collabora.co.uk
---
arch/arm/mach-exynos/suspend.c | 18 ++
1 file
Javier,
On Mon, Oct 20, 2014 at 9:58 AM, Javier Martinez Canillas
javier.marti...@collabora.co.uk wrote:
It turns out that regulator_suspend_finish() actually returns an error
code. Could you print a warning if you see it?
Yes, I noticed this when looking at Chris patch for Rockchip but
.
It is better to call the regulator suspend prepare/finish functions
from platform code instead so this patch reverts the mentioned commit.
Suggested-by: Doug Anderson diand...@chromium.org
Signed-off-by: Javier Martinez Canillas javier.marti...@collabora.co.uk
Reviewed-by: Doug Anderson
a
register base address variable in mct controller driver), so 1.5
years. The broken code has been running for a long time, but that
doesn't mean that there isn't a subtle issue...
In any case, it seems like we should take this fix, assuming someone
has tested it well.
Reviewed-by: Doug Anderson diand
Javier,
On Mon, Nov 17, 2014 at 9:43 AM, Javier Martinez Canillas
javier.marti...@collabora.co.uk wrote:
+spi_1 {
+ status = okay;
+ samsung,spi-src-clk = 0;
+ num-cs = 1;
+ cs-gpios = gpa2 5 0;
+
+ spidev@0 {
+ compatible = spidev;
This is
Javier,
On Wed, Nov 19, 2014 at 2:07 AM, Javier Martinez Canillas
javier.marti...@collabora.co.uk wrote:
Hello Doug,
Thanks for your feedback.
On 11/18/2014 06:50 PM, Doug Anderson wrote:
This is common practice in the Chrome OS tree, but we've gotten
pushback from upstream questioning
Linus,
I'm currently working towards adapting exynos5250-snow (the ARM
Chromebook) to work well in the new world of pinctrl. We've got a
backport of exynos5250 pinctrl in our kernel-3.8 tree and are now
fixing all of the bugs that have popped up. Patches will be sent
upstream (where applicable)
Tomasz,
Thanks for your comments. I'm glad I'm not totally off-track. I'll
respond to most things in reply to Linus' email, but a few here:
On Wed, May 15, 2013 at 10:26 AM, Tomasz Figa tomasz.f...@gmail.com wrote:
pinctrl@1140 {
cyapa_irq: cyapa-irq {
samsung,pins = gpx1-2;
Linus,
Thank you for your comments. See below...
Stephen: sorry for missing you earlier! :(
On Wed, May 15, 2013 at 11:29 AM, Linus Walleij
linus.wall...@linaro.org wrote:
But please use the preprocessor to provide symbolic names for
the configurations. See for example these two patches
Tomasz,
On Wed, May 15, 2013 at 2:36 PM, Tomasz Figa tomasz.f...@gmail.com wrote:
One potential reason for leaving them is the hopes that it might cause
a little less line glitching, especially in the case of outputs.
There is some delay between the pinmux being configured at the start
of
Tomasz,
On Wed, May 15, 2013 at 2:41 PM, Tomasz Figa tomasz.f...@gmail.com wrote:
This will be hard, since the phandle in interrupt-parent is represented by
an IRQ domain in kernel code. One-interrupt IRQ domains seem a bit awkward
to me.
Since we are already going to modify the binding,
Tomasz / Linus,
On Wed, May 15, 2013 at 3:06 PM, Tomasz Figa tomasz.f...@gmail.com wrote:
Yes. I don't like the current way too much either, duplication being one
of the reasons.
Do you have any other ideas? It sounds like Linus didn't like my
suggestion and makes some good points...
I don't
Stephen,
On Wed, May 15, 2013 at 4:51 PM, Stephen Warren swar...@wwwdotorg.org wrote:
I don't really see much disadvantage here; the interrupt bindings
specify things related to interrupts and the pinctrl bindings specify
thing related to pin configuration.
OK. If this is the best way then I
Tomasz,
On Wed, May 15, 2013 at 5:13 PM, Tomasz Figa tomasz.f...@gmail.com wrote:
I don't have anything interesting at the moment. It's a bit late now here
(2 AM), so I'm going to get some sleep first.
Sorry for keeping you up. Sleep is good!
Also after reading Stephen's reply, I'm
Tomasz,
On Wed, May 15, 2013 at 5:19 PM, Tomasz Figa tomasz.f...@gmail.com wrote:
Hmm, last thing before I fell asleep: We already have support for power
down configuration in pinctrl-samsung. See samsung,pin-conpdn and
samsung,pin-pudpdn.
Dang it! OK, we'll work on using those.
Also I
/gerrit/#/c/51342/3
Doug Anderson (1):
pinctrl: samsung: fix suspend/resume functionality
Prathyush K (1):
pinctrl: exynos: fix eint wakeup by using irq_set_wake()
drivers/pinctrl/pinctrl-exynos.c | 45 ++---
drivers/pinctrl/pinctrl-exynos.h | 3 +-
drivers/pinctrl/pinctrl-samsung.c
samsung_gpio_pm_2bit_resume(), which seemed to
do this a reasonable way.
Patch originally from Prathyush K prathyus...@samsung.com but
rewritten by Doug Anderson diand...@chromium.org.
Signed-off-by: Prathyush K prathyus...@samsung.com
Signed-off-by: Doug Anderson diand...@chromium.org
---
drivers/pinctrl/pinctrl
From: Prathyush K prathyus...@samsung.com
Add the irq_set_wake function for exynos pinctrl to configure the
external interrupt wakeup mask register.
[dianders: minor nit fixes; port to ToT]
Signed-off-by: Prathyush K prathyus...@samsung.com
Signed-off-by: Doug Anderson diand...@chromium.org
Tomasz / Stephen,
On Wed, May 15, 2013 at 5:55 PM, Doug Anderson diand...@google.com wrote:
Also after reading Stephen's reply, I'm wondering if hogging wouldn't
solve the problem indeed. (It might have to be fixed on pinctrl-samsung
first, as last time I tried to use it, it caused some errors
Tomasz,
Thanks for the review! I'll get a new patch out either today or tomorrow...
On Thu, May 16, 2013 at 12:19 PM, Tomasz Figa tomasz.f...@gmail.com wrote:
+/**
+ * samsung_pinctrl_resume_noirq - save pinctrl state for suspend
+ *
+ * Save data for all banks handled by this device.
+ */
Tomasz,
On Thu, May 16, 2013 at 2:27 PM, Tomasz Figa tomasz.f...@gmail.com wrote:
OK. I will be fine to go with your patches, after addressing the comments.
In the end it's good that you posted them, as reviewing them allowed me to
find even better ways of doing some things than I had in mine
Tomasz,
On Thu, May 16, 2013 at 12:26 PM, Tomasz Figa tomasz.f...@gmail.com wrote:
On Thursday 16 of May 2013 10:12:32 Doug Anderson wrote:
From: Prathyush K prathyus...@samsung.com
Add the irq_set_wake function for exynos pinctrl to configure the
external interrupt wakeup mask register
Tomasz,
On Thu, May 16, 2013 at 3:56 PM, Tomasz Figa tomasz.f...@gmail.com wrote:
Seems like I need some sleep, as I'm already starting to overlook large
blobs of code.
Originally, GPIO suspend/resume handlers have been configured in
drivers/gpio/gpio-samsung.c, by setting pm field of
their powerdown state.
Patch originally from Prathyush K prathyus...@samsung.com but
rewritten by Doug Anderson diand...@chromium.org.
Signed-off-by: Prathyush K prathyus...@samsung.com
Signed-off-by: Doug Anderson diand...@chromium.org
---
Changes in v2:
- Now uses sycore_ops to make sure we're early
Tomasz,
On Thu, May 16, 2013 at 3:37 PM, Tomasz Figa tomasz.f...@gmail.com wrote:
Well, to make long story short, including headers from plat/ and mach/
from files outside plat/ or mach/ is no longer valid with
CONFIG_MULTIPLATFORM, because more than one plat and/or mach can be
enabled at the
501 - 600 of 710 matches
Mail list logo
