Re: [PATCH v2] arm64: dts: qcom: sc8280xp: fix apps_smmu irq

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

 



On 7/14/2022 12:47 PM, Sai Prakash Ranjan wrote:
Hi Parikshit,

On 7/12/2022 7:30 PM, Parikshit Pareek wrote:
Wrong values have been introduced for interrupts property. Fix those
ones, and correct the mapping of context banks to irq number.

Fixes: 152d1faf1e2f ("arm64: dts: qcom: add SC8280XP platform")
Signed-off-by: Parikshit Pareek <quic_ppareek@xxxxxxxxxxx>
---
  arch/arm64/boot/dts/qcom/sc8280xp.dtsi | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/sc8280xp.dtsi b/arch/arm64/boot/dts/qcom/sc8280xp.dtsi
index 7945cbb57bb4..1276a833251e 100644
--- a/arch/arm64/boot/dts/qcom/sc8280xp.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc8280xp.dtsi
@@ -1580,7 +1580,6 @@
                       <GIC_SPI 412 IRQ_TYPE_LEVEL_HIGH>,
                       <GIC_SPI 421 IRQ_TYPE_LEVEL_HIGH>,
                       <GIC_SPI 706 IRQ_TYPE_LEVEL_HIGH>,
-                     <GIC_SPI 424 IRQ_TYPE_LEVEL_HIGH>,
                       <GIC_SPI 423 IRQ_TYPE_LEVEL_HIGH>,
                       <GIC_SPI 424 IRQ_TYPE_LEVEL_HIGH>,
                       <GIC_SPI 425 IRQ_TYPE_LEVEL_HIGH>,
@@ -1591,6 +1590,7 @@
                       <GIC_SPI 693 IRQ_TYPE_LEVEL_HIGH>,
                       <GIC_SPI 694 IRQ_TYPE_LEVEL_HIGH>,
                       <GIC_SPI 695 IRQ_TYPE_LEVEL_HIGH>,
+                     <GIC_SPI 696 IRQ_TYPE_LEVEL_HIGH>,
                       <GIC_SPI 410 IRQ_TYPE_LEVEL_HIGH>,
                       <GIC_SPI 411 IRQ_TYPE_LEVEL_HIGH>,
                       <GIC_SPI 420 IRQ_TYPE_LEVEL_HIGH>,

Cross checking with the IP doc, 696 is not the only one missing, couple of other interrupts are also missing, below is the list.

Interrupt 689 and 706 are not present in the IP doc, remove it.
Interrupts 696, 697, 716, 913 are missing, I see this patch adds 696, add 697 as well.
Interrupt 890 is not a context interrupt, remove it.



Checking offline with Parikshit, looks like adding all these entries results in interrupt selftest failures.
So for now, we can just have the original patch merged till these failures are debugged further with internal teams.

Thanks,
Sai



[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