linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] arm64: dts: qcom: sc7180: Fix one forgotten interconnect reference
@ 2020-10-01 21:18 Douglas Anderson
  2020-10-02 16:18 ` Georgi Djakov
  0 siblings, 1 reply; 2+ messages in thread
From: Douglas Anderson @ 2020-10-01 21:18 UTC (permalink / raw)
  To: Bjorn Andersson
  Cc: Douglas Anderson, Andy Gross, Georgi Djakov, Matthias Kaehlcke,
	Rob Herring, Sibi Sankar, devicetree, linux-arm-msm,
	linux-kernel

In commit e23b1220a246 ("arm64: dts: qcom: sc7180: Increase the number
of interconnect cells") we missed increasing the cells on one
interconnect.  That's no bueno.  Fix it.

NOTE: it appears that things aren't totally broken without this fix,
but clearly something isn't going to be working right.  If nothing
else, without this fix I see this in the logs:

  OF: /soc@0/mdss@ae00000: could not get #interconnect-cells for /soc@0/interrupt-controller@17a00000

Fixes: e23b1220a246 ("arm64: dts: qcom: sc7180: Increase the number of interconnect cells")
Signed-off-by: Douglas Anderson <dianders@chromium.org>
---

 arch/arm64/boot/dts/qcom/sc7180.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/sc7180.dtsi b/arch/arm64/boot/dts/qcom/sc7180.dtsi
index 6678f1e8e395..a02776ce77a1 100644
--- a/arch/arm64/boot/dts/qcom/sc7180.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7180.dtsi
@@ -2811,7 +2811,7 @@ mdss: mdss@ae00000 {
 			interrupt-controller;
 			#interrupt-cells = <1>;
 
-			interconnects = <&mmss_noc MASTER_MDP0 &mc_virt SLAVE_EBI1>;
+			interconnects = <&mmss_noc MASTER_MDP0 0 &mc_virt SLAVE_EBI1 0>;
 			interconnect-names = "mdp0-mem";
 
 			iommus = <&apps_smmu 0x800 0x2>;
-- 
2.28.0.709.gb0816b6eb0-goog


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] arm64: dts: qcom: sc7180: Fix one forgotten interconnect reference
  2020-10-01 21:18 [PATCH] arm64: dts: qcom: sc7180: Fix one forgotten interconnect reference Douglas Anderson
@ 2020-10-02 16:18 ` Georgi Djakov
  0 siblings, 0 replies; 2+ messages in thread
From: Georgi Djakov @ 2020-10-02 16:18 UTC (permalink / raw)
  To: Douglas Anderson, Bjorn Andersson
  Cc: Andy Gross, Matthias Kaehlcke, Rob Herring, Sibi Sankar,
	devicetree, linux-arm-msm, linux-kernel

Thanks for the patch!

On 10/2/20 00:18, Douglas Anderson wrote:
> In commit e23b1220a246 ("arm64: dts: qcom: sc7180: Increase the number
> of interconnect cells") we missed increasing the cells on one
> interconnect.  That's no bueno.  Fix it.
> 
> NOTE: it appears that things aren't totally broken without this fix,
> but clearly something isn't going to be working right.  If nothing
> else, without this fix I see this in the logs:
> 
>   OF: /soc@0/mdss@ae00000: could not get #interconnect-cells for /soc@0/interrupt-controller@17a00000
> 
> Fixes: e23b1220a246 ("arm64: dts: qcom: sc7180: Increase the number of interconnect cells")
> Signed-off-by: Douglas Anderson <dianders@chromium.org>

Reviewed-by: Georgi Djakov <georgi.djakov@linaro.org>

BR,
Georgi

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-10-02 16:18 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-01 21:18 [PATCH] arm64: dts: qcom: sc7180: Fix one forgotten interconnect reference Douglas Anderson
2020-10-02 16:18 ` Georgi Djakov

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).