Skip to content

Add runtime PM support for clocks (on Exynos SoC example)

Tobias Jakobi linux-arm-kernel linux-clk linux-pm linux-samsung-soc 2 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 5 patches 0 repliesNew
  2. v2 posting 1/20 of 5 patches 0 repliesSuperseded

Patches in v2 posting 2/2 No applies recorded yet

None of the 5 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/5] Add runtime PM support for clocks (on Exynos SoC example)

Hey Marek,

Quoted 16 lines · Marek Szyprowski

Hi Tobias,

Quoted 12 lines · Tobias Jakobi

Hello Marek,

I'm using the patches from the v4.8-clocks-pm-v2 branch plus the ones from the v4.8-clocks-pm-v2 branch on top of 4.8.0.

I see some warnings on boot coming from driver core. It appears that the warnings are actually meaningful, since IOMMUs stop working completly. E.g. if I modprobe s5p-mfc later, firmware loading fails because apparantly the IOMMU domain isn't online.

Quoted 2 lines
WARNING: CPU: 0 PID: 1 at drivers/base/core.c:356
device_links_driver_bound+0x124/0x12c
Quoted 8 lines
Quoted 2 lines

I added some debug printk() to device_links_driver_bound(), to show the link status. Apparantly it is always DEVICE_LINK_AVAILABLE.

Please note that some additional patches are needed to get IOMMU working properly with both runtime-pm patches and deferred probe, which feature in turn is needed to get it working after adding clocks-pm changes. I will send such patch soon (as a new version of IOMMU deferred probe support patches were posted a few days ago).

I see. I thought that this was supposed to work with the patches in v4.8-clocks-pm-v2.

I picked the patches in this range (endpoints included).
- arm: dma-mapping: Don't override dma_ops in arch_setup_dma_ops()
- clocks: exynos5433: add runtime pm support

And this range in v4.8-iommu-pm-v4>
- driver core: Add a wrapper around __device_release_driver()
- iommu/exynos: Add proper runtime pm support

Anyway, looking forward to test the new patchset.

- Tobias

Quoted 4 lines
Quoted 2 lines

(...)

Best regards

Read the cover letter in its thread