Re: [PATCH 1/2] arm64: dts: qcom: sc8280xp: add several missing pdc map entries

[Pengyu Luo]

On Mon, Apr 20, 2026 at 4:32 PM Konrad Dybcio
<konrad.dybcio@xxxxxxxxxxxxxxxx> wrote:
>
> On 4/19/26 7:32 PM, Pengyu Luo wrote:
> > pdc 215, 256, 257 are missing, but we can find tlmm pin 103, 84, 90
> > are mapped to them respectively, so add the map entries from pdc to
> > gic. These entries are reversed from .data section of qcgpio.sys
> >
> > Signed-off-by: Pengyu Luo <mitltlatltl@xxxxxxxxx>
> > ---
>
> Reviewed-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxxxxxxxx>
>
> Konrad
>
> The below change on top will fully align it with the data in the docs
> (no functional change)
>

Glad to know. Could you please help to check the tlmm map too? When I
was parsing the binary, I found

tlmm 65535 => pdc 70 => gic 520
tlmm 65535 => pdc 174 => gic 733
tlmm 65535 => pdc 175 => gic 734
tlmm 65535 => pdc 176 => gic 735
tlmm 65535 => pdc 177 => gic 736
tlmm 65535 => pdc 178 => gic 737
tlmm 65535 => pdc 184 => gic 743
tlmm 65535 => pdc 185 => gic 744
tlmm 65535 => pdc 186 => gic 745
tlmm 65535 => pdc 187 => gic 746
tlmm 65535 => pdc 188 => gic 747
tlmm 65535 => pdc 194 => gic 753
tlmm 65535 => pdc 195 => gic 754
tlmm 65535 => pdc 196 => gic 755
tlmm 65535 => pdc 197 => gic 756
tlmm 65535 => pdc 198 => gic 757
tlmm 65535 => pdc 199 => gic 416
tlmm 65535 => pdc 204 => gic 462
tlmm 65535 => pdc 205 => gic 264

If 65536 means the pin is missing, I will send v2 to remove the tlmm
map together with the pdc removal.

Best wishes,
Pengyu


> (yes that's a removal of one irq mapping)
>
> diff --git a/arch/arm64/boot/dts/qcom/sc8280xp.dtsi b/arch/arm64/boot/dts/qcom/sc8280xp.dtsi
> index 761f229e8f47..23e80c765384 100644
> --- a/arch/arm64/boot/dts/qcom/sc8280xp.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc8280xp.dtsi
> @@ -5310,7 +5310,7 @@ pdc: interrupt-controller@b220000 {
>                                           <66 438 3>,
>                                           <69 86 1>,
>                                           <70 520 54>,
> -                                         <124 609 28>,
> +                                         <124 609 27>,
>                                           <159 638 1>,
>                                           <160 720 8>,
>                                           <168 801 1>,