Received-SPF: pass (google.com: domain of linux-kernel+bounces-139441-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Message-ID: <7064ab28-a780-4a6e-93f0-5dc26c62e72c@intel.com>
Date: Wed, 10 Apr 2024 15:30:29 -0700
User-Agent: Mozilla Thunderbird
Subject: Re: [net-next PatchV2] octeontx2-af: map management port always to
 first PF
To: Hariprasad Kelam <hkelam@marvell.com>, <netdev@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>
CC: <kuba@kernel.org>, <davem@davemloft.net>, <sgoutham@marvell.com>,
	<gakula@marvell.com>, <jerinj@marvell.com>, <lcherian@marvell.com>,
	<sbhatta@marvell.com>, <naveenm@marvell.com>, <edumazet@google.com>,
	<pabeni@redhat.com>
References: <20240410132538.20158-1-hkelam@marvell.com>
Content-Language: en-US
From: Jacob Keller <jacob.e.keller@intel.com>
In-Reply-To: <20240410132538.20158-1-hkelam@marvell.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?Tld2T0hCeGUzWXBhTGpMZmVPenFUNXlFbUJKOHAxc0tYYnRobStYbTllSkhx?=
 =?utf-8?B?d3NCa29IcFYwSG9MOVpIclYxd1YzMHNPV3RmZCtjelcvaDdicUM0MW5tWWRv?=
 =?utf-8?B?eDZEU3JBUkx5bWZZUmt5b3YwUE16cVFJSkVmWVFMSzVTSy9HWVFGeWVGeWI2?=
 =?utf-8?B?ZUZNNCs2bXhHbU14elJ5cXJNU1FNdzlleThHbStSRVI3TkhKWVl6TkpjdEFX?=
 =?utf-8?B?U3VNSW9OQXUveTdWREI0WGMySjFJbHhCdFUveGJpaENpUmxTZEk1bTRkOFNJ?=
 =?utf-8?B?cVlxWWVxL09Zd1BiNXdLY1c5a21WNzk3UXZITkh2eUZCQ21sS3RVSUQvUkhO?=
 =?utf-8?B?NWJ1Ky9ycjZvVXFGbGZwbnl3bnNwTkNGK2llMmtsZC84Z1hQUlNaUkdDRGFH?=
 =?utf-8?B?c1NuQm1ldmVhZDkvOUJyMnpNUmZsbW0rbjh0dkFCVTVjNU9lZjRwMVZHV2NC?=
 =?utf-8?B?MzZvZVkzMTlpNUppSDRXM0wyTVUvckhWTXFvNDJUTkVzdFZzbzV1RmFCcXlt?=
 =?utf-8?B?ZjczNExQT1p6djhSc1JJS0RHMlVJYS85ZmNKWlUxTlMxVjFhUnpVWEcwdFZq?=
 =?utf-8?B?MDVuQSs3dmhBUm8rVUgxSmxieEZjUlh6SHIzbi9FNnFKQzQweWxxcTlyYzJx?=
 =?utf-8?B?UDE3aDhMZmd6cGJtb1FmazkyWUlzQTM4aU9xTW1TcjFYajRGclVsVzRnQzVi?=
 =?utf-8?B?cnJ1VEsxNExtWWFmRXhJZ1Q3RTAyQzg3Nlp3cmRUQXhKcE9vRUNZenB1K0xu?=
 =?utf-8?B?M2l1RkhzNVFnbmhKM1N0YlU3VFFQMnJlVnp2OHVnR2dzRVBjMGhkMXkvZytv?=
 =?utf-8?B?dVZRNlUrbWh6WDkzUlpvVzBSeGh0NWZpS1pqeFNBek9YUkR3LzUvT3luRFFh?=
 =?utf-8?B?OGl4ekRzRGgxWFpzdldlWENtSXQwOUJXOFN5eWpGTGIzVGNweTBGaGdlcjRJ?=
 =?utf-8?B?bHBISG85VklXRWs5VGt1L2RKdXM1Qzk1MnFZcjk4SllsVTZJc1ZWRHEvcHVN?=
 =?utf-8?B?WFFuN25RamRvVzdHQXVEZkxzTCtpVEg5UFBadjFqU2tnRHVYWGlMM00vM2Ir?=
 =?utf-8?B?MFhVaXN5UGMycGp4TCsxU3hIWGFvMlVQMUh2TTZEWDJGSGFLMXJwSDl5Qk8v?=
 =?utf-8?B?emlhdkEwSkhGaTdrc2ZWcUFZcVRoL2NZakFPQXNDQU1QZ1MybmpMdFZwYmhy?=
 =?utf-8?B?UDVieUN4Zm1NOG04Qy9mR1Jyd0h2azliWWZlbnZMYTdJM3pOem53MkdlSUdk?=
 =?utf-8?B?MFJCMWJaU3Uzd1dibTVva0kvaG0xSFd3L3doSXZIc1UwQngycXJ3V0dja0hi?=
 =?utf-8?B?WkwrTDQxcFVhQVh5VUlyNHlWOFQvMlNjdzVXNmtrQ2oyMFl0SWZKOTRHaUxJ?=
 =?utf-8?B?dm1Zb1JXdGJLZEN0NVVhenlHdjRqdTJlK1NHTGlSaURSZWpxWER5UHNWTFkx?=
 =?utf-8?B?R2VISldRczV2VjlIbU1nWDZpUkdrRUxETFdWQ1NlK2NuMWxka2NMbWFrWXg2?=
 =?utf-8?B?bHk2aG9OdmwycERlaHVUa0lQU0JMSmJCa0VvZHYvcHZXWldCaVY1U1lPeTFt?=
 =?utf-8?B?SXpheE1uWmF3UXFkeGxrdFVJZWU0ckZXbGlNZzdaN3p3ZDcxcklzTzFjRlhR?=
 =?utf-8?B?UjAzYWNSZHFOWUt4VUJhakl1QVY5eGRCOFdWSFAxRUVGZXBaZytNVVBsSTFH?=
 =?utf-8?B?WVYxOGw3c1lJZnFtV1U2UnREVnNRakFoZE5zZDZXR2VHQkNjNVp0ZDJNL1NS?=
 =?utf-8?B?M0ZXemlYd1NFakJ3STh0MUdqb3hQYmNoUWhoRHZGVE1HYytKY0c4Ti9ZWVlY?=
 =?utf-8?B?c2J4b2RxS3ZtMlY2M0V4eGNqSk5RdDNuZ3RwdWVsZjB5Y3gwTU5PZ296Sm42?=
 =?utf-8?B?WVNkZDljNnc5OFhYYVYrbXlVSUxUY1FneS9zcVdJVytwbUZ2OUlwZW56ZWZ6?=
 =?utf-8?B?VGZrRGNvOW9Rdm1qRTVFTWNFN2E5ZnN4UFJRaHBXWVJXWGtXald3Wm1LWCtY?=
 =?utf-8?B?Q3psQ0lGQzdoUmI0Z3k3NXE1bjlpS0FqUFJTeVd4U0cwQmI2aUl5OW9ZOXht?=
 =?utf-8?B?dzY2VW1ha2pIS3hiamc3RlhxOUh6MDA5T1dLSFFmSWpleGtDRWFxVUhGSjJs?=
 =?utf-8?B?SFA3T20xbnVueE4wdWxtRnd6OFF6VU9CQXNPSHBUMWh2NlhHSXFxV1FYQW1t?=
 =?utf-8?B?M1E9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 317fa0be-14b6-4fd1-4e4f-08dc59add507
X-MS-Exchange-CrossTenant-AuthSource: CO1PR11MB5089.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Apr 2024 22:30:31.8744
 (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: EVBLUmaPsDbIEpYGhEn5Tl7gNoTsrBbVxXzsly+i7NVo/oDGkjUisNbw2kVRSQIdir92V1LKnttE4nW2hQKBU9yloW8GGIQdm+34yw6Z2Pk=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN9PR11MB5257
X-OriginatorOrg: intel.com


On 4/10/2024 6:25 AM, Hariprasad Kelam wrote:
> The user can enable or disable any MAC block or a few ports of the
> block. The management port's interface name varies depending on the
> setup of the user if its not mapped to the first pf.
> 
> The management port mapping is now configured to always connect to the
> first PF. This patch implements this change.
> 

nit: it's generally preferred to avoid "this patch.." language in commit
messages, as it is clear from the context.

That being said, its not worth a re-roll just to fix this.

Reviewed-by: Jacob Keller <jacob.keller@intel.com>