Skip to content

Commit 022d43b

Browse files
palitrini
authored andcommitted
pci: mediatek: Use PCI_CONF1_EXT_ADDRESS() macro
PCI mediatek driver uses extended format of Config Address for PCI Configuration Mechanism #1 but with cleared Enable bit. So use new U-Boot macro PCI_CONF1_EXT_ADDRESS() with clearing PCI_CONF1_ENABLE bit and remove old custom driver address macros. Signed-off-by: Pali Rohár <pali@kernel.org> Reviewed-by: Simon Glass <sjg@chromium.org>
1 parent f031f07 commit 022d43b

File tree

1 file changed

+8
-9
lines changed

1 file changed

+8
-9
lines changed

drivers/pci/pcie_mediatek.c

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -41,10 +41,6 @@
4141
#define PCIE_BAR_ENABLE BIT(0)
4242
#define PCIE_REVISION_ID BIT(0)
4343
#define PCIE_CLASS_CODE (0x60400 << 8)
44-
#define PCIE_CONF_REG(regn) (((regn) & GENMASK(7, 2)) | \
45-
((((regn) >> 8) & GENMASK(3, 0)) << 24))
46-
#define PCIE_CONF_ADDR(regn, bdf) \
47-
(PCIE_CONF_REG(regn) | (bdf))
4844

4945
/* MediaTek specific configuration registers */
5046
#define PCIE_FTS_NUM 0x70c
@@ -147,8 +143,11 @@ static int mtk_pcie_config_address(const struct udevice *udev, pci_dev_t bdf,
147143
uint offset, void **paddress)
148144
{
149145
struct mtk_pcie *pcie = dev_get_priv(udev);
146+
u32 val;
150147

151-
writel(PCIE_CONF_ADDR(offset, bdf), pcie->base + PCIE_CFG_ADDR);
148+
val = PCI_CONF1_EXT_ADDRESS(PCI_BUS(bdf), PCI_DEV(bdf),
149+
PCI_FUNC(bdf), offset) & ~PCI_CONF1_ENABLE;
150+
writel(val, pcie->base + PCIE_CFG_ADDR);
152151
*paddress = pcie->base + PCIE_CFG_DATA + (offset & 3);
153152

154153
return 0;
@@ -330,7 +329,6 @@ static void mtk_pcie_port_free(struct mtk_pcie_port *port)
330329
static int mtk_pcie_startup_port(struct mtk_pcie_port *port)
331330
{
332331
struct mtk_pcie *pcie = port->pcie;
333-
u32 slot = PCI_DEV(port->slot << 11);
334332
u32 val;
335333
int err;
336334

@@ -357,13 +355,14 @@ static int mtk_pcie_startup_port(struct mtk_pcie_port *port)
357355
writel(PCIE_CLASS_CODE | PCIE_REVISION_ID, port->base + PCIE_CLASS);
358356

359357
/* configure FC credit */
360-
writel(PCIE_CONF_ADDR(PCIE_FC_CREDIT, slot),
361-
pcie->base + PCIE_CFG_ADDR);
358+
val = PCI_CONF1_EXT_ADDRESS(0, port->slot, 0, PCIE_FC_CREDIT) & ~PCI_CONF1_ENABLE;
359+
writel(val, pcie->base + PCIE_CFG_ADDR);
362360
clrsetbits_le32(pcie->base + PCIE_CFG_DATA, PCIE_FC_CREDIT_MASK,
363361
PCIE_FC_CREDIT_VAL(0x806c));
364362

365363
/* configure RC FTS number to 250 when it leaves L0s */
366-
writel(PCIE_CONF_ADDR(PCIE_FTS_NUM, slot), pcie->base + PCIE_CFG_ADDR);
364+
val = PCI_CONF1_EXT_ADDRESS(0, port->slot, 0, PCIE_FTS_NUM) & ~PCI_CONF1_ENABLE;
365+
writel(val, pcie->base + PCIE_CFG_ADDR);
367366
clrsetbits_le32(pcie->base + PCIE_CFG_DATA, PCIE_FTS_NUM_MASK,
368367
PCIE_FTS_NUM_L0(0x50));
369368

0 commit comments

Comments
 (0)