| Message ID | 1316844884-21700-1-git-send-email-santosh.shilimkar@ti.com |
|---|---|
| State | New |
| Headers | show |
Hi Santosh, Santosh Shilimkar <santosh.shilimkar@ti.com> writes: > The series adds OMAP4 MPUSS (MPU SubSystem) power management support for > suspend (S2R), CPU hotplug and CPUidle. No need to repost, but can you update the versions in your branch to have an ARM: prefix in the subject per Arnd's recent request? Thanks, Kevin
On Tuesday 04 October 2011 10:35 PM, Kevin Hilman wrote: > Hi Santosh, > > Santosh Shilimkar <santosh.shilimkar@ti.com> writes: > >> The series adds OMAP4 MPUSS (MPU SubSystem) power management support for >> suspend (S2R), CPU hotplug and CPUidle. > > No need to repost, but can you update the versions in your branch to > have an ARM: prefix in the subject per Arnd's recent request? > Done. Branch for_3_2/omap4-mpuss-pm is updated with ARM on all patches. Regards Santosh
Hi Santosh, Santosh Shilimkar <santosh.shilimkar@ti.com> writes: > The series adds OMAP4 MPUSS (MPU SubSystem) power management support for > suspend (S2R), CPU hotplug and CPUidle. Just noticed when doing some final build/sanity testing that there are a few of compile/link warnings/problems when the kernel is built without CONFIG_PM Some warnings: /work/kernel/omap/pm/arch/arm/mach-omap2/omap-wakeupgen.c:203:13: warning: 'irq_save_context' defined but not used /work/kernel/omap/pm/arch/arm/mach-omap2/omap-wakeupgen.c:270:13: warning: 'irq_save_secure_context' defined but not used and some failures (these happen when CONFIG_SUSPEND=n but CONFIG_PM_RUNTIME=y: arch/arm/mach-omap2/built-in.o: In function `skip_l2en': /work/kernel/omap/pm/arch/arm/mach-omap2/sleep44xx.S:323: undefined reference to `cpu_resume' arch/arm/mach-omap2/built-in.o: In function `omap_sram_idle': /work/kernel/omap/pm/arch/arm/mach-omap2/pm34xx.c:429: undefined reference to `cpu_suspend' arch/arm/mach-omap2/built-in.o: In function `skipl2reen': /work/kernel/omap/pm/arch/arm/mach-omap2/sleep34xx.S:534: undefined reference to `cpu_resume' arch/arm/mach-omap2/built-in.o: In function `omap4_enter_lowpower': /work/kernel/omap/pm/arch/arm/mach-omap2/omap-mpuss-lowpower.c:281: undefined reference to `cpu_suspend' Turns out these are not directly related to your series and OMAP3 exhibits the same problems. I just posted a patch[1] to attempt to address these. But could you fixup the first problem and test with [1] to see if things still build/work as expected. Thanks, Kevin [1] http://lists.infradead.org/pipermail/linux-arm-kernel/2011-October/068606.html
Kevin Hilman <khilman@ti.com> writes: > Hi Santosh, > > Santosh Shilimkar <santosh.shilimkar@ti.com> writes: > >> The series adds OMAP4 MPUSS (MPU SubSystem) power management support for >> suspend (S2R), CPU hotplug and CPUidle. > > Just noticed when doing some final build/sanity testing that there are a > few of compile/link warnings/problems when the kernel is built without > CONFIG_PM > > Some warnings: > > /work/kernel/omap/pm/arch/arm/mach-omap2/omap-wakeupgen.c:203:13: warning: 'irq_save_context' defined but not used > /work/kernel/omap/pm/arch/arm/mach-omap2/omap-wakeupgen.c:270:13: warning: 'irq_save_secure_context' defined but not used > > and some failures (these happen when CONFIG_SUSPEND=n but CONFIG_PM_RUNTIME=y: > > arch/arm/mach-omap2/built-in.o: In function `skip_l2en': > /work/kernel/omap/pm/arch/arm/mach-omap2/sleep44xx.S:323: undefined reference to `cpu_resume' > arch/arm/mach-omap2/built-in.o: In function `omap_sram_idle': > /work/kernel/omap/pm/arch/arm/mach-omap2/pm34xx.c:429: undefined reference to `cpu_suspend' > arch/arm/mach-omap2/built-in.o: In function `skipl2reen': > /work/kernel/omap/pm/arch/arm/mach-omap2/sleep34xx.S:534: undefined reference to `cpu_resume' > arch/arm/mach-omap2/built-in.o: In function `omap4_enter_lowpower': > /work/kernel/omap/pm/arch/arm/mach-omap2/omap-mpuss-lowpower.c:281: undefined reference to `cpu_suspend' > > Turns out these are not directly related to your series and OMAP3 > exhibits the same problems. > > I just posted a patch[1] to attempt to address these. > > But could you fixup the first problem and test with [1] to see if things > still build/work as expected. OK, ignore my patch. Arnd already has a fix for these link problems in his randconfig/arm branch. Please just cleanup the wakeupgen compile warnings and update your branch. Kevin
On Saturday 08 October 2011 04:20 AM, Kevin Hilman wrote: > Kevin Hilman <khilman@ti.com> writes: > >> Hi Santosh, >> >> Santosh Shilimkar <santosh.shilimkar@ti.com> writes: >> >>> The series adds OMAP4 MPUSS (MPU SubSystem) power management support for >>> suspend (S2R), CPU hotplug and CPUidle. >> >> Just noticed when doing some final build/sanity testing that there are a >> few of compile/link warnings/problems when the kernel is built without >> CONFIG_PM >> >> Some warnings: >> >> /work/kernel/omap/pm/arch/arm/mach-omap2/omap-wakeupgen.c:203:13: warning: 'irq_save_context' defined but not used >> /work/kernel/omap/pm/arch/arm/mach-omap2/omap-wakeupgen.c:270:13: warning: 'irq_save_secure_context' defined but not used >> Sory I missed it in my builds. [...] > Please just cleanup the wakeupgen compile warnings and update your > branch. > Done !! Updated branch : for_3_2/omap4-mpuss-pm_2 Regards Santosh
Hi Santosh, Santosh Shilimkar <santosh.shilimkar@ti.com> writes: > The series adds OMAP4 MPUSS (MPU SubSystem) power management support for > suspend (S2R), CPU hotplug and CPUidle. There are a few more compile errors when doing OMAP1-only builds. You'll need a way to eliminate the secure calls for OMAP1. This series causes a couple build errors when doing OMAP1-only builds (I used omap1_defconfig): First: /work/kernel/omap/pm/arch/arm/plat-omap/common.c:24:30: fatal error: mach/omap-secure.h: No such file or directory And trying creating a dummy header to see if it would continue to build gives: /work/kernel/omap/pm/arch/arm/plat-omap/common.c: In function 'omap_reserve': /work/kernel/omap/pm/arch/arm/plat-omap/common.c:70:2: error: implicit declaration of function 'omap_secure_ram_reserve_memblock' make[2]: *** [arch/arm/plat-omap/common.o] Error 1 make[1]: *** [arch/arm/plat-omap] Error 2 Kevin
Santosh Shilimkar <santosh.shilimkar@ti.com> writes: > On Monday 10 October 2011 11:31 PM, Kevin Hilman wrote: >> Hi Santosh, >> >> Santosh Shilimkar <santosh.shilimkar@ti.com> writes: >> >>> The series adds OMAP4 MPUSS (MPU SubSystem) power management support for >>> suspend (S2R), CPU hotplug and CPUidle. >> >> There are a few more compile errors when doing OMAP1-only builds. >> You'll need a way to eliminate the secure calls for OMAP1. >> >> This series causes a couple build errors when doing OMAP1-only builds >> (I used omap1_defconfig): >> > OK. Thanks for catching it. > >> First: >> >> /work/kernel/omap/pm/arch/arm/plat-omap/common.c:24:30: fatal error: mach/omap-secure.h: No such file or directory >> >> And trying creating a dummy header to see if it would continue to build gives: >> >> /work/kernel/omap/pm/arch/arm/plat-omap/common.c: In function 'omap_reserve': >> /work/kernel/omap/pm/arch/arm/plat-omap/common.c:70:2: error: implicit declaration of function 'omap_secure_ram_reserve_memblock' >> make[2]: *** [arch/arm/plat-omap/common.o] Error 1 >> make[1]: *** [arch/arm/plat-omap] Error 2 >> > For OMAP1 build, omap_secure_ram_reserve_memblock() definition is > needed since common.c is built for OMAP1 as well. > The trivial change in the end of this email fixes the omap1 build. Looks good. > Have folded that change in original patch and updated the OMAP4 > PM branch - for_3_2/omap4-mpuss-pm_2 Thanks. > Regards > Santosh > P.S : Even with this fix, omap1_defconfig build breaks because of USB. > I removed USB from build to check rest of the build and it was > fine then. OK. Kevin
The series adds OMAP4 MPUSS (MPU SubSystem) power management support for suspend (S2R), CPU hotplug and CPUidle. This is a repost with minor changelog updates, re-basing against 3.1-rc6, adding reiwed-by/tested-by etc and re-ordering of errata patch. The main change is the errata i688 is cleanly separated and kept as last patch in the series as per Tony's suggestion. Now Russell's L2 suspend series is merged in his for-next branch and CPU PM notifiers series pull request is out, most of the dependencies are taken care. The irq/core patch is also in Thomas's 3.2 queue. An integrated branch with these branches merged can be found here [1]. The series is tested on OMAP4430 SDP for suspend, hotplug and CPUidle with OMAP4 GP and HS (secure) devices. The following changes since commit b6fd41e29dea9c6753b1843a77e50433e6123bcb: Linux 3.1-rc6 (2011-09-12 14:02:02 -0700) are available in the git repository at: git://gitorious.org/omap-sw-develoment/linux-omap-dev.git for_3_2/omap4-mpuss-pm Santosh Shilimkar (24): OMAP4: Use WARN_ON() instead of BUG_ON() with graceful exit OMAP4: Export omap4_get_base*() rather than global address pointers OMAP4: PM: Add SAR RAM support OMAP4: PM: Keep static dep between MPUSS-EMIF and MPUSS-L3/L4 and DUCATI-L3 OMAP4: PM: Avoid omap4_pm_init() on OMAP4430 ES1.0 OMAP4: PM: Initialise all the clockdomains to supported states OMAP: Add Secure HAL and monitor mode API infrastructure. OMAP: PM: Add support to allocate the memory for secure RAM OMAP4: PM: Add WakeupGen module as OMAP gic_arch_extn OMAP4: PM: Add CPUX OFF mode support OMAP4: Remove __INIT from omap_secondary_startup() to re-use it for hotplug. OMAP4: PM: Program CPU1 to hit OFF when off-lined OMAP4: PM: CPU1 wakeup workaround from Low power modes OMAP4: PM: Use custom omap_do_wfi() for default idle. OMAP4: suspend: Add MPUSS power domain RETENTION support OMAP4: Remove un-used do_wfi() macro. OMAP4: PM: Add WakeupGen and secure GIC low power support OMAP4: PM: Add L2X0 cache lowpower support OMAP4: PM: Add MPUSS power domain OSWR support OMAP4: PM: Add power domain statistics support OMAP4: PM: Add CPUidle support OMAP4: cpuidle: Switch to gptimer from twd in deeper C-states. OMAP3: CPUidle: Make use of CPU PM notifiers OMAP4: Fix errata i688 with MPU interconnect barriers. arch/arm/mach-omap2/Kconfig | 21 + arch/arm/mach-omap2/Makefile | 16 +- arch/arm/mach-omap2/cpuidle34xx.c | 15 + arch/arm/mach-omap2/cpuidle44xx.c | 218 +++++++++++ arch/arm/mach-omap2/include/mach/barriers.h | 31 ++ arch/arm/mach-omap2/include/mach/omap-secure.h | 57 +++ arch/arm/mach-omap2/include/mach/omap-wakeupgen.h | 39 ++ arch/arm/mach-omap2/include/mach/omap4-common.h | 71 +++- arch/arm/mach-omap2/omap-headsmp.S | 5 - arch/arm/mach-omap2/omap-hotplug.c | 14 +- arch/arm/mach-omap2/omap-mpuss-lowpower.c | 398 ++++++++++++++++++++ arch/arm/mach-omap2/omap-secure.c | 81 ++++ arch/arm/mach-omap2/{omap44xx-smc.S => omap-smc.S} | 23 ++ arch/arm/mach-omap2/omap-smp.c | 45 +++ arch/arm/mach-omap2/omap-wakeupgen.c | 387 +++++++++++++++++++ arch/arm/mach-omap2/omap4-common.c | 88 +++++- arch/arm/mach-omap2/omap4-sar-layout.h | 50 +++ arch/arm/mach-omap2/pm.h | 1 + arch/arm/mach-omap2/pm44xx.c | 156 ++++++++- arch/arm/mach-omap2/sleep44xx.S | 378 +++++++++++++++++++ arch/arm/plat-omap/common.c | 3 + arch/arm/plat-omap/include/plat/omap44xx.h | 1 + arch/arm/plat-omap/sram.c | 43 ++- 23 files changed, 2091 insertions(+), 50 deletions(-) create mode 100644 arch/arm/mach-omap2/cpuidle44xx.c create mode 100644 arch/arm/mach-omap2/include/mach/barriers.h create mode 100644 arch/arm/mach-omap2/include/mach/omap-secure.h create mode 100644 arch/arm/mach-omap2/include/mach/omap-wakeupgen.h create mode 100644 arch/arm/mach-omap2/omap-mpuss-lowpower.c create mode 100644 arch/arm/mach-omap2/omap-secure.c rename arch/arm/mach-omap2/{omap44xx-smc.S => omap-smc.S} (70%) create mode 100644 arch/arm/mach-omap2/omap-wakeupgen.c create mode 100644 arch/arm/mach-omap2/omap4-sar-layout.h create mode 100644 arch/arm/mach-omap2/sleep44xx.S Regards, Santosh [1] git://gitorious.org/omap-sw-develoment/linux-omap-dev.git for_3_2/omap4_mpuss_pm-integrated