All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 00/28] Updates for Tegra support in 2.6.39
@ 2011-01-24  2:01 Colin Cross
  2011-01-24  2:01   ` Colin Cross
                   ` (27 more replies)
  0 siblings, 28 replies; 140+ messages in thread
From: Colin Cross @ 2011-01-24  2:01 UTC (permalink / raw)
  To: linux-arm-kernel

This patch set adds support for suspend, idle, and cpufreq
for the Tegra subsystem.

Patch 2 adds functions to save and restore the GIC state
during suspend and resume, and patch 3 exports the GIC irq 
chip functions to support custom IRQ chips that need to
update the GIC as well as another block during IRQ 
operations.  Patches 2 and 3 depend on patch 1, which fixes
a naming conflict in mach-tegra, so the whole stack will
need to go in through one tree.

 arch/arm/common/gic.c                          |  138 ++++-
 arch/arm/configs/tegra_defconfig               |  123 ++++
 arch/arm/include/asm/hardware/cache-l2x0.h     |    3 +-
 arch/arm/include/asm/hardware/gic.h            |   13 +
 arch/arm/mach-tegra/Makefile                   |   12 +-
 arch/arm/mach-tegra/board.h                    |    6 +
 arch/arm/mach-tegra/common.c                   |   49 ++-
 arch/arm/mach-tegra/cortex-a9.S                |  504 +++++++++++++++
 arch/arm/mach-tegra/cpu-tegra.c                |   75 ++-
 arch/arm/mach-tegra/cpuidle.c                  |  714 ++++++++++++++++++++++
 arch/arm/mach-tegra/dma.c                      |  198 ++++---
 arch/arm/mach-tegra/gpio.c                     |   15 +-
 arch/arm/mach-tegra/headsmp-t2.S               |  216 +++++++
 arch/arm/mach-tegra/headsmp.S                  |   61 --
 arch/arm/mach-tegra/hotplug.c                  |  126 ----
 arch/arm/mach-tegra/include/mach/clk.h         |    2 +
 arch/arm/mach-tegra/include/mach/clkdev.h      |    2 +
 arch/arm/mach-tegra/include/mach/debug-macro.S |   25 +-
 arch/arm/mach-tegra/include/mach/iomap.h       |   69 ++-
 arch/arm/mach-tegra/include/mach/irqs.h        |   14 +-
 arch/arm/mach-tegra/include/mach/legacy_irq.h  |    7 +-
 arch/arm/mach-tegra/include/mach/pinmux-t2.h   |   10 +
 arch/arm/mach-tegra/include/mach/powergate.h   |   40 ++
 arch/arm/mach-tegra/include/mach/suspend.h     |   80 +++
 arch/arm/mach-tegra/include/mach/system.h      |   10 +-
 arch/arm/mach-tegra/include/mach/uncompress.h  |   18 +-
 arch/arm/mach-tegra/irq.c                      |  334 ++++++++---
 arch/arm/mach-tegra/legacy_irq.c               |  109 ++++-
 arch/arm/mach-tegra/pinmux-t2-tables.c         |   26 +-
 arch/arm/mach-tegra/platsmp.c                  |   89 +++-
 arch/arm/mach-tegra/power-macros.S             |   57 ++
 arch/arm/mach-tegra/power.h                    |   64 ++
 arch/arm/mach-tegra/powergate.c                |  212 +++++++
 arch/arm/mach-tegra/suspend-t2.c               |  414 +++++++++++++
 arch/arm/mach-tegra/suspend.c                  |  776 ++++++++++++++++++++++++
 arch/arm/mach-tegra/tegra2_clocks.c            |  264 ++++++++-
 arch/arm/mach-tegra/tegra2_save.S              |  413 +++++++++++++
 arch/arm/mach-tegra/timer.c                    |   96 +++-
 arch/arm/mm/cache-l2x0.c                       |   14 +-
 39 files changed, 4898 insertions(+), 500 deletions(-)

^ permalink raw reply	[flat|nested] 140+ messages in thread
* [PATCH v2 20/28] ARM: tegra: cpufreq: Disable cpufreq during suspend
@ 2011-01-25 10:56 MyungJoo Ham
  2011-01-25 12:40   ` Mark Brown
  0 siblings, 1 reply; 140+ messages in thread
From: MyungJoo Ham @ 2011-01-25 10:56 UTC (permalink / raw)
  To: linux-arm-kernel

2011. 1. 25. ?? 1:27? "Kyungmin Park" <kmpark@infradead.org>?? ??:
>
> On Tue, Jan 25, 2011 at 4:35 AM, Mark Brown
> <broonie@opensource.wolfsonmicro.com> wrote:
> > On Mon, Jan 24, 2011 at 10:50:53AM -0800, Colin Cross wrote:
> >> On Mon, Jan 24, 2011 at 6:41 AM, Mark Brown
> >
> >> > Hrm, what's the situation where that happens and why does it cause
> >> > problems?  The regulator API doesn't care if suspend is going on, and
> >> > nor do any of the current drivers for regulators.  There is an issue
> >> > with keeping things like I2C alive until the bitter end of suspend so
> >> > you've got a control bus to the regulators but that's a generic issue
> >> > which crops up with other subsystems too so a regulator-specific
> >> > workaround seems dodgy.
> >
> >> The end result is that the regulator driver calls the i2c driver after
> >> the i2c driver has been suspended.  On Tegra, this happens because of
> >> voltage scaling on the CPU regulator, which can be on the i2c bus.  A
> >> sleep in a suspend handler after the i2c bus has been suspended can
> >> cause the cpufreq governor to lower the frequency, and try to lower
> >> the voltage as well.
> >
> > OK, that's a general problem - you need to ensure that the I2C and SPI
> > controllers are suspended really late.  The OMAP guys also have this
> > problem for some RTCs, though in their case things are slightly
> > different due to their use of runtime PM.  Looking at your I2C driver
> > (BTW, I'd suggest reminding Ben on a more regular basis about that) I
> > suspect that moving your suspend and resume callbacks to the _noirq
> > varaints will cover a lot of this.
>
> Right, it's same as Samsung SoCs. we also disable to access the
> cpufreq when suspending.
> the PMIC is connected with I2C  and I2C is suspended before PMIC.
>
> Thank you,
> Kyungmin Park
>

Moreover, the suspend-related cpufreq issue gets worse with hibernation.
Generally PMICs preserve their register values during suspend-to-mem;
however, they lose the values with power-off (and hibernation).

In samsung SoCs, we have mitigated the issue by stopping cpufreq at pm_begin
(could have a bit different name) and resume with end.  When stopping
cpufreq, we have made it to fix at the boot freq so that we dont face into
the inconsistency with hibernation.

Sometimes, when we set voltages for the core, bus, and others, simply saving
the whole things and restoring them does not work because the order how the
voltages change may matter (and it does with some of Samsung SoCs). Thus, we
made them to stop changing and fix at the boot-time default during suspend
and hibernation.

Cheers!

MyungJoo

> >
> >> The problem isn't limited to i2c busses, there are some regulators on
> >> spi busses.  You would need to ensure that any driver that has a
> >
> > For all practical purposes in this sort of discussion you can typically
> > do a s/I2C/I2C and SPI/ - in terms of system integration issues they're
> > very similar.
> >
> >> regulator consumer suspends before the regulator driver it
> >> communicates with, and that still wouldn't fix cpufreq, which has a
> >> sysdev suspend handler.
> >
> > So you actively need to push the processor into low power mode during
> > suspend?  I'm still not 100% clear what's triggering the issues you're
> > seeing and why this seems to be Tegra-only.  If there were a general
> > cpufreq/regulator interaction here I'd expect to see all ARM cpufreq
> > drivers needing to do the same thing (and probably some handling of this
> > in cpufreq core as a result).  If it's not such an issue I'd expect
> > there's also entertaining suspend ordering issues elsewhere.
> >
> > Typically this stuff isn't a problem for regulators themselves since if
> > they're active by the time I2C is off they normally get suspended by
> > hardware handshakes from the CPU as that enters low power mode - right
> > now we have no regulators at all in mainline that do anything in
> > software on suspend.  We can get a long way by just ignoring what
> > happens to regulators over suspend (there is some work to do here but
> > it's orthogonal to this sort of issue).
> >
> > _______________________________________________
> > linux-arm-kernel mailing list
> > linux-arm-kernel at lists.infradead.org
> > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
> >
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20110125/5ffddccf/attachment-0001.html>

^ permalink raw reply	[flat|nested] 140+ messages in thread

end of thread, other threads:[~2011-02-07  6:13 UTC | newest]

Thread overview: 140+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-01-24  2:01 [PATCH v2 00/28] Updates for Tegra support in 2.6.39 Colin Cross
2011-01-24  2:01 ` [PATCH v2 01/28] ARM: tegra: irq: Rename gic pointers to avoid conflicts Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-26  7:10   ` Colin Cross
2011-01-26  7:10     ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 02/28] ARM: gic: Add functions to save and restore gic state Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-02-01 13:03   ` Russell King - ARM Linux
2011-02-01 13:03     ` Russell King - ARM Linux
2011-01-24  2:01 ` [PATCH v2 03/28] ARM: gic: Export irq chip functions Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  6:39   ` Santosh Shilimkar
2011-01-24  6:39     ` Santosh Shilimkar
2011-01-24  2:01 ` [PATCH v2 04/28] ARM: mm: cache-l2x0: Add support for re-enabling l2x0 Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-25 15:14   ` Catalin Marinas
2011-01-25 15:14     ` Catalin Marinas
2011-01-25 15:41     ` Russell King - ARM Linux
2011-01-25 15:41       ` Russell King - ARM Linux
2011-01-25 18:14       ` Catalin Marinas
2011-01-25 18:14         ` Catalin Marinas
2011-01-25 18:32         ` Santosh Shilimkar
2011-01-25 18:32           ` Santosh Shilimkar
2011-01-25 18:39           ` Will Deacon
     [not found]           ` <-8932138696981683633@unknownmsgid>
2011-02-04 23:32             ` Colin Cross
2011-02-04 23:32               ` Colin Cross
2011-02-04 23:32               ` Colin Cross
2011-02-04 23:43               ` Russell King - ARM Linux
2011-02-04 23:43                 ` Russell King - ARM Linux
     [not found]                 ` <20110204234331.GF8732-l+eeeJia6m9vn6HldHNs0ANdhmdF6hFW@public.gmane.org>
2011-02-05  1:44                   ` Colin Cross
2011-02-05  1:44                     ` Colin Cross
2011-02-05  1:44                     ` Colin Cross
     [not found]                     ` <AANLkTi=fHnivHXHnYrQvdP6JWbEA3t1X3DuBxj5gN3H0-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-02-05  7:51                       ` Santosh Shilimkar
2011-02-05  7:51                         ` Santosh Shilimkar
2011-02-05  7:51                         ` Santosh Shilimkar
     [not found]                         ` <1bebe4b5c8590059b70a146d5486fa6a-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-02-05  9:47                           ` Russell King - ARM Linux
2011-02-05  9:47                             ` Russell King - ARM Linux
2011-02-05  9:47                             ` Russell King - ARM Linux
2011-02-05 10:41                             ` [PATCH v2 04/28] ARM: mm: cache-l2x0: Add support forre-enabling l2x0 Santosh Shilimkar
2011-02-05 10:41                               ` Santosh Shilimkar
2011-02-05 16:36                               ` Colin Cross
2011-02-05 16:36                                 ` Colin Cross
     [not found]                                 ` <AANLkTik_r4k_5o+F47vRbGPcWLwfgHgWqhym49XfhBZ9-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-02-07  6:13                                   ` Santosh Shilimkar
2011-02-07  6:13                                     ` Santosh Shilimkar
2011-02-07  6:13                                     ` Santosh Shilimkar
2011-01-24  2:01 ` [PATCH v2 05/28] ARM: tegra: Centralize macros to define debug uart base Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 06/28] ARM: tegra: Add api to control internal powergating Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 07/28] ARM: tegra: irqs: Update irq list Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 08/28] ARM: tegra: Add prototypes for subsystem suspend functions Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 09/28] ARM: tegra: clock: Suspend fixes, and add new clocks Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 10/28] ARM: tegra: pinmux: Add missing drive pingroups and fix suspend Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 11/28] ARM: tegra: timer: Add idle and suspend support to timers Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 12/28] ARM: tegra: irq: Add support for suspend wake sources Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 13/28] ARM: tegra: Add suspend and hotplug support Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  7:31   ` Colin Cross
2011-01-24  7:31     ` Colin Cross
2011-01-24  9:07   ` Russell King - ARM Linux
2011-01-24  9:07     ` Russell King - ARM Linux
2011-01-24  9:26     ` Colin Cross
2011-01-24  9:26       ` Colin Cross
2011-01-24 10:15       ` Russell King - ARM Linux
2011-01-24 10:15         ` Russell King - ARM Linux
2011-01-24  2:01 ` [PATCH v2 14/28] ARM: tegra: irq: Add set_wake and set_type support for suspend Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 15/28] ARM: tegra: irq: Add debugfs file to show wake irqs Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-27 19:45   ` Stephen Boyd
2011-01-27 19:45     ` Stephen Boyd
2011-01-24  2:01 ` [PATCH v2 16/28] ARM: tegra: irq: Implement retrigger Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 17/28] ARM: tegra: gpio: Add support for waking from suspend Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 18/28] ARM: tegra: add CPU_IDLE driver Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-27 20:38   ` Stephen Boyd
2011-01-27 20:38     ` Stephen Boyd
2011-01-27 21:51     ` Colin Cross
2011-01-27 21:51       ` Colin Cross
2011-01-27 22:07       ` Colin Cross
2011-01-27 22:07         ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 19/28] ARM: tegra: iomap: Add missing devices, fix use of SZ_8, SZ_64 Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 20/28] ARM: tegra: cpufreq: Disable cpufreq during suspend Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24 14:41   ` Mark Brown
2011-01-24 14:41     ` Mark Brown
2011-01-24 18:50     ` Colin Cross
2011-01-24 18:50       ` Colin Cross
2011-01-24 19:35       ` Mark Brown
2011-01-24 19:35         ` Mark Brown
2011-01-24 19:52         ` Colin Cross
2011-01-24 19:52           ` Colin Cross
2011-01-24 20:26           ` Mark Brown
2011-01-24 20:26             ` Mark Brown
2011-01-24 20:52             ` Colin Cross
2011-01-24 20:52               ` Colin Cross
2011-01-24 21:08               ` Mark Brown
2011-01-24 21:08                 ` Mark Brown
2011-01-24 21:24                 ` Colin Cross
2011-01-24 21:24                   ` Colin Cross
2011-01-25  4:26         ` Kyungmin Park
2011-01-25  4:26           ` Kyungmin Park
2011-01-24  2:01 ` [PATCH v2 21/28] ARM: tegra: Allow overriding arch_reset Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 22/28] ARM: tegra: dma: Fix critical data corruption bugs Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 23/28] ARM: tegra: add tegra_defconfig Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 24/28] ARM: tegra: Use writel_relaxed in tegra_init_cache Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 25/28] ARM: tegra: suspend: Save protected aperture across LP0 Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 26/28] ARM: tegra: suspend: protect suspend functions with CONFIG_PM Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 27/28] ARM: tegra: enable emc clock updates after lp0 Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24  2:01 ` [PATCH v2 28/28] ARM: tegra: clock: Add forward reference to struct clk Colin Cross
2011-01-24  2:01   ` Colin Cross
2011-01-24 10:26   ` Russell King - ARM Linux
2011-01-24 10:26     ` Russell King - ARM Linux
2011-01-25  1:23     ` Colin Cross
2011-01-25  1:23       ` Colin Cross
2011-01-26  3:19   ` Grant Likely
2011-01-26  3:19     ` Grant Likely
2011-01-26  3:21     ` Colin Cross
2011-01-26  3:21       ` Colin Cross
2011-01-26  4:16       ` Grant Likely
2011-01-26  4:16         ` Grant Likely
2011-01-25 10:56 [PATCH v2 20/28] ARM: tegra: cpufreq: Disable cpufreq during suspend MyungJoo Ham
2011-01-25 12:40 ` Mark Brown
2011-01-25 12:40   ` Mark Brown

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.