Re: [net-next PATCH] octeontx2-af: Add NIX Errata workaround on CN10K silicon

From: Leon Romanovsky
Date: Sun Feb 19 2023 - 04:17:48 EST


On Fri, Feb 17, 2023 at 11:21:12AM +0530, Sai Krishna wrote:
> From: Geetha sowjanya <gakula@xxxxxxxxxxx>
>
> This patch adds workaround for below 2 HW erratas
>
> 1. Due to improper clock gating, NIXRX may free the same
> NPA buffer multiple times.. to avoid this, always enable
> NIX RX conditional clock.
>
> 2. NIX FIFO does not get initialized on reset, if the SMQ
> flush is triggered before the first packet is processed, it
> will lead to undefined state. The workaround to perform SMQ
> flush only if packet count is non-zero in MDQ.
>
> Signed-off-by: Geetha sowjanya <gakula@xxxxxxxxxxx>
> Signed-off-by: Sunil Kovvuri Goutham <sgoutham@xxxxxxxxxxx>
> Signed-off-by: Sai Krishna <saikrishnag@xxxxxxxxxxx>
> ---
> .../net/ethernet/marvell/octeontx2/af/rvu.h | 3 +++
> .../ethernet/marvell/octeontx2/af/rvu_cn10k.c | 18 ++++++++++++++++++
> .../ethernet/marvell/octeontx2/af/rvu_nix.c | 10 ++++++++++
> .../ethernet/marvell/octeontx2/af/rvu_reg.h | 2 ++
> 4 files changed, 33 insertions(+)

Just curious, why aren't these erratas coded as PCI quirks?

Thanks