Skip to content

ARM: davinci: PM: cleanup init, add DT support

Kevin Hilman linux-arm-kernel 3 versions – New

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest posting 2/20 of 3 patches 0 repliesNew
  2. v2 posting 1/23 patches 0 repliesSuperseded
  3. v12 patches 0 repliesSuperseded

Patches in v2 posting 2/2 No applies recorded yet

None of the 3 patches the cover letter announces are in this archive.

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v2 yet.

Read the v2 thread

Cover letter [PATCH v2 0/3] ARM: davinci: PM: cleanup init, add DT support
Quoted 11 lines · Sekhar Nori

Hi Kevin,

Quoted 5 lines · Kevin Hilman

This series removes the fake platform_device used to initialize PM on legacy davinci platforms, and also adds PM support for DT platforms.

Tested with legacy boot on da850-evm, and DT boot on da850-lcdk. Used RTC to wake from suspend-to-RAM: e.g. rtcwake -m mem -s 4 -d /dev/rtc0

With these patches applied, I saw the following randconfig build error.

Quoted 8 lines

arch/arm/mach-davinci/built-in.o: In function `davinci_pm_init':
pm_domain.c:(.init.text+0x13d8): undefined reference to da8xx_get_mem_ctlr' pm_domain.c:(.init.text+0x14bc): undefined reference to da8xx_syscfg1_base'
make: *** [vmlinux] Error 1

Attached is the defconfig. I did not check to see if your patches introduced it or if this is something that always existed and never caught.

It looks like a new problem caused by moving those calls into pm.c.

The problem is that currently pm.o can be built even when da850 support is not enabled, which is the case in your defconfig.

I'll hae a look at how to cleanup that dependency.

Kevin