Re: [PATCH 3/2] arm64: dts: qcom: sc7180: Remove ipa interconnect node

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 20.04.22 5:56, Bjorn Andersson wrote:
On Thu 14 Apr 17:58 PDT 2022, Stephen Boyd wrote:

This device node is unused now that we've removed the driver that
consumed it in the kernel. Drop the unused node to save some space.


I'm expecting that merging patch 3 and 4 will work, but cause sync_state
to not happen until the driver changes are merged.

Can you confirm my expectation? And perhaps confirm that it's fine for
Georgi to pick the driver changes independently of the dts changes...

I have picked the driver changes, as the boot failure definitely needs to
be addressed. The sync-state might not happen until we have the DT changes
merged, as the framework is matching the count of probed drivers with the
count of providers in DT.

Cc: Alex Elder <elder@xxxxxxxxxx>
Cc: Taniya Das <quic_tdas@xxxxxxxxxxx>
Cc: Mike Tipton <quic_mdtipton@xxxxxxxxxxx>
Cc: Georgi Djakov <djakov@xxxxxxxxxx>
Signed-off-by: Stephen Boyd <swboyd@xxxxxxxxxxxx>

Acked-by: Georgi Djakov <djakov@xxxxxxxxxx>

Thanks,
Georgi

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

diff --git a/arch/arm64/boot/dts/qcom/sc7180.dtsi b/arch/arm64/boot/dts/qcom/sc7180.dtsi
index e1c46b80f14a..1ff96ef30e3f 100644
--- a/arch/arm64/boot/dts/qcom/sc7180.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7180.dtsi
@@ -1421,13 +1421,6 @@ mmss_noc: interconnect@1740000 {
  			qcom,bcm-voters = <&apps_bcm_voter>;
  		};
- ipa_virt: interconnect@1e00000 {
-			compatible = "qcom,sc7180-ipa-virt";
-			reg = <0 0x01e00000 0 0x1000>;
-			#interconnect-cells = <2>;
-			qcom,bcm-voters = <&apps_bcm_voter>;
-		};
-
  		ipa: ipa@1e40000 {
  			compatible = "qcom,sc7180-ipa";
--
https://chromeos.dev





[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux