On Tue, Apr 30, 2024 at 11:28:44PM -0500, Alexandru Gagniuc wrote: > IPQ9574 has four PCIe controllers: two single-lane Gen3, and two > dual-lane Gen3. The controllers are identical from a software You mean to say, 'identical to IPQ8074 Gen3 platform' since you are reusing the 2_9_0 cfg? > perspective, with the differences appearing in the PHYs. > > Add a compatible for the PCIe on IPQ9574. > > Signed-off-by: Alexandru Gagniuc <mr.nuke.me@xxxxxxxxx> But the change looks good to me. Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@xxxxxxxxxx> - Mani > --- > drivers/pci/controller/dwc/pcie-qcom.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/pci/controller/dwc/pcie-qcom.c b/drivers/pci/controller/dwc/pcie-qcom.c > index ea81ff68d433..e61888e6c63d 100644 > --- a/drivers/pci/controller/dwc/pcie-qcom.c > +++ b/drivers/pci/controller/dwc/pcie-qcom.c > @@ -1551,6 +1551,7 @@ static const struct of_device_id qcom_pcie_match[] = { > { .compatible = "qcom,pcie-ipq8064-v2", .data = &cfg_2_1_0 }, > { .compatible = "qcom,pcie-ipq8074", .data = &cfg_2_3_3 }, > { .compatible = "qcom,pcie-ipq8074-gen3", .data = &cfg_2_9_0 }, > + { .compatible = "qcom,pcie-ipq9574", .data = &cfg_2_9_0 }, > { .compatible = "qcom,pcie-msm8996", .data = &cfg_2_3_2 }, > { .compatible = "qcom,pcie-qcs404", .data = &cfg_2_4_0 }, > { .compatible = "qcom,pcie-sa8540p", .data = &cfg_sc8280xp }, > -- > 2.40.1 > -- மணிவண்ணன் சதாசிவம்