Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.32 as permitted sender) client-ip=23.128.96.32;
Date:   Mon, 18 Sep 2023 19:17:34 -0400
From:   Richard Acayan <mailingradian@gmail.com>
To:     Konrad Dybcio <konrad.dybcio@linaro.org>
Cc:     Andy Gross <agross@kernel.org>,
        Bjorn Andersson <andersson@kernel.org>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Conor Dooley <conor+dt@kernel.org>,
        Marijn Suijten <marijn.suijten@somainline.org>,
        linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH] arm64: dts: qcom: sdm670: Fix pdc mapping
Message-ID: <ZQjajpc4VLslnGgv@radian>
References: <20230818-topic-670_pdc_fix-v1-1-1ba025041de7@linaro.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20230818-topic-670_pdc_fix-v1-1-1ba025041de7@linaro.org>
Precedence: bulk

On Fri, Aug 18, 2023 at 01:19:09PM +0200, Konrad Dybcio wrote:
> As pointed out by Richard, I missed a non-continuity in one of the ranges.
> Fix it.
>
> Reported-by: Richard Acayan <mailingradian@gmail.com>
> Fixes: b51ee205dc4f ("arm64: dts: qcom: sdm670: Add PDC")
> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
> ---

Just a friendly reminder to everyone that this patch is still here.

Also, I see no issues arising from this patch.

Acked-by: Richard Acayan <mailingradian@gmail.com>

>  arch/arm64/boot/dts/qcom/sdm670.dtsi | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sdm670.dtsi b/arch/arm64/boot/dts/qcom/sdm670.dtsi
> index 84cd2e39266f..ba2043d67370 100644
> --- a/arch/arm64/boot/dts/qcom/sdm670.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm670.dtsi
> @@ -1328,7 +1328,8 @@ pdc: interrupt-controller@b220000 {
>  			compatible = "qcom,sdm670-pdc", "qcom,pdc";
>  			reg = <0 0x0b220000 0 0x30000>;
>  			qcom,pdc-ranges = <0 480 40>, <41 521 7>, <49 529 4>,
> -					  <54 534 24>, <79 559 30>, <115 630 7>;
> +					  <54 534 24>, <79 559 15>, <94 609 15>,
> +					  <115 630 7>;
>  			#interrupt-cells = <2>;
>  			interrupt-parent = <&intc>;
>  			interrupt-controller;
>
> ---
> base-commit: 7271b2a530428b879443b274c191b34536a4ea11
> change-id: 20230818-topic-670_pdc_fix-d07fbdf4d179
>
> Best regards,
> -- 
> Konrad Dybcio <konrad.dybcio@linaro.org>
>