On 05/26, Bjorn Andersson wrote: > diff --git a/arch/arm64/boot/dts/qcom/msm8916.dtsi b/arch/arm64/boot/dts/qcom/msm8916.dtsi > index ab3093995ded..33013835639d 100644 > --- a/arch/arm64/boot/dts/qcom/msm8916.dtsi > +++ b/arch/arm64/boot/dts/qcom/msm8916.dtsi > @@ -241,6 +241,8 @@ > clocks = <&gcc GCC_CRYPTO_CLK>, <&gcc GCC_CRYPTO_AXI_CLK>, <&gcc GCC_CRYPTO_AHB_CLK>; > clock-names = "core", "bus", "iface"; > #reset-cells = <1>; > + > + qcom,dload-mode-addr = <0x0 0x193d100>; This is TCSR, so why not a phandle to a node? > }; > }; > > diff --git a/arch/arm64/boot/dts/qcom/msm8996.dtsi b/arch/arm64/boot/dts/qcom/msm8996.dtsi > index 9bc9c857a000..7e64884ab7c7 100644 > --- a/arch/arm64/boot/dts/qcom/msm8996.dtsi > +++ b/arch/arm64/boot/dts/qcom/msm8996.dtsi > @@ -261,6 +261,8 @@ > firmware { > scm { > compatible = "qcom,scm-msm8996"; > + > + qcom,dload-mode-addr = <0x0 0x7b3000>; Also TCSR. Same question. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html