On Thu, 5 Nov 2020 at 05:42, Stephen Boyd <sboyd@xxxxxxxxxx> 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@xxxxxxxxxx> > > --- > > 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. -- With best wishes Dmitry