linux-kernel - Re: [PATCH v2] arm64: dts: qcom: sc8280xp: fix apps

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Fri, 15 Jul 2022 13:38:58 +0530
From:   Sai Prakash Ranjan <quic_saipraka@...cinc.com>
To:     Parikshit Pareek <quic_ppareek@...cinc.com>,
        Andy Gross <agross@...nel.org>,
        Bjorn Andersson <bjorn.andersson@...aro.org>
CC:     Rajendra Nayak <quic_rjendra@...cinc.com>,
        Prasanna Kumar <quic_kprasan@...cinc.com>,
        <linux-arm-msm@...r.kernel.org>,
        "Manivannan Sadhasivam" <manivannan.sadhasivam@...aro.org>,
        Johan Hovold <johan@...nel.org>,
        Dmitry Baryshkov <dmitry.baryshkov@...aro.org>,
        "Linux Kernel Mailing List" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2] arm64: dts: qcom: sc8280xp: fix apps_smmu irq

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@...cinc.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