Re: [PATCH v3] net: ethernet: ti: am65-cpsw-qos: Add Frame Preemption MAC Merge support
From: Paolo Abeni
Date: Tue Sep 19 2023 - 09:14:27 EST
On Mon, 2023-09-18 at 12:53 +0300, Roger Quadros wrote:
> Add driver support for viewing / changing the MAC Merge sublayer
> parameters and seeing the verification state machine's current state
> via ethtool.
>
> As hardware does not support interrupt notification for verification
> events we resort to polling on link up. On link up we try a couple of
> times for verification success and if unsuccessful then give up.
>
> The Frame Preemption feature is described in the Technical Reference
> Manual [1] in section:
> 12.3.1.4.6.7 Intersperced Express Traffic (IET – P802.3br/D2.0)
>
> Due to Silicon Errata i2208 [2] we set limit min IET fragment size to 124.
>
> [1] AM62x TRM - https://www.ti.com/lit/ug/spruiv7a/spruiv7a.pdf
> [2] AM62x Silicon Errata - https://www.ti.com/lit/er/sprz487c/sprz487c.pdf
>
> Signed-off-by: Roger Quadros <rogerq@xxxxxxxxxx>
> ---
> drivers/net/ethernet/ti/am65-cpsw-ethtool.c | 150 ++++++++++++
> drivers/net/ethernet/ti/am65-cpsw-nuss.c | 2 +
> drivers/net/ethernet/ti/am65-cpsw-nuss.h | 5 +
> drivers/net/ethernet/ti/am65-cpsw-qos.c | 240 ++++++++++++++++----
> drivers/net/ethernet/ti/am65-cpsw-qos.h | 104 +++++++++
> 5 files changed, 454 insertions(+), 47 deletions(-)
>
> Changelog:
>
> v3:
> - Rebase on top of v6.6-rc1 and mqprio support [1]
I'm unsure if this will require a rebase for the next revision of the
mqprio support. Anyhow the two patches are related, it's probably
better bundle them in a series so that the dep is straight-forward.
When reposting, please insert the target tree in the subject profix
(net-next in this case).
Thanks!
Paolo