* [PATCH 1/2] mt76: mt7921: fix the base of PCIe interrupt
@ 2021-02-26 8:23 sean.wang
2021-02-26 8:23 ` [PATCH 2/2] mt76: mt7921: fix the base of the dynamic remap sean.wang
2021-02-26 11:09 ` [PATCH 1/2] mt76: mt7921: fix the base of PCIe interrupt Lorenzo Bianconi
0 siblings, 2 replies; 3+ messages in thread
From: sean.wang @ 2021-02-26 8:23 UTC (permalink / raw)
To: nbd, lorenzo.bianconi
Cc: robin.chiu, ch.yeh, Eric.Liang, sean.wang, linux-wireless,
posh.sun, YN.Chen, linux-mediatek, Soul.Huang, Stella.Chang
From: Sean Wang <sean.wang@mediatek.com>
Should use 0x10000 as the base to operate PCIe interrupt according
to the vendor reference driver.
Fixes: ffa1bf97425b ("mt76: mt7921: introduce PM support")
Co-developed-by: YN Chen <YN.Chen@mediatek.com>
Signed-off-by: YN Chen <YN.Chen@mediatek.com>
Signed-off-by: Sean Wang <sean.wang@mediatek.com>
---
drivers/net/wireless/mediatek/mt76/mt7921/pci.c | 4 ++--
drivers/net/wireless/mediatek/mt76/mt7921/regs.h | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/net/wireless/mediatek/mt76/mt7921/pci.c b/drivers/net/wireless/mediatek/mt76/mt7921/pci.c
index 3e430c04c97e..def8b7816609 100644
--- a/drivers/net/wireless/mediatek/mt76/mt7921/pci.c
+++ b/drivers/net/wireless/mediatek/mt76/mt7921/pci.c
@@ -137,7 +137,7 @@ static int mt7921_pci_probe(struct pci_dev *pdev,
mt76_wr(dev, MT_WFDMA0_HOST_INT_ENA, 0);
- mt7921_l1_wr(dev, MT_PCIE_MAC_INT_ENABLE, 0xff);
+ mt76_wr(dev, MT_PCIE_MAC_INT_ENABLE, 0xff);
ret = devm_request_irq(mdev->dev, pdev->irq, mt7921_irq_handler,
IRQF_SHARED, KBUILD_MODNAME, dev);
@@ -248,7 +248,7 @@ static int mt7921_pci_resume(struct pci_dev *pdev)
return 0;
/* enable interrupt */
- mt7921_l1_wr(dev, MT_PCIE_MAC_INT_ENABLE, 0xff);
+ mt76_wr(dev, MT_PCIE_MAC_INT_ENABLE, 0xff);
mt7921_irq_enable(dev, MT_INT_RX_DONE_ALL | MT_INT_TX_DONE_ALL |
MT_INT_MCU_CMD);
diff --git a/drivers/net/wireless/mediatek/mt76/mt7921/regs.h b/drivers/net/wireless/mediatek/mt76/mt7921/regs.h
index 419add751980..c4bd31cc81cb 100644
--- a/drivers/net/wireless/mediatek/mt76/mt7921/regs.h
+++ b/drivers/net/wireless/mediatek/mt76/mt7921/regs.h
@@ -391,7 +391,7 @@
#define MT_HW_CHIPID 0x70010200
#define MT_HW_REV 0x70010204
-#define MT_PCIE_MAC_BASE 0x74030000
+#define MT_PCIE_MAC_BASE 0x10000
#define MT_PCIE_MAC(ofs) (MT_PCIE_MAC_BASE + (ofs))
#define MT_PCIE_MAC_INT_ENABLE MT_PCIE_MAC(0x188)
--
2.25.1
_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH 2/2] mt76: mt7921: fix the base of the dynamic remap
2021-02-26 8:23 [PATCH 1/2] mt76: mt7921: fix the base of PCIe interrupt sean.wang
@ 2021-02-26 8:23 ` sean.wang
2021-02-26 11:09 ` [PATCH 1/2] mt76: mt7921: fix the base of PCIe interrupt Lorenzo Bianconi
1 sibling, 0 replies; 3+ messages in thread
From: sean.wang @ 2021-02-26 8:23 UTC (permalink / raw)
To: nbd, lorenzo.bianconi
Cc: robin.chiu, ch.yeh, Eric.Liang, sean.wang, linux-wireless,
posh.sun, YN.Chen, linux-mediatek, Soul.Huang, Stella.Chang
From: Sean Wang <sean.wang@mediatek.com>
We should change the base for the dynamic remap into another one, because
the current base (0xe0000) have been the one used to operate the device
ownership.
Fixes: 163f4d22c118 ("mt76: mt7921: add MAC support")
Co-developed-by: YN Chen <YN.Chen@mediatek.com>
Signed-off-by: YN Chen <YN.Chen@mediatek.com>
Signed-off-by: Sean Wang <sean.wang@mediatek.com>
---
drivers/net/wireless/mediatek/mt76/mt7921/regs.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/wireless/mediatek/mt76/mt7921/regs.h b/drivers/net/wireless/mediatek/mt76/mt7921/regs.h
index c4bd31cc81cb..f45f05c794cd 100644
--- a/drivers/net/wireless/mediatek/mt76/mt7921/regs.h
+++ b/drivers/net/wireless/mediatek/mt76/mt7921/regs.h
@@ -364,11 +364,11 @@
#define MT_INFRA_CFG_BASE 0xfe000
#define MT_INFRA(ofs) (MT_INFRA_CFG_BASE + (ofs))
-#define MT_HIF_REMAP_L1 MT_INFRA(0x260)
+#define MT_HIF_REMAP_L1 MT_INFRA(0x24c)
#define MT_HIF_REMAP_L1_MASK GENMASK(15, 0)
#define MT_HIF_REMAP_L1_OFFSET GENMASK(15, 0)
#define MT_HIF_REMAP_L1_BASE GENMASK(31, 16)
-#define MT_HIF_REMAP_BASE_L1 0xe0000
+#define MT_HIF_REMAP_BASE_L1 0x40000
#define MT_SWDEF_BASE 0x41f200
#define MT_SWDEF(ofs) (MT_SWDEF_BASE + (ofs))
--
2.25.1
_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 1/2] mt76: mt7921: fix the base of PCIe interrupt
2021-02-26 8:23 [PATCH 1/2] mt76: mt7921: fix the base of PCIe interrupt sean.wang
2021-02-26 8:23 ` [PATCH 2/2] mt76: mt7921: fix the base of the dynamic remap sean.wang
@ 2021-02-26 11:09 ` Lorenzo Bianconi
1 sibling, 0 replies; 3+ messages in thread
From: Lorenzo Bianconi @ 2021-02-26 11:09 UTC (permalink / raw)
To: sean.wang
Cc: robin.chiu, ch.yeh, Eric.Liang, linux-wireless, posh.sun, YN.Chen,
Stella.Chang, linux-mediatek, lorenzo.bianconi, Soul.Huang, nbd
[-- Attachment #1.1: Type: text/plain, Size: 2178 bytes --]
> From: Sean Wang <sean.wang@mediatek.com>
>
> Should use 0x10000 as the base to operate PCIe interrupt according
> to the vendor reference driver.
>
> Fixes: ffa1bf97425b ("mt76: mt7921: introduce PM support")
> Co-developed-by: YN Chen <YN.Chen@mediatek.com>
> Signed-off-by: YN Chen <YN.Chen@mediatek.com>
> Signed-off-by: Sean Wang <sean.wang@mediatek.com>
For the series:
Tested-by: Lorenzo Bianconi <lorenzo@kernel.org>
> ---
> drivers/net/wireless/mediatek/mt76/mt7921/pci.c | 4 ++--
> drivers/net/wireless/mediatek/mt76/mt7921/regs.h | 2 +-
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/net/wireless/mediatek/mt76/mt7921/pci.c b/drivers/net/wireless/mediatek/mt76/mt7921/pci.c
> index 3e430c04c97e..def8b7816609 100644
> --- a/drivers/net/wireless/mediatek/mt76/mt7921/pci.c
> +++ b/drivers/net/wireless/mediatek/mt76/mt7921/pci.c
> @@ -137,7 +137,7 @@ static int mt7921_pci_probe(struct pci_dev *pdev,
>
> mt76_wr(dev, MT_WFDMA0_HOST_INT_ENA, 0);
>
> - mt7921_l1_wr(dev, MT_PCIE_MAC_INT_ENABLE, 0xff);
> + mt76_wr(dev, MT_PCIE_MAC_INT_ENABLE, 0xff);
>
> ret = devm_request_irq(mdev->dev, pdev->irq, mt7921_irq_handler,
> IRQF_SHARED, KBUILD_MODNAME, dev);
> @@ -248,7 +248,7 @@ static int mt7921_pci_resume(struct pci_dev *pdev)
> return 0;
>
> /* enable interrupt */
> - mt7921_l1_wr(dev, MT_PCIE_MAC_INT_ENABLE, 0xff);
> + mt76_wr(dev, MT_PCIE_MAC_INT_ENABLE, 0xff);
> mt7921_irq_enable(dev, MT_INT_RX_DONE_ALL | MT_INT_TX_DONE_ALL |
> MT_INT_MCU_CMD);
>
> diff --git a/drivers/net/wireless/mediatek/mt76/mt7921/regs.h b/drivers/net/wireless/mediatek/mt76/mt7921/regs.h
> index 419add751980..c4bd31cc81cb 100644
> --- a/drivers/net/wireless/mediatek/mt76/mt7921/regs.h
> +++ b/drivers/net/wireless/mediatek/mt76/mt7921/regs.h
> @@ -391,7 +391,7 @@
> #define MT_HW_CHIPID 0x70010200
> #define MT_HW_REV 0x70010204
>
> -#define MT_PCIE_MAC_BASE 0x74030000
> +#define MT_PCIE_MAC_BASE 0x10000
> #define MT_PCIE_MAC(ofs) (MT_PCIE_MAC_BASE + (ofs))
> #define MT_PCIE_MAC_INT_ENABLE MT_PCIE_MAC(0x188)
>
> --
> 2.25.1
>
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
[-- Attachment #2: Type: text/plain, Size: 170 bytes --]
_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-02-26 11:10 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-02-26 8:23 [PATCH 1/2] mt76: mt7921: fix the base of PCIe interrupt sean.wang
2021-02-26 8:23 ` [PATCH 2/2] mt76: mt7921: fix the base of the dynamic remap sean.wang
2021-02-26 11:09 ` [PATCH 1/2] mt76: mt7921: fix the base of PCIe interrupt Lorenzo Bianconi
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox