| Message ID | 20220519214133.1728979-2-vladimir.zapolskiy@linaro.org |
|---|---|
| State | Accepted |
| Commit | 103dd2338bbff567bce7acd00fc5a09c806b38ec |
| Headers | show |
| Series | clk: qcom: Fix topology around titan_top power domain | expand |
On Thu, 19 May 2022 at 23:41, Vladimir Zapolskiy <vladimir.zapolskiy@linaro.org> wrote: > > On SDM845 two found VFE GDSC power domains shall not be operated, if > titan top is turned off, thus the former power domains will be set as > subdomains by a GDSC registration routine. > > Fixes: 78412c262004 ("clk: qcom: Add camera clock controller driver for SDM845") > Signed-off-by: Vladimir Zapolskiy <vladimir.zapolskiy@linaro.org> > --- > drivers/clk/qcom/camcc-sdm845.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/drivers/clk/qcom/camcc-sdm845.c b/drivers/clk/qcom/camcc-sdm845.c > index be3f95326965..27d44188a7ab 100644 > --- a/drivers/clk/qcom/camcc-sdm845.c > +++ b/drivers/clk/qcom/camcc-sdm845.c > @@ -1534,6 +1534,8 @@ static struct clk_branch cam_cc_sys_tmr_clk = { > }, > }; > > +static struct gdsc titan_top_gdsc; > + > static struct gdsc bps_gdsc = { > .gdscr = 0x6004, > .pd = { > @@ -1567,6 +1569,7 @@ static struct gdsc ife_0_gdsc = { > .name = "ife_0_gdsc", > }, > .flags = POLL_CFG_GDSCR, > + .parent = &titan_top_gdsc.pd, > .pwrsts = PWRSTS_OFF_ON, > }; > > @@ -1576,6 +1579,7 @@ static struct gdsc ife_1_gdsc = { > .name = "ife_1_gdsc", > }, > .flags = POLL_CFG_GDSCR, > + .parent = &titan_top_gdsc.pd, > .pwrsts = PWRSTS_OFF_ON, > }; > > -- > 2.33.0 > Reviewed-by: Robert Foss <robert.foss@linaro.org>
diff --git a/drivers/clk/qcom/camcc-sdm845.c b/drivers/clk/qcom/camcc-sdm845.c index be3f95326965..27d44188a7ab 100644 --- a/drivers/clk/qcom/camcc-sdm845.c +++ b/drivers/clk/qcom/camcc-sdm845.c @@ -1534,6 +1534,8 @@ static struct clk_branch cam_cc_sys_tmr_clk = { }, }; +static struct gdsc titan_top_gdsc; + static struct gdsc bps_gdsc = { .gdscr = 0x6004, .pd = { @@ -1567,6 +1569,7 @@ static struct gdsc ife_0_gdsc = { .name = "ife_0_gdsc", }, .flags = POLL_CFG_GDSCR, + .parent = &titan_top_gdsc.pd, .pwrsts = PWRSTS_OFF_ON, }; @@ -1576,6 +1579,7 @@ static struct gdsc ife_1_gdsc = { .name = "ife_1_gdsc", }, .flags = POLL_CFG_GDSCR, + .parent = &titan_top_gdsc.pd, .pwrsts = PWRSTS_OFF_ON, };
On SDM845 two found VFE GDSC power domains shall not be operated, if titan top is turned off, thus the former power domains will be set as subdomains by a GDSC registration routine. Fixes: 78412c262004 ("clk: qcom: Add camera clock controller driver for SDM845") Signed-off-by: Vladimir Zapolskiy <vladimir.zapolskiy@linaro.org> --- drivers/clk/qcom/camcc-sdm845.c | 4 ++++ 1 file changed, 4 insertions(+)