Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.36 as permitted sender) client-ip=23.128.96.36;
Message-ID: <ff97d821-be9f-4625-815a-d9a9d65d68f9@intel.com>
Date:   Thu, 23 Nov 2023 11:47:05 +0100
User-Agent: Mozilla Thunderbird
Subject: Re: [net PATCH 2/5] octeontx2-af: Fix mcs sa cam entries size
Content-Language: en-US
To:     Geetha sowjanya <gakula@marvell.com>, <netdev@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>
CC:     <kuba@kernel.org>, <davem@davemloft.net>, <pabeni@redhat.com>,
        <edumazet@google.com>, <sgoutham@marvell.com>,
        <sbhatta@marvell.com>, <hkelam@marvell.com>
References: <20231123055941.19430-1-gakula@marvell.com>
 <20231123055941.19430-3-gakula@marvell.com>
From:   Wojciech Drewek <wojciech.drewek@intel.com>
In-Reply-To: <20231123055941.19430-3-gakula@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?VWhpMWQ5NFdOODhRTXQ5cUhJUmM0WFkwd2hSYUVzUWR0aFZubmRLRUFucTdE?=
 =?utf-8?B?azdwRkpUQVFTdm9CaHh1Nkd4VHRUdFQzb1pVUjc2MGMzMUl4ZnlZOEpraFhv?=
 =?utf-8?B?R3hsSVlRNUtGTEFoamxCTWt5eEdzdVZVbWFSdkRPdzFYQ1ZJbGlva2N6bDBl?=
 =?utf-8?B?dlA3ZDc4S0xLT1ZGSkg3RlBOc2NVTXQwNG1kMi9vMnk5ZmNEd1FXYi9INk1W?=
 =?utf-8?B?ckJ3dVJwd1ExOHJRY1hjd2I2NTNmekhMWGY2Mm5xT3FHOWZXaTZ2YlJWdThS?=
 =?utf-8?B?elpBVFZEZkd4Q01NUTl1WFlEdERzdGg0Q1AxTHVyU1BEWWZyaEZ5a0crbFFy?=
 =?utf-8?B?Y2hUQnBUK253eTVOWGloamNvbmdnam9lc2oyRXd3aTdKa1VwOTlZQjNldmJE?=
 =?utf-8?B?SytSSitrVXFOL1ZITUpGbFlNREdaaFI2eUhRVTAwdkJZSXgrVU4xR3hJVlM1?=
 =?utf-8?B?SnhOYkJIWEUremFPa08wZk15ci9KT1ZGc3I5MjgrZk1Oa2VEOGF5WDNLaWlQ?=
 =?utf-8?B?S3JSSHN6eG1OSFpWQnkyaEUvWXFTbUkrNnhDd293SS93Y1FubTl2Z243cXBO?=
 =?utf-8?B?akJkazdMRlZlNnRvM0dGTTFrRU9YWWRYZkdxWHBUUHAwak5kU2NXTlZkR04y?=
 =?utf-8?B?SzJSMllzZ3o0UVRXTTQyVnNNTkdVUXVNYVJJVlFoSkZSNGR3alkwM2FKNis5?=
 =?utf-8?B?b081RmcrdWovS2FkeUdrUjE1L0Z4S25QYkx2eUtTdy9WNndxc0RLRS9VRHdL?=
 =?utf-8?B?YUJIVHpMWDdKaXE2UVpMZmQxdHloaG9wR0FOZVR5eW9KMlIzWVpqZTJpN0Rv?=
 =?utf-8?B?MEswZU1YNitEVFI0a3c3MGdkUG9nOFpjMS9lZXNodWVQU1NLNGd1QXhWNkxC?=
 =?utf-8?B?NUZHOGE4NUFTT1BLMHNKTVFaWkNCQmxFcFVqa0RraUE3cEpFV3l4emNNVzNo?=
 =?utf-8?B?VDQ2VTdBNTlDa1J0bzU2c2ttc055Mjh1cDM4Tk9aNk45dW5xNWFwR2pDTy80?=
 =?utf-8?B?R1ZDL2RyY294NEtjRHZ6ZnJMa0tSM1l6a1hMM0taVkFFZkhmQ3RQd3ZKTUl1?=
 =?utf-8?B?eXhQRzJSa1lSMVo2YkxBaTg3Qm5LODY2anNIVmhmMGxPTlhPWDVRSXplU3Qv?=
 =?utf-8?B?ZXdndmQxMVlscTBhNkUzZXFyL0lmYWE2OEpqVG1EWTV0dVljbklOTksvWExS?=
 =?utf-8?B?TUNWamM5NnJGYjdyNFBDRGh6bjB2SjAwY2VFWjJYcG1Kck9JK3B2NnhaUjRl?=
 =?utf-8?B?anAzZGlIWDNnbVlUcFFURE9nQW5nL0ZrU24wc1VUWVVYOEh0N2YzR1NKVkxi?=
 =?utf-8?B?OE85M1RaVnlZU0tibFZUV0pyVWZBaTJGOW1Qa25JOXN1OXYrVWRzVzZJdlJS?=
 =?utf-8?B?UFBmVExTNHpxSVhIV1pTNTlDTEpzY2d4TmFmR2VqN2xqMUFwMXN5N1V1a0xT?=
 =?utf-8?B?ZTczOUozU3IzZGF6UE9RcFV5NTFUOHVrSVNsYVVhaW9PblViMXF3Z24zTU9o?=
 =?utf-8?B?elFhalhtRVRMNFVXakgyZWFXZUk0Z2J4OWEzcXVEUFI3cE9KeFpuRU1YVTZO?=
 =?utf-8?B?Y1hrdWZGNURRYnVKdUZSNmFHK2pDOUhQdWdkRVY0SXZrT3hPVmNWTjMxanE2?=
 =?utf-8?B?Zmx0RkpUdXZMeEFuRE1Lc1B3STN4M3drSmRtNGxQbVorV2RFdHozWTJqODNO?=
 =?utf-8?B?dDRjdTJHV2paM0FMY0tEVXJYTXM1V3F3NzBEV1NIZE1WVmgzek84ZWt0WmNN?=
 =?utf-8?B?YXQ4RnEwUWxHMFRXQXVwNzFyQkhtQW02TzM5ZzdMTDVUeUVrM1RpTDRWOElx?=
 =?utf-8?B?b1VvRndlRGpqN3JSZkprQWFpV1owS25CUk50Vm1aT1p3dWd1MXJRU05hNFpG?=
 =?utf-8?B?NXk2MTBKT01aT0dJWWdEUUN6bUpDY3NZV2NCTWxQSkZpMUE5NU1QRXozZDg1?=
 =?utf-8?B?MmFXaDl0TWk3ZkE1RkZXVmhLMkpkek5kaU5KUmc0alphYWFaSFNKWHVBd0cz?=
 =?utf-8?B?a0IxdTYrK2x0azAzRVdWb0NMdzZXc3FvY2RqK00vU0syL2l1VWJLeHJ5WGlM?=
 =?utf-8?B?NFFhZUhXdTFwclp1NS82QnBMeHVmK1cxRDdPQjVpZ2cycDhWbUhKY1NINmNQ?=
 =?utf-8?B?eHB6MXhZc09SQVl4ZE0zYVREenhZdjRVZEU2TjFvbHdpcGFOMWNzNUFSY1JE?=
 =?utf-8?B?c3c9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 19e784e8-c1c1-473e-20df-08dbec1189ef
X-MS-Exchange-CrossTenant-AuthSource: MW4PR11MB5776.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Nov 2023 10:47:07.8248
 (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: 4EtgRg2fkYTXl9jEQZwrpoovfj6yp1K5UkpNXVEhArN0vnUSD+odXoB664aTBdVBjhWMchyTeHKeHD7ss5bGdMh9z7CxdlNYbGoTVF/EZC8=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS0PR11MB7311
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-0.9 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,T_SCC_BODY_TEXT_LINE
	autolearn=unavailable autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on pete.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 (pete.vger.email [0.0.0.0]); Thu, 23 Nov 2023 02:47:48 -0800 (PST)


On 23.11.2023 06:59, Geetha sowjanya wrote:
> On latest silicon versions SA cam entries increased to 256.
> This patch fixes the datatype of sa_entries in mcs_hw_info
> struct to u16 to hold 256 entries.
> 
> Fixes: 080bbd19c9dd ("octeontx2-af: cn10k: mcs: Add mailboxes for port related operations")
> Signed-off-by: Geetha sowjanya <gakula@marvell.com>
> ---

Reviewed-by: Wojciech Drewek <wojciech.drewek@intel.com>

>  drivers/net/ethernet/marvell/octeontx2/af/mbox.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/ethernet/marvell/octeontx2/af/mbox.h b/drivers/net/ethernet/marvell/octeontx2/af/mbox.h
> index 6845556581c3..5df42634ceb8 100644
> --- a/drivers/net/ethernet/marvell/octeontx2/af/mbox.h
> +++ b/drivers/net/ethernet/marvell/octeontx2/af/mbox.h
> @@ -1945,7 +1945,7 @@ struct mcs_hw_info {
>  	u8 tcam_entries;	/* RX/TX Tcam entries per mcs block */
>  	u8 secy_entries;	/* RX/TX SECY entries per mcs block */
>  	u8 sc_entries;		/* RX/TX SC CAM entries per mcs block */
> -	u8 sa_entries;		/* PN table entries = SA entries */
> +	u16 sa_entries;		/* PN table entries = SA entries */
>  	u64 rsvd[16];
>  };
>