Received-SPF: pass (google.com: domain of linux-kernel+bounces-130143-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Message-ID: <0214214a-73c4-46b4-a099-189036954aa1@cornelisnetworks.com>
Date: Wed, 3 Apr 2024 11:54:52 -0400
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v0 01/14] IB/hfi1, IB/qib: Make I2C terminology more
 inclusive
To: Leon Romanovsky <leon@kernel.org>,
 Easwar Hariharan <eahariha@linux.microsoft.com>
Cc: Jason Gunthorpe <jgg@ziepe.ca>,
 "open list:HFI1 DRIVER" <linux-rdma@vger.kernel.org>,
 open list <linux-kernel@vger.kernel.org>,
 Wolfram Sang <wsa+renesas@sang-engineering.com>,
 "open list:RADEON and AMDGPU DRM DRIVERS" <amd-gfx@lists.freedesktop.org>,
 "open list:DRM DRIVERS" <dri-devel@lists.freedesktop.org>,
 "open list:INTEL DRM DISPLAY FOR XE AND I915 DRIVERS"
 <intel-gfx@lists.freedesktop.org>,
 "open list:INTEL DRM DISPLAY FOR XE AND I915 DRIVERS"
 <intel-xe@lists.freedesktop.org>,
 "open list:DRM DRIVER FOR NVIDIA GEFORCE/QUADRO GPUS"
 <nouveau@lists.freedesktop.org>,
 "open list:I2C SUBSYSTEM HOST DRIVERS" <linux-i2c@vger.kernel.org>,
 "open list:BTTV VIDEO4LINUX DRIVER" <linux-media@vger.kernel.org>,
 "open list:FRAMEBUFFER LAYER" <linux-fbdev@vger.kernel.org>
References: <20240329170038.3863998-1-eahariha@linux.microsoft.com>
 <20240329170038.3863998-2-eahariha@linux.microsoft.com>
 <20240403083025.GT11187@unreal>
Content-Language: en-US
From: Dennis Dalessandro <dennis.dalessandro@cornelisnetworks.com>
In-Reply-To: <20240403083025.GT11187@unreal>
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?Q1ZxcVpScWJjb2RsY3lNeEszSU9lbyt5UWh6VXpjeTdURzhxOEhvQUwyUXQ2?=
 =?utf-8?B?eWJ0OUtaUHJjYkVtYzBpVjg5dFpUNUhsa2FEUDViTEVVS2NFcHZ0eWxtdjNH?=
 =?utf-8?B?bzI5a2psb0YwcWFxVWwvTmdQUDFhVFFQWFM3TGtGNkVhZG1kOFNTMEhJMEw1?=
 =?utf-8?B?OXNYb0Y1SUlnbDhJVGFIWGVkdXZYZFUwWUxtN0YzOGMzc0JSK0Iwb1RTWEl6?=
 =?utf-8?B?RE12NjlTdEFybjNuOFNaTldSSzYrTSswaUNBcU9adXdlbnp5NW10TE1sb2gw?=
 =?utf-8?B?bEJmNFRwekhWM1RXZU1OWHQxWEhMc1h6NHhtZEZJcTExa0prNlVCT2VobEd4?=
 =?utf-8?B?ZGlQUnZnVWw4RUk4MDNlVmVyMC9WRFZHZ1pkN1o5ME9qZXZRVzlHZzhKSkpP?=
 =?utf-8?B?aklESmljei9JYzFjSGExL1laZUZjME5YeVBhcDhQbFJPOFFydnh4bmxOcUkz?=
 =?utf-8?B?NzhBVnhoTzZiRE93Yk5YbVFtbmdhMXk0bFJRdmJVT0RwTkRMMUtnQW5UWGQ2?=
 =?utf-8?B?ZGx4SE9QSEozWnhhOXM4MW51ZEZwRG5ONHhPZDB4TWVPM2xCekRDRVJGK3R0?=
 =?utf-8?B?b2lVajdXZ0MzRGxaWS8xVFR6L0lPYkhMVjJya0hubEp2VFlVMmRtS0t0U0Z5?=
 =?utf-8?B?VWtnS1Z0SmphYWtlNi9abjEyLzZ2ZWVKL0d1ZU1kNWt6NHRkbmZYcy9uREhW?=
 =?utf-8?B?a0xreGxoMXJoV1RPUWJva3NaSG9QWlVMRHBXbEJsM0lzZ2pwRlRuc3BxWCtu?=
 =?utf-8?B?VjVKN05ONnNhanFnbzA2MVEwZU9PVjByMlM1RFNYNm5QbVNGS29oKzhJNWh3?=
 =?utf-8?B?TnVTN0FYNERqMzZLMVBVaHVJWFRqamVYdlN0bzdmVWt2d0VFd3Q5ZitLM2ZV?=
 =?utf-8?B?NnkwR1ZPNUFWdEdXYzZqUWVabWc3TmVBOXZ3RjJZeUNtOWh2YmxUOWM0ZFYv?=
 =?utf-8?B?TEpJbWNSMFNaeWZudVpUWElIQngwempTVnQzbFRScjlTQkF4SWt5OFZkNytS?=
 =?utf-8?B?K2w2Vy9xak13WUxKR2ZiVTYrVG9yWlM5dThpVzdVWXNIOHN3VHl2cjZhRCtp?=
 =?utf-8?B?TUxnVDJiQjNhMGpqRmRobGlmMExISml0blMwQXlXWnYydGM0V1VuSFRrbjV5?=
 =?utf-8?B?N21paW95WmRLdFYyTk5YNDlTaUlSUWg1bS9qblVpbjdYcms0eW1SN3dhcllM?=
 =?utf-8?B?cnpXQk0wLzVzWFJqNTdWVnF1SEVLMklIYmw4dnAvaVU2S3I2QlVXcWUwWFZD?=
 =?utf-8?B?N21tVytYZDgrbUI1SXpQbVNvQnpRU09GemtKR24vMEZqNjJXVUJIQ3p3S3ZN?=
 =?utf-8?B?YjdveVJrRFRzbm5LQ3BiZkErTkl0MXdNd29xMVkzdkpxbktvNmlqbDlFbEhC?=
 =?utf-8?B?V3Urc3JuR1BuQXlvd0UwMzlqSk40NjF2ZWlEVHhSRE9aUk80ZW5UUjhJbkhW?=
 =?utf-8?B?NGhQMHppZm9uTDEzc0FCQ0dFVFUzWGRrNXBMdGhNUHlEU21YQm8xQXpURlJN?=
 =?utf-8?B?ZHN3UVBjeVJPWElkaDBRLzdPUDRzYVk2ZmVncVE1TTlUMTB3d2UyZHlXRHZk?=
 =?utf-8?B?S3ZJOVZ5RXhjYkgwYTRuVzNFcm91dmtxZjZyWVF6OHNCQ0tqVVBHaHhuTnJa?=
 =?utf-8?B?REIrKzFvdE1mc3UyRHhsZFpacmxwYjNvNlpCc3RKZTVIVkQxY3FEekYxK3ZQ?=
 =?utf-8?B?amRuTy9OMXU2L2VQS25KRDEvdHg0bS94ajB2SDdOUWQ3TjZqaFZTQndxdzdB?=
 =?utf-8?B?QytMbUVIK1dzT2xJRTA5UENZYnRnRzdvWEFTQlREUmtMM0lpcnpOTEJxbkky?=
 =?utf-8?B?V0NvVlpveWtRWUJvc2hsZDN3QUZneEE1MUJkaWEvcndMMkdSUEh6NWU5dTEx?=
 =?utf-8?B?a2M5V3RrMmpjS0g0WkFzd1Q1N0oxcUM3azFjYUJCajZsN1M4cllnQ0wwK1dL?=
 =?utf-8?B?Uk1TWlRUSWl3TDFNdFAwNlVnZE00REhVSVB1UlR1SHBDQU1GQ2laU0l0aTN4?=
 =?utf-8?B?TFJ0eDhwMWZSYzVpU01ScVovVE1DQ3Z3ZmZiTjgva0tWa29jU1puNzBEQ1ls?=
 =?utf-8?B?N1lsTVlOTnZYYWJXUU02K1pDcXh3RFFQSDdTMExTMkRaNFdXRmVCWmY2UzN1?=
 =?utf-8?B?cWVUdjJLeWFWZGtwenFzNmp6eFpXamg0WXFnYWFXUHVKUzNLUU56WXdGckpQ?=
 =?utf-8?Q?HBnu6TdOM8KTUzV2ADxO0sc=3D?=
X-OriginatorOrg: cornelisnetworks.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 52238616-eb3f-4744-034c-08dc53f66b1a
X-MS-Exchange-CrossTenant-AuthSource: SJ0PR01MB6158.prod.exchangelabs.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Apr 2024 15:55:00.6323
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4dbdb7da-74ee-4b45-8747-ef5ce5ebe68a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: /8lVdPrIYGpjpJ7zVHlhIgtO5fHdjrEYdcgLp3d/P/2yZRzBetv72+urNfNc7PJ+Nsq6FicmOPTKil+GN8mKQjY1wmStmxxhDq6SyEB2eyOYWaVm8WopdgjdtSgfjZEV
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR01MB6603


On 4/3/24 4:30 AM, Leon Romanovsky wrote:
> On Fri, Mar 29, 2024 at 05:00:25PM +0000, Easwar Hariharan wrote:
>> I2C v7, SMBus 3.2, and I3C specifications have replaced "master/slave"
>> with more appropriate terms. Inspired by and following on to Wolfram's series
>> to fix drivers/i2c[1], fix the terminology where I had a role to play, now that
>> the approved verbiage exists in the specification.
>>
>> Compile tested, no functionality changes intended
>>
>> [1]: https://lore.kernel.org/all/20240322132619.6389-1-wsa+renesas@sang-engineering.com/
>>
>> Signed-off-by: Easwar Hariharan <eahariha@linux.microsoft.com>
>> ---
>>  drivers/infiniband/hw/hfi1/chip.c           |  6 ++--
>>  drivers/infiniband/hw/hfi1/chip.h           |  2 +-
>>  drivers/infiniband/hw/hfi1/chip_registers.h |  2 +-
>>  drivers/infiniband/hw/hfi1/file_ops.c       |  2 +-
>>  drivers/infiniband/hw/hfi1/firmware.c       | 22 ++++++-------
>>  drivers/infiniband/hw/hfi1/pcie.c           |  2 +-
>>  drivers/infiniband/hw/hfi1/qsfp.c           | 36 ++++++++++-----------
>>  drivers/infiniband/hw/hfi1/user_exp_rcv.c   |  2 +-
>>  drivers/infiniband/hw/qib/qib_twsi.c        |  6 ++--
>>  9 files changed, 40 insertions(+), 40 deletions(-)
> 
> hfi1 and qib work perfectly fine with the current terminology. There is
> no need to change old code just for the sake of change.
> 
> Let's drop this patch.

Agreed.