On Wed, Dec 21, 2016 at 03:49:35AM -0800, Bjorn Andersson wrote: > As per the device tree binding the apq8064 scm node requires the core > clock to be specified, so add this. > > Cc: John Stultz <john.stu...@linaro.org> > Signed-off-by: Bjorn Andersson <bjorn.anders...@linaro.org> > --- > arch/arm/boot/dts/qcom-apq8064.dtsi | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi > b/arch/arm/boot/dts/qcom-apq8064.dtsi > index 268bd470c865..78bf155a52f3 100644 > --- a/arch/arm/boot/dts/qcom-apq8064.dtsi > +++ b/arch/arm/boot/dts/qcom-apq8064.dtsi > @@ -303,6 +303,9 @@ > firmware { > scm { > compatible = "qcom,scm-apq8064"; > + > + clocks = <&gcc CE3_CORE_CLK>; > + clock-names = "core";
Isn't this supposed to be the DFAB clk? The RPM one? I think that's why we let the clock just fall through optionally before the recent changes that broke this. Regards, Andy