Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.35 as permitted sender) client-ip=23.128.96.35;
Message-ID: <0cc19076-7c9b-4f6e-80e2-1a403fc0e501@intel.com>
Date:   Fri, 27 Oct 2023 12:10:04 +0200
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH net v1 1/2] octeontx2-pf: Fix error codes
Content-Language: en-US
To:     Ratheesh Kannoth <rkannoth@marvell.com>, <netdev@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>
CC:     <sgoutham@marvell.com>, <gakula@marvell.com>,
        <sbhatta@marvell.com>, <hkelam@marvell.com>, <davem@davemloft.net>,
        <edumazet@google.com>, <kuba@kernel.org>, <pabeni@redhat.com>
References: <20231027021953.1819959-1-rkannoth@marvell.com>
From:   Wojciech Drewek <wojciech.drewek@intel.com>
In-Reply-To: <20231027021953.1819959-1-rkannoth@marvell.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?Nnk0ZE03ek5abVdCQjNnK2FoTEhiM0l6UVhMVDMrVDVxSU5SdkdnUStXNWNm?=
 =?utf-8?B?dit1OHgxWW9LQTJlKy9KaitGZkZ2VVVPZlZMWldYOFRCTGJHSnhyMnJnMldq?=
 =?utf-8?B?enpwMEpLZGtxVHlwcWx5UVFKZ1pYY2NGdngwU2hUbkRwUTRPdjJMQ3NwRTg5?=
 =?utf-8?B?cFZKQWNKMXVKTmRzdE1DWXdqUU9HSmRtZ3p4dlpvMmpRTlJsdmpKWkZjWTJk?=
 =?utf-8?B?MDE0YVd0dGxJa0o0UUFhLzg5TklILzA5L3N3Nk4wMWFRazFzOHB3dXd0b3JL?=
 =?utf-8?B?ckRINVhKSFd5ZGJ6VWs1OFE5OGc0V2hXVi9sRUhocHppNy9sTVdqSUtPOVBi?=
 =?utf-8?B?Q0dBRTI2VG53cmk2OFVqNUxvNzcvbzFEeTNVdmM0b3BObTlVVVFjRjFXR0NH?=
 =?utf-8?B?L0dVclNabyt4WU0xbzBsMmZnS214clpCR2U2WGxLTExmdE9KRFJLdEpyeTEy?=
 =?utf-8?B?bEl2ZEVVaXVxY0V5cTFTS01Kdkt5NnFzMkNLQzRUcFVhK3RGNWNVbkJYcERm?=
 =?utf-8?B?bVFwNEZPb24wYmlEU2hRUzcyb093SzVrQVg0WTJiT1dTaGhKcUtPZXJPSXpY?=
 =?utf-8?B?ZlhiV2FCclU1cEdXckFNcU9nNm5oaHNLUGpkeEI4MUhPdllBekR3NVBnVE43?=
 =?utf-8?B?RjQ0USswYnNpaTE5Nmpub0JIQVpDOE14N0E2VGtTVVRtRTdPZ1hqZE1HUlVu?=
 =?utf-8?B?OVZlL2Y1YnQ2MldnSnJNWEdQZjNhREZRREtzU2tyYi81ZGVYQmtNcS9Wakd2?=
 =?utf-8?B?cGlhaUtRV1Zjd3MxTDN6cGZVeUxQQ3pnZGRDV29YZkNwN25lN1ZteXVLUFBW?=
 =?utf-8?B?R2YwbUtmK2dPZ3M4bDMrWE9XUVVqemRZdE9JaytQKzhaeE1GMm5RSStWeWlO?=
 =?utf-8?B?UXpnMWo1Y3VqR0hiOWI2UVZMSEFad0ltNEs0TVNySHF0dXYvQ3NYbHNPYmRL?=
 =?utf-8?B?M3RLYkRrWHllMFA0NWNiQ1dydzJvZ1hrT1pONDhGYnRWN0ZmUjZyZERKQUE0?=
 =?utf-8?B?SnpvamtPejNyRlUyRUtvWXhqRkhmTUg1aHEyTDBwYTVsd1dIREtLMVlNbXFE?=
 =?utf-8?B?WVBkZXlJQ1FDRVMzZlM0bjA5RWRJdVExSGFOV2UwZm1Wc0NJdnpib2EwMnBa?=
 =?utf-8?B?WEVLS0dac1AxZ2FpcjZIek5CeTdxbzJwb0wzQm84U1FPcWNGYVkxZU5vZzBP?=
 =?utf-8?B?a0JqTHpxODNWd0FVRGJ5cVVWNXgvUHlRMEhvTkpTZXZpdGR2VXQrK2tOSmxy?=
 =?utf-8?B?SWlXdjVYZDgxYVRjVjRxbnlIak5kVmpOR3E2U3hKRDZLWlJRRS9pNHkzdzJ3?=
 =?utf-8?B?VHVhN1FmS2dDM0tZWDZ4S3Rkcml3NDFoZlRBVGw2Q3ZyMnk1K1BiZFZlY3Jp?=
 =?utf-8?B?TlVrL25YSE8wdFYyM21aZU50YU9JOU1oTmlIeU01TW1sMWo0MDRaSGtYcktt?=
 =?utf-8?B?ZE00UEgxZmpZNXZ6WWtCQmk1ZE0zWkV2QlFQZ25PUFZNbzZCYm5yUTVrT1pJ?=
 =?utf-8?B?clJZcFlwWCtnb3cyTld6T0owbEpzTzRkN1Ezc08yZGpSOHd6bUFGTWtVc0o0?=
 =?utf-8?B?cjZsZ0VuMWpGQkk4WW5laXk0ZGp3R2c0d2R2OHcvVHoyUFNkWnE2c2UwUmVN?=
 =?utf-8?B?emZZYXEvNFUwZHQ2S2c2MjU3d0lTSTVzcXBDVmFNWjdabGtGZDlLeXV1UnJC?=
 =?utf-8?B?MUE4STNhQXZKZjlHMXpQOWJZL21qdnFteFlTeHdXNDFKV08xOU95dE95ejN0?=
 =?utf-8?B?MkdlN2RpQTZUVEE5TGJ6MEJXVlNlV0pxUk54MlRIQVhRWjhleUZQYUhvUG8z?=
 =?utf-8?B?Y2h0SE0xVVE1Tnk4ZHhiUTQ3RWRaWVI4c1FCM0plKzgwQ3R0cE92SVdsaXla?=
 =?utf-8?B?SkdxYWlTYmpuQlppY3FiRFA2SmVMWklkbnNxM0crcHhVb051Sk9yTFJCVDgv?=
 =?utf-8?B?aGlvd2hOb0EwbWtPai83UHhMakhkOUs2OUdwWWZURDJNZFBuZU1ZeXJONWNr?=
 =?utf-8?B?ZmVEMGg5enc5cnp2cE55T01CZVhaTWNRb1FOd0F3ckNVenZKSUhxU25FK0lp?=
 =?utf-8?B?TGc4TTlTd1d6cmFUT21sVG1iVFZrd042RVpnYys2cVZRM2VaUDgyRkxFWmEw?=
 =?utf-8?B?Y2pER2ZJdnZKODZsdkVUZUJ1NlRLamhVRUdDcHpKNnpRclJENmIzUDFNeGNX?=
 =?utf-8?B?alE9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 8b4cdd55-0172-4a7b-6395-08dbd6d4e58a
X-MS-Exchange-CrossTenant-AuthSource: MW4PR11MB5776.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 Oct 2023 10:10:07.7612
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: ZwHg4pZlvAGXoyr9F59grKsUz0c4suIGpna1TbiW/mHWt/994VxM6EbNaPYdZfBFWXvLzjJz0XPcAMqIaPrUFsdVgSU2H3FwgrFYi/Tp8BM=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW4PR11MB6667
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-0.1 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,UPPERCASE_50_75
	autolearn=unavailable autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on groat.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (groat.vger.email [0.0.0.0]); Fri, 27 Oct 2023 03:10:31 -0700 (PDT)


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,
>  };
>