Hi Bjorn, Thanks for the reminder, I will send patches to update these entries. Thanks. On Fri, 2021-11-05 at 15:29 -0500, Bjorn Helgaas wrote: > We have two MediaTek drivers: pcie-mediatek.c, which claims: > > .compatible = "mediatek,mt2701-pcie" > .compatible = "mediatek,mt7623-pcie" > .compatible = "mediatek,mt2712-pcie" > .compatible = "mediatek,mt7622-pcie" > .compatible = "mediatek,mt7629-pcie" > > and pcie-mediatek-gen3.c, which claims: > > .compatible = "mediatek,mt8192-pcie" > > The Kconfig text does not distinguish them. Can somebody update > these > entries so they do? It's nice if we can mention model numbers or > product names that a user would recognize. > > config PCIE_MEDIATEK > tristate "MediaTek PCIe controller" > depends on ARCH_MEDIATEK || COMPILE_TEST > depends on OF > depends on PCI_MSI_IRQ_DOMAIN > help > Say Y here if you want to enable PCIe controller support on > MediaTek SoCs. > > config PCIE_MEDIATEK_GEN3 > tristate "MediaTek Gen3 PCIe controller" > depends on ARCH_MEDIATEK || COMPILE_TEST > depends on PCI_MSI_IRQ_DOMAIN > help > Adds support for PCIe Gen3 MAC controller for MediaTek > SoCs. > This PCIe controller is compatible with Gen3, Gen2 and Gen1 > speed, > and support up to 256 MSI interrupt numbers for > multi-function devices. > > Say Y here if you want to enable Gen3 PCIe controller > support on > MediaTek SoCs. > > Both drivers are also named "mtk-pcie" and use the same internal > "mtk_" prefix on structs and functions. Not a *huge* problem, but > not > really ideal either. > > Bjorn