From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from linux.microsoft.com (linux.microsoft.com [13.77.154.182]) by smtp.subspace.kernel.org (Postfix) with ESMTP id C16136CDB3; Tue, 2 Apr 2024 12:06:32 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=13.77.154.182 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1712059594; cv=none; b=N3zzxDBvc7Fnl8yIg/YkPFn0aKx7RnH0rlCHED82BSIrRzsa42HYSpQnzBOpiXBA6pc8ApDXeVZu0nc86i+GIGgzyKf1BcN8WrWzRcQaT2Knswzc46MxECcL1IRfozkYZ8ZWNXvDSg2BzboIHoFg9STcG/ezPuJDNzWkrLTOwMw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1712059594; c=relaxed/simple; bh=YM1wpGIbCTafqqk2E0Qt4/0Do5RjQ4D516TIYPBj+xc=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=TcXi3VWheFpFpps5FdSTqseQ7Zfm80DVNOK8iHySKNbp9eMMOm32tNPX82qyF4R0vwxZuBEcEgG8R/TlEE4875VSLEKgexq28umZLqauuJLUahoEtpe22DmGvyYBdc63bLcp5xFPcIy61gDukCW4BxNbSFudGjJ4NyHh4sSKuVQ= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.microsoft.com; spf=pass smtp.mailfrom=linux.microsoft.com; dkim=pass (1024-bit key) header.d=linux.microsoft.com header.i=@linux.microsoft.com header.b=JoPMQQrX; arc=none smtp.client-ip=13.77.154.182 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.microsoft.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.microsoft.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linux.microsoft.com header.i=@linux.microsoft.com header.b="JoPMQQrX" Received: from [100.64.128.57] (unknown [20.29.225.195]) by linux.microsoft.com (Postfix) with ESMTPSA id 1F2EA201F162; Tue, 2 Apr 2024 04:59:50 -0700 (PDT) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com 1F2EA201F162 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.microsoft.com; s=default; t=1712059190; bh=4QKcorRa2j3e9zMA898RRqI/yThkz1ahxlu7DAFGz60=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=JoPMQQrXjyUzoMzuxml6ieVZsmT+qn3e34FDKzFiEoOoqLmxXELbe9/YF/ja2sKWk GUCjiPUbanllynPF79fDgPPayCnsbRuDSs5v4oYgLp5tigqr9JXB7AWGAa/BO04Q/C 9YbOKSm5RNmqEtccfO2K5qHk7dW/x5RqEeZsJkpQ= Message-ID: <7d5e6ed0-ffe9-46c2-b3b4-a4a47c09532e@linux.microsoft.com> Date: Tue, 2 Apr 2024 04:59:48 -0700 Precedence: bulk X-Mailing-List: linux-fbdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v0 03/14] drm/gma500,drm/i915: Make I2C terminology more inclusive To: Jani Nikula , Patrik Jakobsson , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Daniel Vetter , Rodrigo Vivi , Joonas Lahtinen , Tvrtko Ursulin , Zhenyu Wang , Zhi Wang , dri-devel@lists.freedesktop.org, open list , "open list:INTEL DRM DISPLAY FOR XE AND I915 DRIVERS" , "open list:INTEL DRM DISPLAY FOR XE AND I915 DRIVERS" , "open list:INTEL GVT-g DRIVERS (Intel GPU Virtualization)" Cc: Wolfram Sang , "open list:RADEON and AMDGPU DRM DRIVERS" , "open list:DRM DRIVER FOR NVIDIA GEFORCE/QUADRO GPUS" , "open list:I2C SUBSYSTEM HOST DRIVERS" , "open list:BTTV VIDEO4LINUX DRIVER" , "open list:FRAMEBUFFER LAYER" References: <20240329170038.3863998-1-eahariha@linux.microsoft.com> <20240329170038.3863998-4-eahariha@linux.microsoft.com> <87a5mcfbms.fsf@intel.com> Content-Language: en-CA From: Easwar Hariharan In-Reply-To: <87a5mcfbms.fsf@intel.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 4/2/2024 12:48 AM, Jani Nikula wrote: > On Fri, 29 Mar 2024, 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 for users of >> I2C_ALGOBIT bitbanging interface, now that the approved verbiage exists >> in the specification. > > gma500 and i915 changes should be split. See MAINTAINERS. > > Might also split the i915 changes to smaller pieces, it's kind of > random. And the changes here are not strictly related to I2C AFAICT, so > the commit message should be updated. > > BR, > Jani. > > I will split gma500 and i915 into their respective patches if possible in v2. Can you say more about the changes being "not strictly related to I2C"? My heuristic was to grep for master/slave, and look in the surrounding context for i2c-related terminology (i2c_pin, 7-bit address, struct i2c_adapter, i2c_bus, etc) to confirm that they are i2c-related, then following the references around to make the compiler happy. For e.g., I did not change the many references to bigjoiner master and slave because I understood from context they were not i2c references. A couple examples would help me restrict the changes to I2C, since as mentioned in the discussion on Wolfram's thread, there are places where migrating away from master/slave terms in the code would conflict with the original technical manuals and reduce correlation and understanding of the code. Thanks, Easwar