Re: [PATCH v3 08/10] pmdomain: samsung: selectively handle enforced sync_state

Unconditionally calling of_genpd_sync_state() causes issues on
platforms with child domains as the parent domain will be turned off
before the child domain was even registered during boot.

This in particular is an issue for the upcoming Google gs101 support -
all operations on child domains registered after the parent domain
misbehave.

Add a flag to the probe data to be able to sync_state conditionally
only, and enable that flag on the two platforms currently supported by
this driver.

Signed-off-by: André Draszik <andre.draszik@linaro.org>

---
v2:
* use bool for need_early_sync_state (Krzysztof)
---
 drivers/pmdomain/samsung/exynos-pm-domains.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/drivers/pmdomain/samsung/exynos-pm-domains.c b/drivers/pmdomain/samsung/exynos-pm-domains.c
index 638d286b57f716140b2401092415644a6805870e..15a1582aa92103a07335eb681600d9415369fefd 100644
--- a/drivers/pmdomain/samsung/exynos-pm-domains.c
+++ b/drivers/pmdomain/samsung/exynos-pm-domains.c

@@ -20,6 +20,7 @@
 struct exynos_pm_domain_config {
        /* Value for LOCAL_PWR_CFG and STATUS fields for each domain */
        u32 local_pwr_cfg;
+       bool need_early_sync_state;
 };

 /*

@@ -69,10 +70,12 @@ static int exynos_pd_power_off(struct generic_pm_domain *domain)

 static const struct exynos_pm_domain_config exynos4210_cfg = {
        .local_pwr_cfg          = 0x7,
+       .need_early_sync_state  = true,
 };

 static const struct exynos_pm_domain_config exynos5433_cfg = {
        .local_pwr_cfg          = 0xf,
+       .need_early_sync_state  = true,
 };

 static const struct of_device_id exynos_pm_domain_of_match[] = {

@@ -179,7 +182,7 @@ static int exynos_pd_probe(struct platform_device *pdev)
         * reset during boot. As a temporary hack to manage this, let's enforce
         * a sync_state.
         */
-       if (!ret)
+       if (pm_domain_cfg->need_early_sync_state && !ret)
                of_genpd_sync_state(np);