On 12/3/2018 9:13 AM, Marc Gonzalez wrote:
On 03/12/2018 17:03, Jeffrey Hugo wrote:
On 12/3/2018 8:53 AM, Marc Gonzalez wrote:
On 03/12/2018 16:47, Jeffrey Hugo wrote:
On 12/3/2018 8:27 AM, Jeffrey Hugo wrote:
On 12/3/2018 8:18 AM, Marc Gonzalez wrote:
+ ufsphy: phy@1da7000 {
+ compatible = "qcom,sdm845-qmp-ufs-phy";
+ reg = <0x1da7000 0x18c>;
+ #address-cells = <1>;
+ #size-cells = <1>;
+ ranges;
+ clock-names =
+ "ref_clk_src",
+ "ref_clk",
+ "ref_aux_clk";
+ clocks =
+ <&rpmcc 0>,
+ <&gcc GCC_UFS_CLKREF_CLK>,
Also, "GCC_UFS_CLKREF_CLK" is not defined in
include/dt-bindings/clock/qcom,gcc-msm8998.h which is probably the cause
of your -2 error listed above.
If it were undefined, dtc would have kicked me in the nads :-)
$ git grep GCC_UFS_CLKREF_CLK include
include/dt-bindings/clock/qcom,gcc-msm8996.h:#define GCC_UFS_CLKREF_CLK 215
include/dt-bindings/clock/qcom,gcc-msm8998.h:#define GCC_UFS_CLKREF_CLK 168
Not present in v4.20-rc5 nor linux-next. Did you add this yourself? If
so, did you also update the driver?
You're right. I applied Bjorn's patch series.
[PATCH 0/3] clk: qcom: gcc-msm8998: Fixes and clkref clocks
Ok. Then its non-obvious to me why you are getting the ref clock error.
--
Jeffrey Hugo
Qualcomm Datacenter Technologies as an affiliate of Qualcomm
Technologies, Inc.
Qualcomm Technologies, Inc. is a member of the
Code Aurora Forum, a Linux Foundation Collaborative Project.
