* Re: [PATCH v2] arm64: dts: qcom: sc8280xp: fix apps_smmu irq [not found] <20220712140009.20765-1-quic_ppareek@quicinc.com> @ 2022-07-14 7:17 ` Sai Prakash Ranjan 2022-07-15 8:08 ` Sai Prakash Ranjan 0 siblings, 1 reply; 2+ messages in thread From: Sai Prakash Ranjan @ 2022-07-14 7:17 UTC (permalink / raw) To: Parikshit Pareek, Andy Gross, Bjorn Andersson Cc: Rajendra Nayak, Prasanna Kumar, linux-arm-msm, Manivannan Sadhasivam, Johan Hovold, Dmitry Baryshkov, Linux Kernel Mailing List 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@quicinc.com> > --- > 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. Thanks, Sai ^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH v2] arm64: dts: qcom: sc8280xp: fix apps_smmu irq 2022-07-14 7:17 ` [PATCH v2] arm64: dts: qcom: sc8280xp: fix apps_smmu irq Sai Prakash Ranjan @ 2022-07-15 8:08 ` Sai Prakash Ranjan 0 siblings, 0 replies; 2+ messages in thread From: Sai Prakash Ranjan @ 2022-07-15 8:08 UTC (permalink / raw) To: Parikshit Pareek, Andy Gross, Bjorn Andersson Cc: Rajendra Nayak, Prasanna Kumar, linux-arm-msm, Manivannan Sadhasivam, Johan Hovold, Dmitry Baryshkov, Linux Kernel Mailing List 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@quicinc.com> >> --- >> 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 ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-07-15 8:09 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20220712140009.20765-1-quic_ppareek@quicinc.com>
2022-07-14 7:17 ` [PATCH v2] arm64: dts: qcom: sc8280xp: fix apps_smmu irq Sai Prakash Ranjan
2022-07-15 8:08 ` Sai Prakash Ranjan
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®