Thread (43 messages) 43 messages, 7 authors, 2023-08-03

RE: [PATCH v2 00/10] stmmac: Add Loongson platform support

From: Jose Abreu <Jose.Abreu@synopsys.com>
Date: 2023-07-27 09:03:07
Also in: loongarch

From: Feiyang Chen <redacted>
Date: Thu, Jul 27, 2023 at 08:15:44
Extend stmmac functions and macros for Loongson DWMAC.
Add LS7A support for dwmac_loongson.

Feiyang Chen (10):
  net: stmmac: Pass stmmac_priv and chan in some callbacks
  net: stmmac: dwmac1000: Allow platforms to choose some register
    offsets
  net: stmmac: dwmac1000: Add multi-channel support
  net: stmmac: dwmac1000: Add 64-bit DMA support
  net: stmmac: dwmac1000: Add Loongson register definitions
  net: stmmac: Add Loongson HWIF entry
  net: stmmac: dwmac-loongson: Add LS7A support
  net: stmmac: dwmac-loongson: Disable flow control for GMAC
  net: stmmac: dwmac-loongson: Add 64-bit DMA and multi-vector support
  net: stmmac: dwmac-loongson: Add GNET support
I took a quick look at your patches and I'm thinking whether this is the correct way to go.
You are mixing up the stmmac generic layer by adding the Loongson HWIF entry.
The whole idea of HWIF was to have it independent of vendor specific logic.

Can you devise another alternative without mixing up the HWIF?

Thanks,
Jose
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help