On Tue, May 19, 2020 at 10:53:28AM +0200, Geert Uytterhoeven wrote: > Hi Wolfram, > > On Tue, May 19, 2020 at 10:46 AM Wolfram Sang > <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> wrote: > > > > Can't we put it before the custom clk_enable()? And then clean up > > > > further like this to have the main clock only controlled via RPM? > > > > > > I understand what you want to achieve, but to allow that to work we > > > need to consider the below things first. > > > > > > 1. If the driver is built with CONFIG_PM unset, then runtime PM > > > doesn't work and hence the clock won't be managed by a PM domain. > > > 2. If there is a platform configuration where a PM domain (genpd) > > > isn't going to be attached, then the clock needs to be managed locally > > > in the driver. > > > > Similar to what Geert responded, for Renesas SDHI both is always true > > AFAIU. Geert? > > For Renesas SDHI, both are always false ;-) > I.e. CONFIG_PM is always set, genpd is always attached. I hope it was obvious that this is what I meant :)
Attachment:
signature.asc
Description: PGP signature
