Re: [PATCH V2 2/2] arm64: dts: qcom: sc7180: Remove QUP-CORE ICC path

2021-03-18 Thread Matthias Kaehlcke
On Thu, Mar 18, 2021 at 04:40:09PM +0530, Roja Rani Yarubandi wrote:
> We had introduced the QUP-CORE ICC path to put proxy votes from
> QUP wrapper on behalf of earlycon, if other users of QUP-CORE turn
> off this clock before the real console is probed, unclocked access
> to HW was seen from earlycon.
> 
> With ICC sync state support proxy votes are no longer need as ICC
> will ensure that the default bootloader votes are not removed until
> all it's consumer are probed.
> 
> We can safely remove ICC path for QUP-CORE clock from QUP wrapper
> device.
> 
> Signed-off-by: Roja Rani Yarubandi 
> Signed-off-by: Akash Asthana 

Reviewed-by: Matthias Kaehlcke 


[PATCH V2 2/2] arm64: dts: qcom: sc7180: Remove QUP-CORE ICC path

2021-03-18 Thread Roja Rani Yarubandi
We had introduced the QUP-CORE ICC path to put proxy votes from
QUP wrapper on behalf of earlycon, if other users of QUP-CORE turn
off this clock before the real console is probed, unclocked access
to HW was seen from earlycon.

With ICC sync state support proxy votes are no longer need as ICC
will ensure that the default bootloader votes are not removed until
all it's consumer are probed.

We can safely remove ICC path for QUP-CORE clock from QUP wrapper
device.

Signed-off-by: Roja Rani Yarubandi 
Signed-off-by: Akash Asthana 
---
Changes in V2:
 - No chage

 arch/arm64/boot/dts/qcom/sc7180.dtsi | 4 
 1 file changed, 4 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/sc7180.dtsi 
b/arch/arm64/boot/dts/qcom/sc7180.dtsi
index 83fbb481cae5..2709051740d1 100644
--- a/arch/arm64/boot/dts/qcom/sc7180.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7180.dtsi
@@ -768,8 +768,6 @@ qupv3_id_0: geniqup@8c {
#size-cells = <2>;
ranges;
iommus = <_smmu 0x43 0x0>;
-   interconnects = <_virt MASTER_QUP_CORE_0 0 
_virt SLAVE_QUP_CORE_0 0>;
-   interconnect-names = "qup-core";
status = "disabled";
 
i2c0: i2c@88 {
@@ -1059,8 +1057,6 @@ qupv3_id_1: geniqup@ac {
#size-cells = <2>;
ranges;
iommus = <_smmu 0x4c3 0x0>;
-   interconnects = <_virt MASTER_QUP_CORE_1 0 
_virt SLAVE_QUP_CORE_1 0>;
-   interconnect-names = "qup-core";
status = "disabled";
 
i2c6: i2c@a8 {
-- 
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member 
of Code Aurora Forum, hosted by The Linux Foundation