Re: [PATCH v3 3/3] PCI: aardvark: Remove redundant MPS configuration
From: Pali Rohár <pali@kernel.org>
Date: 2025-05-06 17:41:13
Also in:
linux-amlogic, linux-pci, linux-rockchip, lkml
On Wednesday 07 May 2025 01:34:39 Hans Zhang wrote:
quoted hunk ↗ jump to hunk
The Aardvark PCIe controller enforces a fixed 512B payload size via PCI_EXP_DEVCTL_PAYLOAD_512B, overriding hardware capabilities and PCIe core negotiations. Remove explicit MPS overrides (PCI_EXP_DEVCTL_PAYLOAD and PCI_EXP_DEVCTL_PAYLOAD_512B). MPS is now determined by the PCI core during device initialization, leveraging root port configurations and device-specific capabilities. Aligning Aardvark with the unified MPS framework ensures consistency, avoids artificial constraints, and allows the hardware to operate at its maximum supported payload size while adhering to PCIe specifications. Signed-off-by: Hans Zhang <18255117159@163.com> --- drivers/pci/controller/pci-aardvark.c | 2 -- 1 file changed, 2 deletions(-)diff --git a/drivers/pci/controller/pci-aardvark.c b/drivers/pci/controller/pci-aardvark.c index a29796cce420..d8852892994a 100644 --- a/drivers/pci/controller/pci-aardvark.c +++ b/drivers/pci/controller/pci-aardvark.c@@ -549,9 +549,7 @@ static void advk_pcie_setup_hw(struct advk_pcie *pcie) reg = advk_readl(pcie, PCIE_CORE_PCIEXP_CAP + PCI_EXP_DEVCTL); reg &= ~PCI_EXP_DEVCTL_RELAX_EN; reg &= ~PCI_EXP_DEVCTL_NOSNOOP_EN; - reg &= ~PCI_EXP_DEVCTL_PAYLOAD; reg &= ~PCI_EXP_DEVCTL_READRQ; - reg |= PCI_EXP_DEVCTL_PAYLOAD_512B; reg |= PCI_EXP_DEVCTL_READRQ_512B; advk_writel(pcie, reg, PCIE_CORE_PCIEXP_CAP + PCI_EXP_DEVCTL);-- 2.25.1
Please do not remove this code. It is required part of the initialization of the aardvark PCI controller at the specific phase, as defined in the Armada 3700 Functional Specification. There were reported more issues with those Armada PCIe controllers for which were already sent patches to mailing list in last 5 years. But unfortunately not all fixes were taken / applied yet.