• Ulf Hansson's avatar
    Revert "firmware: arm_scmi: Add clock management to the SCMI power domain" · 3c665633
    Ulf Hansson authored
    This reverts commit a3b884ce ("firmware: arm_scmi: Add clock management
    to the SCMI power domain").
    
    Using the GENPD_FLAG_PM_CLK tells genpd to gate/ungate the consumer
    device's clock(s) during runtime suspend/resume through the PM clock API.
    More precisely, in genpd_runtime_resume() the clock(s) for the consumer
    device would become ungated prior to the driver-level ->runtime_resume()
    callbacks gets invoked.
    
    This behaviour isn't a good fit for all platforms/drivers. For example, a
    driver may need to make some preparations of its device in its
    ->runtime_resume() callback, like calling clk_set_rate() before the
    clock(s) should be ungated. In these cases, it's easier to let the clock(s)
    to be managed solely by the driver, rather than at the PM domain level.
    
    For these reasons, let's drop the use GENPD_FLAG_PM_CLK for the SCMI PM
    domain, as to enable it to be more easily adopted across ARM platforms.
    
    Fixes: a3b884ce ("firmware: arm_scmi: Add clock management to the SCMI power domain")
    Cc: Nicolas Pitre <npitre@baylibre.com>
    Cc: stable@vger.kernel.org
    Signed-off-by: default avatarUlf Hansson <ulf.hansson@linaro.org>
    Tested-by: default avatarPeng Fan <peng.fan@nxp.com>
    Acked-by: default avatarSudeep Holla <sudeep.holla@arm.com>
    Link: https://lore.kernel.org/r/20220919122033.86126-1-ulf.hansson@linaro.org
    3c665633
scmi_pm_domain.c 3.22 KB