Skip to contentSkip to conversation

arm64: dts: add prng-ee nodes

PATCH In review Vinod Koul 3 participants 6 messages Oct 1, 2018 – Nov 16, 2018

Subsystems (3)
lore
Share & tools
lore

Vinod Koul

author
#0 lore raw

v6 cover arm64: dts: add prng-ee nodes

This adds prng-ee nodes for msm8996 and sdm845

changes in v6:
- Fix comments given by Stan

changes in v5:
- Add more description in patch logs

changes in v4:
- Fix node address on sdm845

changes in v3:
- Add Bjorn's reviewed-by
- Update patch titles to reflect that they add nodes

changes in v2:
- rebase on v4.19-rc1

Vinod Koul (2):
arm64: dts: msm8996: add prng-ee node
arm64: dts: sdm845: add prng-ee node

 arch/arm64/boot/dts/qcom/msm8996.dtsi | 7 +++++++
 arch/arm64/boot/dts/qcom/sdm845.dtsi  | 8 ++++++++
 2 files changed, 15 insertions(+)
Signature · 1 line
2.14.4
author
#1 lore raw

PATCH v6 1/2 arm64: dts: msm8996: add prng-ee node

RNG hardware in 8996 features (Execution Environment) EE for HLOS to use, add the node for prng-ee for msm8996.

---
 arch/arm64/boot/dts/qcom/msm8996.dtsi | 7 +++++++
 1 file changed, 7 insertions(+)
arch/arm64/boot/dts/qcom/msm8996.dtsi +7 -0
index cd3865e7a270..40e52a43f670 100644
@@ -729,6 +729,13 @@
 			};
 		};
 
+		rng: rng@83000 {
+			compatible = "qcom,prng-ee";
+			reg = <0x00083000 0x1000>;
+			clocks = <&gcc GCC_PRNG_AHB_CLK>;
+			clock-names = "core";
+		};
+
 		mmcc: clock-controller@8c0000 {
 			compatible = "qcom,mmcc-msm8996";
 			#clock-cells = <1>;
Signature · 1 line
2.14.4
author
#2 lore raw

PATCH v6 2/2 arm64: dts: sdm845: add prng-ee node

In reply to Vinod Koul

RNG hardware in SDM845 features (Execution Environment) EE for HLOS to use, add the node for prng-ee for sdm845.

---
 arch/arm64/boot/dts/qcom/sdm845.dtsi | 8 ++++++++
 1 file changed, 8 insertions(+)
arch/arm64/boot/dts/qcom/sdm845.dtsi +8 -0
index 0c9a2aa6a1b5..9d8109761fe8 100644
@@ -9,6 +9,7 @@
 #include <dt-bindings/clock/qcom,rpmh.h>
 #include <dt-bindings/interrupt-controller/arm-gic.h>
 #include <dt-bindings/soc/qcom,rpmh-rsc.h>
+#include <dt-bindings/clock/qcom,gcc-sdm845.h>
 
 / {
 	interrupt-parent = <&intc>;
@@ -997,6 +998,13 @@
 			cell-index = <0>;
 		};
 
+		rng: rng@793000 {
+			compatible = "qcom,prng-ee";
+			reg = <0x00793000 0x1000>;
+			clocks = <&gcc GCC_PRNG_AHB_CLK>;
+			clock-names = "core";
+		};
+
 		apss_shared: mailbox@17990000 {
 			compatible = "qcom,sdm845-apss-shared";
 			reg = <0x17990000 0x1000>;
Signature · 1 line
2.14.4
#3 lore raw
In reply to Vinod Koul
Quoted 1 line · Vinod Koul

This adds prng-ee nodes for msm8996 and sdm845

Ping Andy, would appreciate if you can pick these up.

Thanks

Quoted 27 lines

changes in v6:
- Fix comments given by Stan

changes in v5:
- Add more description in patch logs

changes in v4:
- Fix node address on sdm845

changes in v3:
- Add Bjorn's reviewed-by
- Update patch titles to reflect that they add nodes

changes in v2:
- rebase on v4.19-rc1

Vinod Koul (2):
arm64: dts: msm8996: add prng-ee node
arm64: dts: sdm845: add prng-ee node

 arch/arm64/boot/dts/qcom/msm8996.dtsi | 7 +++++++
 arch/arm64/boot/dts/qcom/sdm845.dtsi  | 8 ++++++++
 2 files changed, 15 insertions(+)
Signature · 1 line
2.14.4
Signature · 1 line
~Vinod
#4 lore raw
Quoted 4 lines · Vinod Koul
Quoted 1 line · Vinod Koul

This adds prng-ee nodes for msm8996 and sdm845

Ping Andy, would appreciate if you can pick these up.

Done. I did have to massage the location in dts for both of these patches to keep the address order correct.

Andy

author
#5 lore raw
Quoted 8 lines · Andy Gross
Quoted 4 lines · Vinod Koul
Quoted 1 line · Vinod Koul

This adds prng-ee nodes for msm8996 and sdm845

Ping Andy, would appreciate if you can pick these up.

Done. I did have to massage the location in dts for both of these patches to keep the address order correct.

Thank you Andy, appreciate it.

I have two more series in queue, QCS404 DTS series https://patchwork.kernel.org/project/linux-arm-msm/list/?series=40841
and QCS404 defconfigs https://patchwork.kernel.org/project/linux-arm-msm/list/?series=40853

Would be great if you can review these as well

Regards

Signature · 1 line
~Vinod