On 13/05/22 00:47, Andrew Lunn wrote: >> Basically, the cpu-subsystems of Prestera 98DX25xx (AC5) and Prestera 98DX35xx (AC5X) are the same. > Great, thanks for the conformation. > >> The DTSI Chris proposed had the PCIe portion removed. >> We have PCIe support for AC5/AC5X so we would obviously like to >> include this portion in both the DTSI and as a patch to the Armada8K >> PCIe driver. > So you can add the needed node to the .dtsi as part of the patch to > the pci-aardvark.c driver. That sounds O.K. I deliberately left the pci stuff out of my submission to keep things simple and because I have no way of testing it. Eventually we (allied telesis) are planning a product variant that will need a 2nd switch connected via pci so I'll be revisiting the pci stuff then (unless someone else beats me to it).