[net,v1,1/2] octeontx2-pf: Fix error codes
Commit Message
Some of error codes were wrong. Fix the same.
Fixes: 51afe9026d0c ("octeontx2-pf: NIX TX overwrites SQ_CTX_HW_S[SQ_INT]")
Signed-off-by: Ratheesh Kannoth <rkannoth@marvell.com>
---
ChangeLog
v0 -> v1: Splitted patch into two
---
.../marvell/octeontx2/nic/otx2_struct.h | 34 +++++++++----------
1 file changed, 17 insertions(+), 17 deletions(-)
Comments
On 27.10.2023 04:19, Ratheesh Kannoth wrote:
> Some of error codes were wrong. Fix the same.
>
> Fixes: 51afe9026d0c ("octeontx2-pf: NIX TX overwrites SQ_CTX_HW_S[SQ_INT]")
> Signed-off-by: Ratheesh Kannoth <rkannoth@marvell.com>
Reviewed-by: Wojciech Drewek <wojciech.drewek@intel.com>
>
> ---
>
> ChangeLog
> v0 -> v1: Splitted patch into two
> ---
> .../marvell/octeontx2/nic/otx2_struct.h | 34 +++++++++----------
> 1 file changed, 17 insertions(+), 17 deletions(-)
>
> diff --git a/drivers/net/ethernet/marvell/octeontx2/nic/otx2_struct.h b/drivers/net/ethernet/marvell/octeontx2/nic/otx2_struct.h
> index fa37b9f312ca..4e5899d8fa2e 100644
> --- a/drivers/net/ethernet/marvell/octeontx2/nic/otx2_struct.h
> +++ b/drivers/net/ethernet/marvell/octeontx2/nic/otx2_struct.h
> @@ -318,23 +318,23 @@ enum nix_snd_status_e {
> NIX_SND_STATUS_EXT_ERR = 0x6,
> NIX_SND_STATUS_JUMP_FAULT = 0x7,
> NIX_SND_STATUS_JUMP_POISON = 0x8,
> - NIX_SND_STATUS_CRC_ERR = 0x9,
> - NIX_SND_STATUS_IMM_ERR = 0x10,
> - NIX_SND_STATUS_SG_ERR = 0x11,
> - NIX_SND_STATUS_MEM_ERR = 0x12,
> - NIX_SND_STATUS_INVALID_SUBDC = 0x13,
> - NIX_SND_STATUS_SUBDC_ORDER_ERR = 0x14,
> - NIX_SND_STATUS_DATA_FAULT = 0x15,
> - NIX_SND_STATUS_DATA_POISON = 0x16,
> - NIX_SND_STATUS_NPC_DROP_ACTION = 0x17,
> - NIX_SND_STATUS_LOCK_VIOL = 0x18,
> - NIX_SND_STATUS_NPC_UCAST_CHAN_ERR = 0x19,
> - NIX_SND_STATUS_NPC_MCAST_CHAN_ERR = 0x20,
> - NIX_SND_STATUS_NPC_MCAST_ABORT = 0x21,
> - NIX_SND_STATUS_NPC_VTAG_PTR_ERR = 0x22,
> - NIX_SND_STATUS_NPC_VTAG_SIZE_ERR = 0x23,
> - NIX_SND_STATUS_SEND_MEM_FAULT = 0x24,
> - NIX_SND_STATUS_SEND_STATS_ERR = 0x25,
> + NIX_SND_STATUS_CRC_ERR = 0x10,
> + NIX_SND_STATUS_IMM_ERR = 0x11,
> + NIX_SND_STATUS_SG_ERR = 0x12,
> + NIX_SND_STATUS_MEM_ERR = 0x13,
> + NIX_SND_STATUS_INVALID_SUBDC = 0x14,
> + NIX_SND_STATUS_SUBDC_ORDER_ERR = 0x15,
> + NIX_SND_STATUS_DATA_FAULT = 0x16,
> + NIX_SND_STATUS_DATA_POISON = 0x17,
> + NIX_SND_STATUS_NPC_DROP_ACTION = 0x20,
> + NIX_SND_STATUS_LOCK_VIOL = 0x21,
> + NIX_SND_STATUS_NPC_UCAST_CHAN_ERR = 0x22,
> + NIX_SND_STATUS_NPC_MCAST_CHAN_ERR = 0x23,
> + NIX_SND_STATUS_NPC_MCAST_ABORT = 0x24,
> + NIX_SND_STATUS_NPC_VTAG_PTR_ERR = 0x25,
> + NIX_SND_STATUS_NPC_VTAG_SIZE_ERR = 0x26,
> + NIX_SND_STATUS_SEND_MEM_FAULT = 0x27,
> + NIX_SND_STATUS_SEND_STATS_ERR = 0x28,
> NIX_SND_STATUS_MAX,
> };
>
Hello:
This series was applied to netdev/net.git (main)
by Paolo Abeni <pabeni@redhat.com>:
On Fri, 27 Oct 2023 07:49:52 +0530 you wrote:
> Some of error codes were wrong. Fix the same.
>
> Fixes: 51afe9026d0c ("octeontx2-pf: NIX TX overwrites SQ_CTX_HW_S[SQ_INT]")
> Signed-off-by: Ratheesh Kannoth <rkannoth@marvell.com>
>
> ---
>
> [...]
Here is the summary with links:
- [net,v1,1/2] octeontx2-pf: Fix error codes
https://git.kernel.org/netdev/net/c/96b9a68d1a6e
- [net,v1,2/2] octeontx2-pf: Fix holes in error code
https://git.kernel.org/netdev/net/c/7aeeb2cb7a25
You are awesome, thank you!
@@ -318,23 +318,23 @@ enum nix_snd_status_e {
NIX_SND_STATUS_EXT_ERR = 0x6,
NIX_SND_STATUS_JUMP_FAULT = 0x7,
NIX_SND_STATUS_JUMP_POISON = 0x8,
- NIX_SND_STATUS_CRC_ERR = 0x9,
- NIX_SND_STATUS_IMM_ERR = 0x10,
- NIX_SND_STATUS_SG_ERR = 0x11,
- NIX_SND_STATUS_MEM_ERR = 0x12,
- NIX_SND_STATUS_INVALID_SUBDC = 0x13,
- NIX_SND_STATUS_SUBDC_ORDER_ERR = 0x14,
- NIX_SND_STATUS_DATA_FAULT = 0x15,
- NIX_SND_STATUS_DATA_POISON = 0x16,
- NIX_SND_STATUS_NPC_DROP_ACTION = 0x17,
- NIX_SND_STATUS_LOCK_VIOL = 0x18,
- NIX_SND_STATUS_NPC_UCAST_CHAN_ERR = 0x19,
- NIX_SND_STATUS_NPC_MCAST_CHAN_ERR = 0x20,
- NIX_SND_STATUS_NPC_MCAST_ABORT = 0x21,
- NIX_SND_STATUS_NPC_VTAG_PTR_ERR = 0x22,
- NIX_SND_STATUS_NPC_VTAG_SIZE_ERR = 0x23,
- NIX_SND_STATUS_SEND_MEM_FAULT = 0x24,
- NIX_SND_STATUS_SEND_STATS_ERR = 0x25,
+ NIX_SND_STATUS_CRC_ERR = 0x10,
+ NIX_SND_STATUS_IMM_ERR = 0x11,
+ NIX_SND_STATUS_SG_ERR = 0x12,
+ NIX_SND_STATUS_MEM_ERR = 0x13,
+ NIX_SND_STATUS_INVALID_SUBDC = 0x14,
+ NIX_SND_STATUS_SUBDC_ORDER_ERR = 0x15,
+ NIX_SND_STATUS_DATA_FAULT = 0x16,
+ NIX_SND_STATUS_DATA_POISON = 0x17,
+ NIX_SND_STATUS_NPC_DROP_ACTION = 0x20,
+ NIX_SND_STATUS_LOCK_VIOL = 0x21,
+ NIX_SND_STATUS_NPC_UCAST_CHAN_ERR = 0x22,
+ NIX_SND_STATUS_NPC_MCAST_CHAN_ERR = 0x23,
+ NIX_SND_STATUS_NPC_MCAST_ABORT = 0x24,
+ NIX_SND_STATUS_NPC_VTAG_PTR_ERR = 0x25,
+ NIX_SND_STATUS_NPC_VTAG_SIZE_ERR = 0x26,
+ NIX_SND_STATUS_SEND_MEM_FAULT = 0x27,
+ NIX_SND_STATUS_SEND_STATS_ERR = 0x28,
NIX_SND_STATUS_MAX,
};