[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <885a8904-9412-411d-9995-7d3ff350f309@loongson.cn>
Date: Tue, 19 Dec 2023 20:35:17 +0800
From: Yanteng Si <siyanteng@...ngson.cn>
To: Serge Semin <fancer.lancer@...il.com>
Cc: andrew@...n.ch, hkallweit1@...il.com, peppe.cavallaro@...com,
alexandre.torgue@...s.st.com, joabreu@...opsys.com, Jose.Abreu@...opsys.com,
chenhuacai@...ngson.cn, linux@...linux.org.uk, guyinggang@...ngson.cn,
netdev@...r.kernel.org, loongarch@...ts.linux.dev,
chris.chenfeiyang@...il.com
Subject: Re: [PATCH v6 0/9] stmmac: Add Loongson platform support
在 2023/12/14 23:15, Serge Semin 写道:
> Hi Yanteng
>
> On Wed, Dec 13, 2023 at 06:12:22PM +0800, Yanteng Si wrote:
>> v6:
>>
>> * Refer to Serge's suggestion:
>> - Add new platform feature flag:
>> include/linux/stmmac.h:
>> +#define STMMAC_FLAG_HAS_LGMAC BIT(13)
>>
>> - Add the IRQs macros specific to the Loongson Multi-channels GMAC:
>> drivers/net/ethernet/stmicro/stmmac/dwmac_dma.h:
>> +#define DMA_INTR_ENA_NIE_LOONGSON 0x00060000 /* Normal Loongson Tx/Rx Summary */
>> #define DMA_INTR_ENA_NIE 0x00010000 /* Normal Summary */
>> ...
>>
>> - Drop all of redundant changes that don't require the
>> prototypes being converted to accepting the stmmac_priv
>> pointer.
>>
>> * Refer to andrew's suggestion:
>> - Drop white space changes.
>> - break patch up into lots of smaller parts.
>> Some small patches have been put into another series as a preparation
>> see <https://lore.kernel.org/loongarch/cover.1702289232.git.siyanteng@loongson.cn/T/#t>
>>
>> *note* : This series of patches relies on the three small patches above.
>> * others
>> - Drop irq_flags changes.
>> - Changed patch order.
> Thanks for submitting the updated series. I'll have a closer look at
> it on the next week.
I have prepared a new patch version and will CC you soon, so you can go
straight
to the v7.
Thank,
Yanteng
>
> -Serge(y)
>
>>
>>
>> v4 -> v5:
>>
>> * Remove an ugly and useless patch (fix channel number).
>> * Remove the non-standard dma64 driver code, and also remove
>> the HWIF entries, since the associated custom callbacks no
>> longer exist.
>> * Refer to Serge's suggestion: Update the dwmac1000_dma.c to
>> support the multi-DMA-channels controller setup.
>>
>> See:
>> v4: <https://lore.kernel.org/loongarch/cover.1692696115.git.chenfeiyang@loongson.cn/>
>> v3: <https://lore.kernel.org/loongarch/cover.1691047285.git.chenfeiyang@loongson.cn/>
>> v2: <https://lore.kernel.org/loongarch/cover.1690439335.git.chenfeiyang@loongson.cn/>
>> v1: <https://lore.kernel.org/loongarch/cover.1689215889.git.chenfeiyang@loongson.cn/>
>>
>> Yanteng Si (9):
>> net: stmmac: Pass stmmac_priv and chan in some callbacks
>> net: stmmac: dwmac-loongson: Refactor code for loongson_dwmac_probe()
>> net: stmmac: dwmac-loongson: Add full PCI support
>> net: stmmac: Add multi-channel supports
>> net: stmmac: Add Loongson-specific register definitions
>> net: stmmac: dwmac-loongson: Add MSI support
>> net: stmmac: dwmac-loongson: Add GNET support
>> net: stmmac: dwmac-loongson: Disable flow control for GMAC
>> net: stmmac: Disable coe for some Loongson GNET
>>
>> drivers/net/ethernet/stmicro/stmmac/common.h | 1 +
>> .../ethernet/stmicro/stmmac/dwmac-loongson.c | 296 ++++++++++++++----
>> .../net/ethernet/stmicro/stmmac/dwmac-sun8i.c | 2 +-
>> .../ethernet/stmicro/stmmac/dwmac1000_dma.c | 61 +++-
>> .../ethernet/stmicro/stmmac/dwmac100_dma.c | 2 +-
>> .../net/ethernet/stmicro/stmmac/dwmac4_dma.c | 2 +-
>> .../net/ethernet/stmicro/stmmac/dwmac_dma.h | 47 ++-
>> .../net/ethernet/stmicro/stmmac/dwmac_lib.c | 65 ++--
>> .../ethernet/stmicro/stmmac/dwxgmac2_dma.c | 2 +-
>> drivers/net/ethernet/stmicro/stmmac/hwif.c | 8 +-
>> drivers/net/ethernet/stmicro/stmmac/hwif.h | 11 +-
>> .../ethernet/stmicro/stmmac/stmmac_ethtool.c | 6 +
>> .../net/ethernet/stmicro/stmmac/stmmac_main.c | 13 +-
>> include/linux/stmmac.h | 4 +
>> 14 files changed, 413 insertions(+), 107 deletions(-)
>>
>> --
>> 2.31.4
>>
Powered by blists - more mailing lists