Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1455768

Re: [PATCH] arm64: dts: rockchip: Provide power-domains for sd/sdio/emmc controllers

From Heiko Stübner <heiko@sntech.de>
Newsgroups linux.kernel
Subject Re: [PATCH] arm64: dts: rockchip: Provide power-domains for sd/sdio/emmc controllers
Date 2016-08-03 14:10 +0200
Message-ID <s239L-2mN-33@gated-at.bofh.it> (permalink)
References <s1h57-3Xb-5@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Hi Shawn,

Am Montag, 1. August 2016, 16:41:14 schrieb Shawn Lin:
> The PM domain will be "powered on" once we have registered our PM domain
> with genpd and when the device (dw_mmc etc) gets attached to the domain.
> The attach occurs when the platform bus'->probe() function get invoked
> for the device.
> 
> Due to this, we should be able to deploy genpd support by adding the
> phandle of related power-domains without first having to deploying
> runtime PM support in the related drivers.
> 
> Let's do this in case of taking risk of accessing controller when the
> genpd is in "power off" status as the genpd will turn off all the
> power domains which are the referenced by drivers.
> 
> Signed-off-by: Shawn Lin <shawn.lin@rock-chips.com>

you're missing the subnodes below power: power-controller defining the power-
domains, - including the needed clocks. Thus currently you end up defering 
probe of the emmc and friends indefinitly.

Ideally do it in this patch together with the assignment.

Thanks
Heiko

Back to linux.kernel | Previous | NextPrevious in thread | Find similar | Unroll thread


Thread

[PATCH] arm64: dts: rockchip: Provide power-domains for sd/sdio/emmc controllers Shawn Lin <shawn.lin@rock-chips.com> - 2016-08-01 10:50 +0200
  Re: [PATCH] arm64: dts: rockchip: Provide power-domains for sd/sdio/emmc controllers Heiko Stübner <heiko@sntech.de> - 2016-08-03 14:10 +0200

csiph-web