Message ID | 20201023131925.334864-6-dmitry.baryshkov@linaro.org (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | [v2,1/5] dt-bindings: regulator: fixed: provide bindings using power domain | expand |
Quoting Dmitry Baryshkov (2020-10-23 06:19:25) > On SM8250 MMCX power domain is required to access MMDS_GDSC registers. > This power domain is expressed as mmcx-supply regulator property. Use > this regulator as MDSS_GDSC supply. > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> > --- > drivers/clk/qcom/dispcc-sm8250.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/clk/qcom/dispcc-sm8250.c b/drivers/clk/qcom/dispcc-sm8250.c > index 07a98d3f882d..588575e1169d 100644 > --- a/drivers/clk/qcom/dispcc-sm8250.c > +++ b/drivers/clk/qcom/dispcc-sm8250.c > @@ -963,6 +963,7 @@ static struct gdsc mdss_gdsc = { > }, > .pwrsts = PWRSTS_OFF_ON, > .flags = HW_CTRL, > + .supply = "mmcx", > }; > Can this patch be applied to clk tree or will it break the tree without the dts/regulator bits in place?
On Thu, 5 Nov 2020 at 05:42, Stephen Boyd <sboyd@kernel.org> wrote: > > Quoting Dmitry Baryshkov (2020-10-23 06:19:25) > > On SM8250 MMCX power domain is required to access MMDS_GDSC registers. > > This power domain is expressed as mmcx-supply regulator property. Use > > this regulator as MDSS_GDSC supply. > > > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> > > --- > > drivers/clk/qcom/dispcc-sm8250.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/drivers/clk/qcom/dispcc-sm8250.c b/drivers/clk/qcom/dispcc-sm8250.c > > index 07a98d3f882d..588575e1169d 100644 > > --- a/drivers/clk/qcom/dispcc-sm8250.c > > +++ b/drivers/clk/qcom/dispcc-sm8250.c > > @@ -963,6 +963,7 @@ static struct gdsc mdss_gdsc = { > > }, > > .pwrsts = PWRSTS_OFF_ON, > > .flags = HW_CTRL, > > + .supply = "mmcx", > > }; > > > > Can this patch be applied to clk tree or will it break the tree without > the dts/regulator bits in place? It can be applied to clk tree. Regulator bits are already accepted by Mark Brown.
Quoting Dmitry Baryshkov (2020-10-23 06:19:25) > On SM8250 MMCX power domain is required to access MMDS_GDSC registers. > This power domain is expressed as mmcx-supply regulator property. Use > this regulator as MDSS_GDSC supply. > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> > --- Applied to clk-next
diff --git a/drivers/clk/qcom/dispcc-sm8250.c b/drivers/clk/qcom/dispcc-sm8250.c index 07a98d3f882d..588575e1169d 100644 --- a/drivers/clk/qcom/dispcc-sm8250.c +++ b/drivers/clk/qcom/dispcc-sm8250.c @@ -963,6 +963,7 @@ static struct gdsc mdss_gdsc = { }, .pwrsts = PWRSTS_OFF_ON, .flags = HW_CTRL, + .supply = "mmcx", }; static struct clk_regmap *disp_cc_sm8250_clocks[] = {
On SM8250 MMCX power domain is required to access MMDS_GDSC registers. This power domain is expressed as mmcx-supply regulator property. Use this regulator as MDSS_GDSC supply. Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> --- drivers/clk/qcom/dispcc-sm8250.c | 1 + 1 file changed, 1 insertion(+)