| Message ID | 20230818-topic-670_pdc_fix-v1-1-1ba025041de7@linaro.org |
|---|---|
| State | New |
| Headers | show |
| Series | arm64: dts: qcom: sdm670: Fix pdc mapping | expand |
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> >
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;
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> --- arch/arm64/boot/dts/qcom/sdm670.dtsi | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) --- base-commit: 7271b2a530428b879443b274c191b34536a4ea11 change-id: 20230818-topic-670_pdc_fix-d07fbdf4d179 Best regards,