[PATCH 2/2] pci: layerscape: Fix the BARs disable function
Priyanka Jain
priyanka.jain at nxp.com
Wed Jan 22 10:52:52 CET 2020
>-----Original Message-----
>From: Z.q. Hou <zhiqiang.hou at nxp.com>
>Sent: Tuesday, December 17, 2019 3:41 PM
>To: u-boot at lists.denx.de; Priyanka Jain <priyanka.jain at nxp.com>; M.h.
>Lian <minghuan.lian at nxp.com>
>Cc: Z.q. Hou <zhiqiang.hou at nxp.com>
>Subject: [PATCH 2/2] pci: layerscape: Fix the BARs disable function
>
>From: Hou Zhiqiang <Zhiqiang.Hou at nxp.com>
>
>There is not any difference for disabling BARs in RC mode between PCIe
>controllers with and without SRIOV.
>
>Fixes: 80afc63fc342 ("pci: layerscape: add pci driver based on DM")
>Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou at nxp.com>
>---
> drivers/pci/pcie_layerscape.c | 11 -----------
> 1 file changed, 11 deletions(-)
>
>diff --git a/drivers/pci/pcie_layerscape.c
>b/drivers/pci/pcie_layerscape.c index
>96533cb2d9..d8ca7e71f8 100644
>--- a/drivers/pci/pcie_layerscape.c
>+++ b/drivers/pci/pcie_layerscape.c
>@@ -312,17 +312,6 @@ static void ls_pcie_drop_msg_tlp(struct ls_pcie
>*pcie)
> /* Disable all bars in RC mode */
> static void ls_pcie_disable_bars(struct ls_pcie *pcie) {
>- u32 sriov;
>-
>- sriov = in_le32(pcie->dbi + PCIE_SRIOV);
>-
>- /*
>- * TODO: For PCIe controller with SRIOV, the method to disable bars
>- * is different and more complex, so will add later.
>- */
Patch description and code comment statement looks contradictory.
Is method to disable BAR same or different for with and without SRIOV.
>- if (PCI_EXT_CAP_ID(sriov) == PCI_EXT_CAP_ID_SRIOV)
>- return;
>-
> dbi_writel(pcie, 0, PCIE_CS2_OFFSET + PCI_BASE_ADDRESS_0);
> dbi_writel(pcie, 0, PCIE_CS2_OFFSET + PCI_BASE_ADDRESS_1);
> dbi_writel(pcie, 0xfffffffe, PCIE_CS2_OFFSET + PCI_ROM_ADDRESS1);
>--
>2.17.1
Priyanka
More information about the U-Boot
mailing list