Received-SPF: pass (google.com: domain of linux-kernel+bounces-168138-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Date: Fri, 3 May 2024 15:34:12 -0400
From: Rodrigo Vivi <rodrigo.vivi@intel.com>
To: Easwar Hariharan <eahariha@linux.microsoft.com>
CC: Jani Nikula <jani.nikula@linux.intel.com>, Joonas Lahtinen
	<joonas.lahtinen@linux.intel.com>, Tvrtko Ursulin <tursulin@ursulin.net>,
	David Airlie <airlied@gmail.com>, Daniel Vetter <daniel@ffwll.ch>, "Zhenyu
 Wang" <zhenyuw@linux.intel.com>, Zhi Wang <zhi.wang.linux@gmail.com>, "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 DRIVERS"
	<dri-devel@lists.freedesktop.org>, open list <linux-kernel@vger.kernel.org>,
	"open list:INTEL GVT-g DRIVERS (Intel GPU Virtualization)"
	<intel-gvt-dev@lists.freedesktop.org>, Wolfram Sang
	<wsa+renesas@sang-engineering.com>, "open list:RADEON and AMDGPU DRM DRIVERS"
	<amd-gfx@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>, Zhi Wang <zhiwang@kernel.org>
Subject: Re: [PATCH v2 03/12] drm/i915: Make I2C terminology more inclusive
Message-ID: <ZjU8NB-71xWI2X73@intel.com>
References: <20240503181333.2336999-1-eahariha@linux.microsoft.com>
 <20240503181333.2336999-4-eahariha@linux.microsoft.com>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <20240503181333.2336999-4-eahariha@linux.microsoft.com>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?XG43Dvc0G4BU2vDZ6vleA1qEkWRc3k9vgNWXfei5NhlpW/T5YFW9lUBdyt9L?=
 =?us-ascii?Q?EDAcJsUKXZL9IzIqbmTCHMpritv3DVaqIlAbOMkHWTpNIOEDW5Mddv+wzKid?=
 =?us-ascii?Q?8eNRKLJuQrMj6nokdxo1gMXTRx0Hh0mHl8KkEio/bv9bRvbDZS0YgCNKCMR9?=
 =?us-ascii?Q?Klr13ud2YSUae69C9PzNsX7AIZPalpST+f53S/RDBI8g/TufMfaxvU6HOQcE?=
 =?us-ascii?Q?KuCxQcTA9Rn1DLnH9WMbCc4khx7L8XYCGcgLuJeTnKIlJ4uVGwVBqcPPaTZy?=
 =?us-ascii?Q?mIetA7qGnMO5rgQ9Pa7qH/VyslIgtVMZMPYJUDQJy4K2y1hAzdmpbPeu/GOE?=
 =?us-ascii?Q?i2eMv9WLqEnqicBC75i03S9p2ilGxL7vehXcRhFtfVxka+p7oC0apQkkE3SL?=
 =?us-ascii?Q?OxpdlfjjcaJGb0YWR0pUOO1pn0Z/YcFK6PvjLIJ3MFBsI7dINyyvlB/uHtLY?=
 =?us-ascii?Q?fPEjKFuFQJuQpZkl/PUjmFmXqMBO1wUSLdeaLsf6L1+gdtFJatqrVHHTbdYy?=
 =?us-ascii?Q?iaO3uwHNO24n/IONZQi5kofj+u80G0mgyewFapwoAEEl3tLJJNrb9KzKiXuf?=
 =?us-ascii?Q?nu+1SiN0sS5k1lUeFkmWHPIiSRC6a+GWlvrsXQhDV8PBhawCtugIwCDlX0t/?=
 =?us-ascii?Q?lZLeISfLI6f/dsgoD19vK1fMR+XiKdZ0DOPtx8MEhbXZEnkAIYTLAIf/pGYK?=
 =?us-ascii?Q?bIv79ReNUZaF5Xtq+BL/OnI6T6ggZ2IZs4InyzGlOdL/dn0IP5w75U/WewFj?=
 =?us-ascii?Q?rvMHdb3ThvMzSC7Es1NM+zJ86QTh8iqSgQ20yfEaC6g/I9ujLYN6y/SWenHE?=
 =?us-ascii?Q?LTbP4wFfnxOf+xfapryY+0jAjUN3CiXwPTMXLKxxPFbyNXSIkRvQXlfkri6c?=
 =?us-ascii?Q?1oyZDrNmW2QU7eSz7hOdcXxl7GqQ/iSbAyCJJwOwZS4LCTdmHiw1oUCDjYpG?=
 =?us-ascii?Q?ki2wicoa094w17PkysBCi26xUB88MyHnKnyxacoJrGLU2dpwdDo0NfG7PB8Y?=
 =?us-ascii?Q?5YasqNGj+jonyBanXepdHvV55h9kEZqzlejt01o9lQspxbZmzcXda1EYLqLD?=
 =?us-ascii?Q?1Ks2LRisvyGOJ89aqig2Bc/gsG5PxC3DLgaZ32cA/PJXXjr4NWUTc4Ngzx51?=
 =?us-ascii?Q?/B3UR0YwPwNsfkOzFKAplYcx/FJFlBOYxdUPGJ4Qip4Rg3TakBI12k5XEZYk?=
 =?us-ascii?Q?w0ciycqsHKq3dUzleChnPJPPGY6j4E2b6OvANTIsXRqy58vaXenihYaG0w/n?=
 =?us-ascii?Q?TMxkzPOhxER2mSAeYMDIZLUWBxnX7u7dr6yPN2AHSHEp7rZsQvfiVuWrE+HJ?=
 =?us-ascii?Q?TLG2RGLez56MxYnZBZokaI4HGIKomwWFweYdmzsNHxp9/DktaymxSCzKYd0w?=
 =?us-ascii?Q?dl8Xi/785Yk1PDDqLQqljE9Kahii5CAFvmjBW1y0BHCZ/wZEEKFXfrionYnY?=
 =?us-ascii?Q?WWBr7JR1UdDrDZuTW0cpBUgL0awp1JYaxIw8AidlzI4cDfmvNWjai5KsN9Db?=
 =?us-ascii?Q?x7EEB/9Kz6h/FMOemgXB1PyTl93buW24v8kUOzDbgT8CO6wyP3vmQeet6WJl?=
 =?us-ascii?Q?uO7X0+e19GwIWaPBrI2lvN2/khinXERv7JAgGRM79sdn8tJq2j7F1A9WLadI?=
 =?us-ascii?Q?DQ=3D=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 669e8ba4-67f6-4f77-651f-08dc6ba8066d
X-MS-Exchange-CrossTenant-AuthSource: MN0PR11MB6059.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 May 2024 19:34:18.9634
 (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: GADHC+6gtpsR4YJvZliRg+WTNhS+oTDdUvWPVZqFr9bJfhk2l8TYnmYPAA5BTBuakRbn+jn5XqZMUT4fNMJf7g==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY8PR11MB7193
X-OriginatorOrg: intel.com

On Fri, May 03, 2024 at 06:13:24PM +0000, Easwar Hariharan wrote:
> I2C v7, SMBus 3.2, and I3C 1.1.1 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.
> 
> Compile tested, no functionality changes intended
> 
> [1]: https://lore.kernel.org/all/20240322132619.6389-1-wsa+renesas@sang-engineering.com/
> 
> Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> Acked-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

It looks like the ack is not needed since we are merging this through
drm-intel-next. But I'm planing to merge this only after seeing the
main drivers/i2c accepting the new terminology. So we don't have a
risk of that getting push back and new names there and we having
to rename it once again.

(more below)

> Acked-by: Zhi Wang <zhiwang@kernel.org>
> Signed-off-by: Easwar Hariharan <eahariha@linux.microsoft.com>

Cc: Jani Nikula <jani.nikula@intel.com>

Jani, what bits were you concerned that were not necessarily i2c?
I believe although not necessarily/directly i2c, I believe they
are related and could benefit from the massive single shot renable.
or do you have any better split to suggest here?

(more below)

> ---
>  drivers/gpu/drm/i915/display/dvo_ch7017.c     | 14 ++++-----
>  drivers/gpu/drm/i915/display/dvo_ch7xxx.c     | 18 +++++------
>  drivers/gpu/drm/i915/display/dvo_ivch.c       | 16 +++++-----
>  drivers/gpu/drm/i915/display/dvo_ns2501.c     | 18 +++++------
>  drivers/gpu/drm/i915/display/dvo_sil164.c     | 18 +++++------
>  drivers/gpu/drm/i915/display/dvo_tfp410.c     | 18 +++++------
>  drivers/gpu/drm/i915/display/intel_bios.c     | 22 +++++++-------
>  drivers/gpu/drm/i915/display/intel_ddi.c      |  2 +-
>  .../gpu/drm/i915/display/intel_display_core.h |  2 +-
>  drivers/gpu/drm/i915/display/intel_dsi.h      |  2 +-
>  drivers/gpu/drm/i915/display/intel_dsi_vbt.c  | 20 ++++++-------
>  drivers/gpu/drm/i915/display/intel_dvo.c      | 14 ++++-----
>  drivers/gpu/drm/i915/display/intel_dvo_dev.h  |  2 +-
>  drivers/gpu/drm/i915/display/intel_gmbus.c    |  4 +--
>  drivers/gpu/drm/i915/display/intel_sdvo.c     | 30 +++++++++----------
>  drivers/gpu/drm/i915/display/intel_vbt_defs.h |  4 +--
>  drivers/gpu/drm/i915/gvt/edid.c               | 28 ++++++++---------
>  drivers/gpu/drm/i915/gvt/edid.h               |  4 +--
>  drivers/gpu/drm/i915/gvt/opregion.c           |  2 +-
>  19 files changed, 119 insertions(+), 119 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/dvo_ch7017.c b/drivers/gpu/drm/i915/display/dvo_ch7017.c
> index d0c3880d7f80..493e730c685b 100644
> --- a/drivers/gpu/drm/i915/display/dvo_ch7017.c
> +++ b/drivers/gpu/drm/i915/display/dvo_ch7017.c
> @@ -170,13 +170,13 @@ static bool ch7017_read(struct intel_dvo_device *dvo, u8 addr, u8 *val)
>  {
>  	struct i2c_msg msgs[] = {
>  		{
> -			.addr = dvo->slave_addr,
> +			.addr = dvo->target_addr,
>  			.flags = 0,
>  			.len = 1,
>  			.buf = &addr,
>  		},
>  		{
> -			.addr = dvo->slave_addr,
> +			.addr = dvo->target_addr,
>  			.flags = I2C_M_RD,
>  			.len = 1,
>  			.buf = val,
> @@ -189,7 +189,7 @@ static bool ch7017_write(struct intel_dvo_device *dvo, u8 addr, u8 val)
>  {
>  	u8 buf[2] = { addr, val };
>  	struct i2c_msg msg = {
> -		.addr = dvo->slave_addr,
> +		.addr = dvo->target_addr,
>  		.flags = 0,
>  		.len = 2,
>  		.buf = buf,
> @@ -197,7 +197,7 @@ static bool ch7017_write(struct intel_dvo_device *dvo, u8 addr, u8 val)
>  	return i2c_transfer(dvo->i2c_bus, &msg, 1) == 1;
>  }
>  
> -/** Probes for a CH7017 on the given bus and slave address. */
> +/** Probes for a CH7017 on the given bus and target address. */
>  static bool ch7017_init(struct intel_dvo_device *dvo,
>  			struct i2c_adapter *adapter)
>  {
> @@ -227,13 +227,13 @@ static bool ch7017_init(struct intel_dvo_device *dvo,
>  		break;
>  	default:
>  		DRM_DEBUG_KMS("ch701x not detected, got %d: from %s "
> -			      "slave %d.\n",
> -			      val, adapter->name, dvo->slave_addr);
> +			      "target %d.\n",
> +			      val, adapter->name, dvo->target_addr);
>  		goto fail;
>  	}
>  
>  	DRM_DEBUG_KMS("%s detected on %s, addr %d\n",
> -		      str, adapter->name, dvo->slave_addr);
> +		      str, adapter->name, dvo->target_addr);
>  	return true;
>  
>  fail:
> diff --git a/drivers/gpu/drm/i915/display/dvo_ch7xxx.c b/drivers/gpu/drm/i915/display/dvo_ch7xxx.c
> index 2e8e85da5a40..534b8544e0a4 100644
> --- a/drivers/gpu/drm/i915/display/dvo_ch7xxx.c
> +++ b/drivers/gpu/drm/i915/display/dvo_ch7xxx.c
> @@ -153,13 +153,13 @@ static bool ch7xxx_readb(struct intel_dvo_device *dvo, int addr, u8 *ch)
>  
>  	struct i2c_msg msgs[] = {
>  		{
> -			.addr = dvo->slave_addr,
> +			.addr = dvo->target_addr,
>  			.flags = 0,
>  			.len = 1,
>  			.buf = out_buf,
>  		},
>  		{
> -			.addr = dvo->slave_addr,
> +			.addr = dvo->target_addr,
>  			.flags = I2C_M_RD,
>  			.len = 1,
>  			.buf = in_buf,
> @@ -176,7 +176,7 @@ static bool ch7xxx_readb(struct intel_dvo_device *dvo, int addr, u8 *ch)
>  
>  	if (!ch7xxx->quiet) {
>  		DRM_DEBUG_KMS("Unable to read register 0x%02x from %s:%02x.\n",
> -			  addr, adapter->name, dvo->slave_addr);
> +			  addr, adapter->name, dvo->target_addr);
>  	}
>  	return false;
>  }
> @@ -188,7 +188,7 @@ static bool ch7xxx_writeb(struct intel_dvo_device *dvo, int addr, u8 ch)
>  	struct i2c_adapter *adapter = dvo->i2c_bus;
>  	u8 out_buf[2];
>  	struct i2c_msg msg = {
> -		.addr = dvo->slave_addr,
> +		.addr = dvo->target_addr,
>  		.flags = 0,
>  		.len = 2,
>  		.buf = out_buf,
> @@ -202,7 +202,7 @@ static bool ch7xxx_writeb(struct intel_dvo_device *dvo, int addr, u8 ch)
>  
>  	if (!ch7xxx->quiet) {
>  		DRM_DEBUG_KMS("Unable to write register 0x%02x to %s:%d.\n",
> -			  addr, adapter->name, dvo->slave_addr);
> +			  addr, adapter->name, dvo->target_addr);
>  	}
>  
>  	return false;
> @@ -229,8 +229,8 @@ static bool ch7xxx_init(struct intel_dvo_device *dvo,
>  
>  	name = ch7xxx_get_id(vendor);
>  	if (!name) {
> -		DRM_DEBUG_KMS("ch7xxx not detected; got VID 0x%02x from %s slave %d.\n",
> -			      vendor, adapter->name, dvo->slave_addr);
> +		DRM_DEBUG_KMS("ch7xxx not detected; got VID 0x%02x from %s target %d.\n",
> +			      vendor, adapter->name, dvo->target_addr);
>  		goto out;
>  	}
>  
> @@ -240,8 +240,8 @@ static bool ch7xxx_init(struct intel_dvo_device *dvo,
>  
>  	devid = ch7xxx_get_did(device);
>  	if (!devid) {
> -		DRM_DEBUG_KMS("ch7xxx not detected; got DID 0x%02x from %s slave %d.\n",
> -			      device, adapter->name, dvo->slave_addr);
> +		DRM_DEBUG_KMS("ch7xxx not detected; got DID 0x%02x from %s target %d.\n",
> +			      device, adapter->name, dvo->target_addr);
>  		goto out;
>  	}
>  
> diff --git a/drivers/gpu/drm/i915/display/dvo_ivch.c b/drivers/gpu/drm/i915/display/dvo_ivch.c
> index eef72bb3b767..0d5cce6051b1 100644
> --- a/drivers/gpu/drm/i915/display/dvo_ivch.c
> +++ b/drivers/gpu/drm/i915/display/dvo_ivch.c
> @@ -198,7 +198,7 @@ static bool ivch_read(struct intel_dvo_device *dvo, int addr, u16 *data)
>  
>  	struct i2c_msg msgs[] = {
>  		{
> -			.addr = dvo->slave_addr,
> +			.addr = dvo->target_addr,
>  			.flags = I2C_M_RD,
>  			.len = 0,
>  		},
> @@ -209,7 +209,7 @@ static bool ivch_read(struct intel_dvo_device *dvo, int addr, u16 *data)
>  			.buf = out_buf,
>  		},
>  		{
> -			.addr = dvo->slave_addr,
> +			.addr = dvo->target_addr,
>  			.flags = I2C_M_RD | I2C_M_NOSTART,
>  			.len = 2,
>  			.buf = in_buf,
> @@ -226,7 +226,7 @@ static bool ivch_read(struct intel_dvo_device *dvo, int addr, u16 *data)
>  	if (!priv->quiet) {
>  		DRM_DEBUG_KMS("Unable to read register 0x%02x from "
>  				"%s:%02x.\n",
> -			  addr, adapter->name, dvo->slave_addr);
> +			  addr, adapter->name, dvo->target_addr);
>  	}
>  	return false;
>  }
> @@ -238,7 +238,7 @@ static bool ivch_write(struct intel_dvo_device *dvo, int addr, u16 data)
>  	struct i2c_adapter *adapter = dvo->i2c_bus;
>  	u8 out_buf[3];
>  	struct i2c_msg msg = {
> -		.addr = dvo->slave_addr,
> +		.addr = dvo->target_addr,
>  		.flags = 0,
>  		.len = 3,
>  		.buf = out_buf,
> @@ -253,13 +253,13 @@ static bool ivch_write(struct intel_dvo_device *dvo, int addr, u16 data)
>  
>  	if (!priv->quiet) {
>  		DRM_DEBUG_KMS("Unable to write register 0x%02x to %s:%d.\n",
> -			  addr, adapter->name, dvo->slave_addr);
> +			  addr, adapter->name, dvo->target_addr);
>  	}
>  
>  	return false;
>  }
>  
> -/* Probes the given bus and slave address for an ivch */
> +/* Probes the given bus and target address for an ivch */
>  static bool ivch_init(struct intel_dvo_device *dvo,
>  		      struct i2c_adapter *adapter)
>  {
> @@ -283,10 +283,10 @@ static bool ivch_init(struct intel_dvo_device *dvo,
>  	 * very unique, check that the value in the base address field matches
>  	 * the address it's responding on.
>  	 */
> -	if ((temp & VR00_BASE_ADDRESS_MASK) != dvo->slave_addr) {
> +	if ((temp & VR00_BASE_ADDRESS_MASK) != dvo->target_addr) {
>  		DRM_DEBUG_KMS("ivch detect failed due to address mismatch "
>  			  "(%d vs %d)\n",
> -			  (temp & VR00_BASE_ADDRESS_MASK), dvo->slave_addr);
> +			  (temp & VR00_BASE_ADDRESS_MASK), dvo->target_addr);
>  		goto out;
>  	}
>  
> diff --git a/drivers/gpu/drm/i915/display/dvo_ns2501.c b/drivers/gpu/drm/i915/display/dvo_ns2501.c
> index 1df212fb000e..43fc0374fc7f 100644
> --- a/drivers/gpu/drm/i915/display/dvo_ns2501.c
> +++ b/drivers/gpu/drm/i915/display/dvo_ns2501.c
> @@ -399,13 +399,13 @@ static bool ns2501_readb(struct intel_dvo_device *dvo, int addr, u8 *ch)
>  
>  	struct i2c_msg msgs[] = {
>  		{
> -		 .addr = dvo->slave_addr,
> +		 .addr = dvo->target_addr,
>  		 .flags = 0,
>  		 .len = 1,
>  		 .buf = out_buf,
>  		 },
>  		{
> -		 .addr = dvo->slave_addr,
> +		 .addr = dvo->target_addr,
>  		 .flags = I2C_M_RD,
>  		 .len = 1,
>  		 .buf = in_buf,
> @@ -423,7 +423,7 @@ static bool ns2501_readb(struct intel_dvo_device *dvo, int addr, u8 *ch)
>  	if (!ns->quiet) {
>  		DRM_DEBUG_KMS
>  		    ("Unable to read register 0x%02x from %s:0x%02x.\n", addr,
> -		     adapter->name, dvo->slave_addr);
> +		     adapter->name, dvo->target_addr);
>  	}
>  
>  	return false;
> @@ -442,7 +442,7 @@ static bool ns2501_writeb(struct intel_dvo_device *dvo, int addr, u8 ch)
>  	u8 out_buf[2];
>  
>  	struct i2c_msg msg = {
> -		.addr = dvo->slave_addr,
> +		.addr = dvo->target_addr,
>  		.flags = 0,
>  		.len = 2,
>  		.buf = out_buf,
> @@ -457,7 +457,7 @@ static bool ns2501_writeb(struct intel_dvo_device *dvo, int addr, u8 ch)
>  
>  	if (!ns->quiet) {
>  		DRM_DEBUG_KMS("Unable to write register 0x%02x to %s:%d\n",
> -			      addr, adapter->name, dvo->slave_addr);
> +			      addr, adapter->name, dvo->target_addr);
>  	}
>  
>  	return false;
> @@ -488,8 +488,8 @@ static bool ns2501_init(struct intel_dvo_device *dvo,
>  		goto out;
>  
>  	if (ch != (NS2501_VID & 0xff)) {
> -		DRM_DEBUG_KMS("ns2501 not detected got %d: from %s Slave %d.\n",
> -			      ch, adapter->name, dvo->slave_addr);
> +		DRM_DEBUG_KMS("ns2501 not detected got %d: from %s Target %d.\n",
> +			      ch, adapter->name, dvo->target_addr);
>  		goto out;
>  	}
>  
> @@ -497,8 +497,8 @@ static bool ns2501_init(struct intel_dvo_device *dvo,
>  		goto out;
>  
>  	if (ch != (NS2501_DID & 0xff)) {
> -		DRM_DEBUG_KMS("ns2501 not detected got %d: from %s Slave %d.\n",
> -			      ch, adapter->name, dvo->slave_addr);
> +		DRM_DEBUG_KMS("ns2501 not detected got %d: from %s Target %d.\n",
> +			      ch, adapter->name, dvo->target_addr);
>  		goto out;
>  	}
>  	ns->quiet = false;
> diff --git a/drivers/gpu/drm/i915/display/dvo_sil164.c b/drivers/gpu/drm/i915/display/dvo_sil164.c
> index 6c461024c8e3..a8dd40c00997 100644
> --- a/drivers/gpu/drm/i915/display/dvo_sil164.c
> +++ b/drivers/gpu/drm/i915/display/dvo_sil164.c
> @@ -79,13 +79,13 @@ static bool sil164_readb(struct intel_dvo_device *dvo, int addr, u8 *ch)
>  
>  	struct i2c_msg msgs[] = {
>  		{
> -			.addr = dvo->slave_addr,
> +			.addr = dvo->target_addr,
>  			.flags = 0,
>  			.len = 1,
>  			.buf = out_buf,
>  		},
>  		{
> -			.addr = dvo->slave_addr,
> +			.addr = dvo->target_addr,
>  			.flags = I2C_M_RD,
>  			.len = 1,
>  			.buf = in_buf,
> @@ -102,7 +102,7 @@ static bool sil164_readb(struct intel_dvo_device *dvo, int addr, u8 *ch)
>  
>  	if (!sil->quiet) {
>  		DRM_DEBUG_KMS("Unable to read register 0x%02x from %s:%02x.\n",
> -			  addr, adapter->name, dvo->slave_addr);
> +			  addr, adapter->name, dvo->target_addr);
>  	}
>  	return false;
>  }
> @@ -113,7 +113,7 @@ static bool sil164_writeb(struct intel_dvo_device *dvo, int addr, u8 ch)
>  	struct i2c_adapter *adapter = dvo->i2c_bus;
>  	u8 out_buf[2];
>  	struct i2c_msg msg = {
> -		.addr = dvo->slave_addr,
> +		.addr = dvo->target_addr,
>  		.flags = 0,
>  		.len = 2,
>  		.buf = out_buf,
> @@ -127,7 +127,7 @@ static bool sil164_writeb(struct intel_dvo_device *dvo, int addr, u8 ch)
>  
>  	if (!sil->quiet) {
>  		DRM_DEBUG_KMS("Unable to write register 0x%02x to %s:%d.\n",
> -			  addr, adapter->name, dvo->slave_addr);
> +			  addr, adapter->name, dvo->target_addr);
>  	}
>  
>  	return false;
> @@ -153,8 +153,8 @@ static bool sil164_init(struct intel_dvo_device *dvo,
>  		goto out;
>  
>  	if (ch != (SIL164_VID & 0xff)) {
> -		DRM_DEBUG_KMS("sil164 not detected got %d: from %s Slave %d.\n",
> -			  ch, adapter->name, dvo->slave_addr);
> +		DRM_DEBUG_KMS("sil164 not detected got %d: from %s Target %d.\n",
> +			  ch, adapter->name, dvo->target_addr);
>  		goto out;
>  	}
>  
> @@ -162,8 +162,8 @@ static bool sil164_init(struct intel_dvo_device *dvo,
>  		goto out;
>  
>  	if (ch != (SIL164_DID & 0xff)) {
> -		DRM_DEBUG_KMS("sil164 not detected got %d: from %s Slave %d.\n",
> -			  ch, adapter->name, dvo->slave_addr);
> +		DRM_DEBUG_KMS("sil164 not detected got %d: from %s Target %d.\n",
> +			  ch, adapter->name, dvo->target_addr);
>  		goto out;
>  	}
>  	sil->quiet = false;
> diff --git a/drivers/gpu/drm/i915/display/dvo_tfp410.c b/drivers/gpu/drm/i915/display/dvo_tfp410.c
> index 0939e097f4f9..d9a0cd753a87 100644
> --- a/drivers/gpu/drm/i915/display/dvo_tfp410.c
> +++ b/drivers/gpu/drm/i915/display/dvo_tfp410.c
> @@ -100,13 +100,13 @@ static bool tfp410_readb(struct intel_dvo_device *dvo, int addr, u8 *ch)
>  
>  	struct i2c_msg msgs[] = {
>  		{
> -			.addr = dvo->slave_addr,
> +			.addr = dvo->target_addr,
>  			.flags = 0,
>  			.len = 1,
>  			.buf = out_buf,
>  		},
>  		{
> -			.addr = dvo->slave_addr,
> +			.addr = dvo->target_addr,
>  			.flags = I2C_M_RD,
>  			.len = 1,
>  			.buf = in_buf,
> @@ -123,7 +123,7 @@ static bool tfp410_readb(struct intel_dvo_device *dvo, int addr, u8 *ch)
>  
>  	if (!tfp->quiet) {
>  		DRM_DEBUG_KMS("Unable to read register 0x%02x from %s:%02x.\n",
> -			  addr, adapter->name, dvo->slave_addr);
> +			  addr, adapter->name, dvo->target_addr);
>  	}
>  	return false;
>  }
> @@ -134,7 +134,7 @@ static bool tfp410_writeb(struct intel_dvo_device *dvo, int addr, u8 ch)
>  	struct i2c_adapter *adapter = dvo->i2c_bus;
>  	u8 out_buf[2];
>  	struct i2c_msg msg = {
> -		.addr = dvo->slave_addr,
> +		.addr = dvo->target_addr,
>  		.flags = 0,
>  		.len = 2,
>  		.buf = out_buf,
> @@ -148,7 +148,7 @@ static bool tfp410_writeb(struct intel_dvo_device *dvo, int addr, u8 ch)
>  
>  	if (!tfp->quiet) {
>  		DRM_DEBUG_KMS("Unable to write register 0x%02x to %s:%d.\n",
> -			  addr, adapter->name, dvo->slave_addr);
> +			  addr, adapter->name, dvo->target_addr);
>  	}
>  
>  	return false;
> @@ -183,15 +183,15 @@ static bool tfp410_init(struct intel_dvo_device *dvo,
>  
>  	if ((id = tfp410_getid(dvo, TFP410_VID_LO)) != TFP410_VID) {
>  		DRM_DEBUG_KMS("tfp410 not detected got VID %X: from %s "
> -				"Slave %d.\n",
> -			  id, adapter->name, dvo->slave_addr);
> +				"Target %d.\n",
> +			  id, adapter->name, dvo->target_addr);
>  		goto out;
>  	}
>  
>  	if ((id = tfp410_getid(dvo, TFP410_DID_LO)) != TFP410_DID) {
>  		DRM_DEBUG_KMS("tfp410 not detected got DID %X: from %s "
> -				"Slave %d.\n",
> -			  id, adapter->name, dvo->slave_addr);
> +				"Target %d.\n",
> +			  id, adapter->name, dvo->target_addr);
>  		goto out;
>  	}
>  	tfp->quiet = false;
> diff --git a/drivers/gpu/drm/i915/display/intel_bios.c b/drivers/gpu/drm/i915/display/intel_bios.c
> index 52bd3576835b..3d4ecfc9462a 100644
> --- a/drivers/gpu/drm/i915/display/intel_bios.c
> +++ b/drivers/gpu/drm/i915/display/intel_bios.c
> @@ -69,8 +69,8 @@ struct intel_bios_encoder_data {
>  	struct list_head node;
>  };
>  
> -#define	SLAVE_ADDR1	0x70
> -#define	SLAVE_ADDR2	0x72
> +#define	TARGET_ADDR1	0x70
> +#define	TARGET_ADDR2	0x72
>  
>  /* Get BDB block size given a pointer to Block ID. */
>  static u32 _get_blocksize(const u8 *block_base)
> @@ -1231,10 +1231,10 @@ parse_sdvo_device_mapping(struct drm_i915_private *i915)
>  		const struct child_device_config *child = &devdata->child;
>  		struct sdvo_device_mapping *mapping;
>  
> -		if (child->slave_addr != SLAVE_ADDR1 &&
> -		    child->slave_addr != SLAVE_ADDR2) {
> +		if (child->target_addr != TARGET_ADDR1 &&
> +		    child->target_addr != TARGET_ADDR2) {
>  			/*
> -			 * If the slave address is neither 0x70 nor 0x72,
> +			 * If the target address is neither 0x70 nor 0x72,
>  			 * it is not a SDVO device. Skip it.
>  			 */
>  			continue;
> @@ -1247,22 +1247,22 @@ parse_sdvo_device_mapping(struct drm_i915_private *i915)
>  			continue;
>  		}
>  		drm_dbg_kms(&i915->drm,
> -			    "the SDVO device with slave addr %2x is found on"
> +			    "the SDVO device with target addr %2x is found on"
>  			    " %s port\n",
> -			    child->slave_addr,
> +			    child->target_addr,
>  			    (child->dvo_port == DEVICE_PORT_DVOB) ?
>  			    "SDVOB" : "SDVOC");
>  		mapping = &i915->display.vbt.sdvo_mappings[child->dvo_port - 1];
>  		if (!mapping->initialized) {
>  			mapping->dvo_port = child->dvo_port;
> -			mapping->slave_addr = child->slave_addr;
> +			mapping->target_addr = child->target_addr;
>  			mapping->dvo_wiring = child->dvo_wiring;
>  			mapping->ddc_pin = child->ddc_pin;
>  			mapping->i2c_pin = child->i2c_pin;
>  			mapping->initialized = 1;
>  			drm_dbg_kms(&i915->drm,
>  				    "SDVO device: dvo=%x, addr=%x, wiring=%d, ddc_pin=%d, i2c_pin=%d\n",
> -				    mapping->dvo_port, mapping->slave_addr,
> +				    mapping->dvo_port, mapping->target_addr,
>  				    mapping->dvo_wiring, mapping->ddc_pin,
>  				    mapping->i2c_pin);
>  		} else {
> @@ -1270,11 +1270,11 @@ parse_sdvo_device_mapping(struct drm_i915_private *i915)
>  				    "Maybe one SDVO port is shared by "
>  				    "two SDVO device.\n");
>  		}
> -		if (child->slave2_addr) {
> +		if (child->target2_addr) {
>  			/* Maybe this is a SDVO device with multiple inputs */
>  			/* And the mapping info is not added */
>  			drm_dbg_kms(&i915->drm,
> -				    "there exists the slave2_addr. Maybe this"
> +				    "there exists the target2_addr. Maybe this"
>  				    " is a SDVO device with multiple inputs.\n");
>  		}
>  		count++;
> diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c b/drivers/gpu/drm/i915/display/intel_ddi.c
> index c17462b4c2ac..64db211148a8 100644
> --- a/drivers/gpu/drm/i915/display/intel_ddi.c
> +++ b/drivers/gpu/drm/i915/display/intel_ddi.c
> @@ -4332,7 +4332,7 @@ static int intel_ddi_compute_config_late(struct intel_encoder *encoder,
>  									connector->tile_group->id);
>  
>  	/*
> -	 * EDP Transcoders cannot be ensalved
> +	 * EDP Transcoders cannot be slaves

                                     ^ here
perhaps you meant 'targeted' ?

>  	 * make them a master always when present
>  	 */
>  	if (port_sync_transcoders & BIT(TRANSCODER_EDP))
> diff --git a/drivers/gpu/drm/i915/display/intel_display_core.h b/drivers/gpu/drm/i915/display/intel_display_core.h
> index 2167dbee5eea..5bfc91f0b563 100644
> --- a/drivers/gpu/drm/i915/display/intel_display_core.h
> +++ b/drivers/gpu/drm/i915/display/intel_display_core.h
> @@ -236,7 +236,7 @@ struct intel_vbt_data {
>  	struct sdvo_device_mapping {
>  		u8 initialized;
>  		u8 dvo_port;
> -		u8 slave_addr;
> +		u8 target_addr;
>  		u8 dvo_wiring;
>  		u8 i2c_pin;
>  		u8 ddc_pin;
> diff --git a/drivers/gpu/drm/i915/display/intel_dsi.h b/drivers/gpu/drm/i915/display/intel_dsi.h
> index e99c94edfaae..e8ba4ccd99d3 100644
> --- a/drivers/gpu/drm/i915/display/intel_dsi.h
> +++ b/drivers/gpu/drm/i915/display/intel_dsi.h
> @@ -66,7 +66,7 @@ struct intel_dsi {
>  	/* number of DSI lanes */
>  	unsigned int lane_count;
>  
> -	/* i2c bus associated with the slave device */
> +	/* i2c bus associated with the target device */
>  	int i2c_bus_num;
>  
>  	/*
> diff --git a/drivers/gpu/drm/i915/display/intel_dsi_vbt.c b/drivers/gpu/drm/i915/display/intel_dsi_vbt.c
> index a5d7fc8418c9..fb0b02e30c8b 100644
> --- a/drivers/gpu/drm/i915/display/intel_dsi_vbt.c
> +++ b/drivers/gpu/drm/i915/display/intel_dsi_vbt.c
> @@ -56,7 +56,7 @@
>  #define MIPI_PORT_SHIFT			3
>  
>  struct i2c_adapter_lookup {
> -	u16 slave_addr;
> +	u16 target_addr;
>  	struct intel_dsi *intel_dsi;
>  	acpi_handle dev_handle;
>  };
> @@ -443,7 +443,7 @@ static int i2c_adapter_lookup(struct acpi_resource *ares, void *data)
>  	if (!i2c_acpi_get_i2c_resource(ares, &sb))
>  		return 1;
>  
> -	if (lookup->slave_addr != sb->slave_address)
> +	if (lookup->target_addr != sb->slave_address)
>  		return 1;
>  
>  	status = acpi_get_handle(lookup->dev_handle,
> @@ -460,12 +460,12 @@ static int i2c_adapter_lookup(struct acpi_resource *ares, void *data)
>  }
>  
>  static void i2c_acpi_find_adapter(struct intel_dsi *intel_dsi,
> -				  const u16 slave_addr)
> +				  const u16 target_addr)
>  {
>  	struct drm_device *drm_dev = intel_dsi->base.base.dev;
>  	struct acpi_device *adev = ACPI_COMPANION(drm_dev->dev);
>  	struct i2c_adapter_lookup lookup = {
> -		.slave_addr = slave_addr,
> +		.target_addr = target_addr,
>  		.intel_dsi = intel_dsi,
>  		.dev_handle = acpi_device_handle(adev),
>  	};
> @@ -476,7 +476,7 @@ static void i2c_acpi_find_adapter(struct intel_dsi *intel_dsi,
>  }
>  #else
>  static inline void i2c_acpi_find_adapter(struct intel_dsi *intel_dsi,
> -					 const u16 slave_addr)
> +					 const u16 target_addr)
>  {
>  }
>  #endif
> @@ -488,17 +488,17 @@ static const u8 *mipi_exec_i2c(struct intel_dsi *intel_dsi, const u8 *data)
>  	struct i2c_msg msg;
>  	int ret;
>  	u8 vbt_i2c_bus_num = *(data + 2);
> -	u16 slave_addr = *(u16 *)(data + 3);
> +	u16 target_addr = *(u16 *)(data + 3);
>  	u8 reg_offset = *(data + 5);
>  	u8 payload_size = *(data + 6);
>  	u8 *payload_data;
>  
> -	drm_dbg_kms(&i915->drm, "bus %d client-addr 0x%02x reg 0x%02x data %*ph\n",
> -		    vbt_i2c_bus_num, slave_addr, reg_offset, payload_size, data + 7);
> +	drm_dbg_kms(&i915->drm, "bus %d target-addr 0x%02x reg 0x%02x data %*ph\n",
> +		    vbt_i2c_bus_num, target_addr, reg_offset, payload_size, data + 7);
>  
>  	if (intel_dsi->i2c_bus_num < 0) {
>  		intel_dsi->i2c_bus_num = vbt_i2c_bus_num;
> -		i2c_acpi_find_adapter(intel_dsi, slave_addr);
> +		i2c_acpi_find_adapter(intel_dsi, target_addr);
>  	}
>  
>  	adapter = i2c_get_adapter(intel_dsi->i2c_bus_num);
> @@ -514,7 +514,7 @@ static const u8 *mipi_exec_i2c(struct intel_dsi *intel_dsi, const u8 *data)
>  	payload_data[0] = reg_offset;
>  	memcpy(&payload_data[1], (data + 7), payload_size);
>  
> -	msg.addr = slave_addr;
> +	msg.addr = target_addr;
>  	msg.flags = 0;
>  	msg.len = payload_size + 1;
>  	msg.buf = payload_data;
> diff --git a/drivers/gpu/drm/i915/display/intel_dvo.c b/drivers/gpu/drm/i915/display/intel_dvo.c
> index c076da75b066..8d4c8f33f776 100644
> --- a/drivers/gpu/drm/i915/display/intel_dvo.c
> +++ b/drivers/gpu/drm/i915/display/intel_dvo.c
> @@ -60,42 +60,42 @@ static const struct intel_dvo_device intel_dvo_devices[] = {
>  		.type = INTEL_DVO_CHIP_TMDS,
>  		.name = "sil164",
>  		.port = PORT_C,
> -		.slave_addr = SIL164_ADDR,
> +		.target_addr = SIL164_ADDR,
>  		.dev_ops = &sil164_ops,
>  	},
>  	{
>  		.type = INTEL_DVO_CHIP_TMDS,
>  		.name = "ch7xxx",
>  		.port = PORT_C,
> -		.slave_addr = CH7xxx_ADDR,
> +		.target_addr = CH7xxx_ADDR,
>  		.dev_ops = &ch7xxx_ops,
>  	},
>  	{
>  		.type = INTEL_DVO_CHIP_TMDS,
>  		.name = "ch7xxx",
>  		.port = PORT_C,
> -		.slave_addr = 0x75, /* For some ch7010 */
> +		.target_addr = 0x75, /* For some ch7010 */
>  		.dev_ops = &ch7xxx_ops,
>  	},
>  	{
>  		.type = INTEL_DVO_CHIP_LVDS,
>  		.name = "ivch",
>  		.port = PORT_A,
> -		.slave_addr = 0x02, /* Might also be 0x44, 0x84, 0xc4 */
> +		.target_addr = 0x02, /* Might also be 0x44, 0x84, 0xc4 */
>  		.dev_ops = &ivch_ops,
>  	},
>  	{
>  		.type = INTEL_DVO_CHIP_TMDS,
>  		.name = "tfp410",
>  		.port = PORT_C,
> -		.slave_addr = TFP410_ADDR,
> +		.target_addr = TFP410_ADDR,
>  		.dev_ops = &tfp410_ops,
>  	},
>  	{
>  		.type = INTEL_DVO_CHIP_LVDS,
>  		.name = "ch7017",
>  		.port = PORT_C,
> -		.slave_addr = 0x75,
> +		.target_addr = 0x75,
>  		.gpio = GMBUS_PIN_DPB,
>  		.dev_ops = &ch7017_ops,
>  	},
> @@ -103,7 +103,7 @@ static const struct intel_dvo_device intel_dvo_devices[] = {
>  		.type = INTEL_DVO_CHIP_LVDS_NO_FIXED,
>  		.name = "ns2501",
>  		.port = PORT_B,
> -		.slave_addr = NS2501_ADDR,
> +		.target_addr = NS2501_ADDR,
>  		.dev_ops = &ns2501_ops,
>  	},
>  };
> diff --git a/drivers/gpu/drm/i915/display/intel_dvo_dev.h b/drivers/gpu/drm/i915/display/intel_dvo_dev.h
> index af7b04539b93..4bf476656b8c 100644
> --- a/drivers/gpu/drm/i915/display/intel_dvo_dev.h
> +++ b/drivers/gpu/drm/i915/display/intel_dvo_dev.h
> @@ -38,7 +38,7 @@ struct intel_dvo_device {
>  	enum port port;
>  	/* GPIO register used for i2c bus to control this device */
>  	u32 gpio;
> -	int slave_addr;
> +	int target_addr;
>  
>  	const struct intel_dvo_dev_ops *dev_ops;
>  	void *dev_priv;
> diff --git a/drivers/gpu/drm/i915/display/intel_gmbus.c b/drivers/gpu/drm/i915/display/intel_gmbus.c
> index d3e03ed5b79c..fe9a3c1f0072 100644
> --- a/drivers/gpu/drm/i915/display/intel_gmbus.c
> +++ b/drivers/gpu/drm/i915/display/intel_gmbus.c
> @@ -478,7 +478,7 @@ gmbus_xfer_read_chunk(struct drm_i915_private *i915,
>  /*
>   * HW spec says that 512Bytes in Burst read need special treatment.
>   * But it doesn't talk about other multiple of 256Bytes. And couldn't locate
> - * an I2C slave, which supports such a lengthy burst read too for experiments.
> + * an I2C target, which supports such a lengthy burst read too for experiments.
>   *
>   * So until things get clarified on HW support, to avoid the burst read length
>   * in fold of 256Bytes except 512, max burst read length is fixed at 767Bytes.
> @@ -701,7 +701,7 @@ do_gmbus_xfer(struct i2c_adapter *adapter, struct i2c_msg *msgs, int num,
>  
>  	/* Toggle the Software Clear Interrupt bit. This has the effect
>  	 * of resetting the GMBUS controller and so clearing the
> -	 * BUS_ERROR raised by the slave's NAK.
> +	 * BUS_ERROR raised by the target's NAK.
>  	 */
>  	intel_de_write_fw(i915, GMBUS1(i915), GMBUS_SW_CLR_INT);
>  	intel_de_write_fw(i915, GMBUS1(i915), 0);
> diff --git a/drivers/gpu/drm/i915/display/intel_sdvo.c b/drivers/gpu/drm/i915/display/intel_sdvo.c
> index 0cd9c183f621..cb50bf9c211d 100644
> --- a/drivers/gpu/drm/i915/display/intel_sdvo.c
> +++ b/drivers/gpu/drm/i915/display/intel_sdvo.c
> @@ -95,7 +95,7 @@ struct intel_sdvo {
>  	struct intel_encoder base;
>  
>  	struct i2c_adapter *i2c;
> -	u8 slave_addr;
> +	u8 target_addr;
>  
>  	struct intel_sdvo_ddc ddc[3];
>  
> @@ -255,13 +255,13 @@ static bool intel_sdvo_read_byte(struct intel_sdvo *intel_sdvo, u8 addr, u8 *ch)
>  	struct drm_i915_private *i915 = to_i915(intel_sdvo->base.base.dev);
>  	struct i2c_msg msgs[] = {
>  		{
> -			.addr = intel_sdvo->slave_addr,
> +			.addr = intel_sdvo->target_addr,
>  			.flags = 0,
>  			.len = 1,
>  			.buf = &addr,
>  		},
>  		{
> -			.addr = intel_sdvo->slave_addr,
> +			.addr = intel_sdvo->target_addr,
>  			.flags = I2C_M_RD,
>  			.len = 1,
>  			.buf = ch,
> @@ -483,14 +483,14 @@ static bool __intel_sdvo_write_cmd(struct intel_sdvo *intel_sdvo, u8 cmd,
>  	intel_sdvo_debug_write(intel_sdvo, cmd, args, args_len);
>  
>  	for (i = 0; i < args_len; i++) {
> -		msgs[i].addr = intel_sdvo->slave_addr;
> +		msgs[i].addr = intel_sdvo->target_addr;
>  		msgs[i].flags = 0;
>  		msgs[i].len = 2;
>  		msgs[i].buf = buf + 2 *i;
>  		buf[2*i + 0] = SDVO_I2C_ARG_0 - i;
>  		buf[2*i + 1] = ((u8*)args)[i];
>  	}
> -	msgs[i].addr = intel_sdvo->slave_addr;
> +	msgs[i].addr = intel_sdvo->target_addr;
>  	msgs[i].flags = 0;
>  	msgs[i].len = 2;
>  	msgs[i].buf = buf + 2*i;
> @@ -499,12 +499,12 @@ static bool __intel_sdvo_write_cmd(struct intel_sdvo *intel_sdvo, u8 cmd,
>  
>  	/* the following two are to read the response */
>  	status = SDVO_I2C_CMD_STATUS;
> -	msgs[i+1].addr = intel_sdvo->slave_addr;
> +	msgs[i+1].addr = intel_sdvo->target_addr;
>  	msgs[i+1].flags = 0;
>  	msgs[i+1].len = 1;
>  	msgs[i+1].buf = &status;
>  
> -	msgs[i+2].addr = intel_sdvo->slave_addr;
> +	msgs[i+2].addr = intel_sdvo->target_addr;
>  	msgs[i+2].flags = I2C_M_RD;
>  	msgs[i+2].len = 1;
>  	msgs[i+2].buf = &status;
> @@ -2655,9 +2655,9 @@ intel_sdvo_select_i2c_bus(struct intel_sdvo *sdvo)
>  	else
>  		pin = GMBUS_PIN_DPB;
>  
> -	drm_dbg_kms(&dev_priv->drm, "[ENCODER:%d:%s] I2C pin %d, slave addr 0x%x\n",
> +	drm_dbg_kms(&dev_priv->drm, "[ENCODER:%d:%s] I2C pin %d, target addr 0x%x\n",
>  		    sdvo->base.base.base.id, sdvo->base.base.name,
> -		    pin, sdvo->slave_addr);
> +		    pin, sdvo->target_addr);
>  
>  	sdvo->i2c = intel_gmbus_get_adapter(dev_priv, pin);
>  
> @@ -2683,7 +2683,7 @@ intel_sdvo_is_hdmi_connector(struct intel_sdvo *intel_sdvo)
>  }
>  
>  static u8
> -intel_sdvo_get_slave_addr(struct intel_sdvo *sdvo)
> +intel_sdvo_get_target_addr(struct intel_sdvo *sdvo)
>  {
>  	struct drm_i915_private *dev_priv = to_i915(sdvo->base.base.dev);
>  	const struct sdvo_device_mapping *my_mapping, *other_mapping;
> @@ -2697,15 +2697,15 @@ intel_sdvo_get_slave_addr(struct intel_sdvo *sdvo)
>  	}
>  
>  	/* If the BIOS described our SDVO device, take advantage of it. */
> -	if (my_mapping->slave_addr)
> -		return my_mapping->slave_addr;
> +	if (my_mapping->target_addr)
> +		return my_mapping->target_addr;
>  
>  	/*
>  	 * If the BIOS only described a different SDVO device, use the
>  	 * address that it isn't using.
>  	 */
> -	if (other_mapping->slave_addr) {
> -		if (other_mapping->slave_addr == 0x70)
> +	if (other_mapping->target_addr) {
> +		if (other_mapping->target_addr == 0x70)
>  			return 0x72;
>  		else
>  			return 0x70;
> @@ -3408,7 +3408,7 @@ bool intel_sdvo_init(struct drm_i915_private *dev_priv,
>  			 "SDVO %c", port_name(port));
>  
>  	intel_sdvo->sdvo_reg = sdvo_reg;
> -	intel_sdvo->slave_addr = intel_sdvo_get_slave_addr(intel_sdvo) >> 1;
> +	intel_sdvo->target_addr = intel_sdvo_get_target_addr(intel_sdvo) >> 1;
>  
>  	intel_sdvo_select_i2c_bus(intel_sdvo);
>  
> diff --git a/drivers/gpu/drm/i915/display/intel_vbt_defs.h b/drivers/gpu/drm/i915/display/intel_vbt_defs.h
> index a9f44abfc9fc..c0d5aae980a8 100644
> --- a/drivers/gpu/drm/i915/display/intel_vbt_defs.h
> +++ b/drivers/gpu/drm/i915/display/intel_vbt_defs.h
> @@ -432,7 +432,7 @@ struct child_device_config {
>  	u16 addin_offset;
>  	u8 dvo_port; /* See DEVICE_PORT_* and DVO_PORT_* above */
>  	u8 i2c_pin;
> -	u8 slave_addr;
> +	u8 target_addr;
>  	u8 ddc_pin;
>  	u16 edid_ptr;
>  	u8 dvo_cfg; /* See DEVICE_CFG_* above */
> @@ -441,7 +441,7 @@ struct child_device_config {
>  		struct {
>  			u8 dvo2_port;
>  			u8 i2c2_pin;
> -			u8 slave2_addr;
> +			u8 target2_addr;
>  			u8 ddc2_pin;
>  		} __packed;
>  		struct {
> diff --git a/drivers/gpu/drm/i915/gvt/edid.c b/drivers/gpu/drm/i915/gvt/edid.c
> index af9afdb53c7f..c022dc736045 100644
> --- a/drivers/gpu/drm/i915/gvt/edid.c
> +++ b/drivers/gpu/drm/i915/gvt/edid.c
> @@ -42,8 +42,8 @@
>  #define GMBUS1_TOTAL_BYTES_MASK 0x1ff
>  #define gmbus1_total_byte_count(v) (((v) >> \
>  	GMBUS1_TOTAL_BYTES_SHIFT) & GMBUS1_TOTAL_BYTES_MASK)
> -#define gmbus1_slave_addr(v) (((v) & 0xff) >> 1)
> -#define gmbus1_slave_index(v) (((v) >> 8) & 0xff)
> +#define gmbus1_target_addr(v) (((v) & 0xff) >> 1)
> +#define gmbus1_target_index(v) (((v) >> 8) & 0xff)
>  #define gmbus1_bus_cycle(v) (((v) >> 25) & 0x7)
>  
>  /* GMBUS0 bits definitions */
> @@ -54,7 +54,7 @@ static unsigned char edid_get_byte(struct intel_vgpu *vgpu)
>  	struct intel_vgpu_i2c_edid *edid = &vgpu->display.i2c_edid;
>  	unsigned char chr = 0;
>  
> -	if (edid->state == I2C_NOT_SPECIFIED || !edid->slave_selected) {
> +	if (edid->state == I2C_NOT_SPECIFIED || !edid->target_selected) {
>  		gvt_vgpu_err("Driver tries to read EDID without proper sequence!\n");
>  		return 0;
>  	}
> @@ -179,7 +179,7 @@ static int gmbus1_mmio_write(struct intel_vgpu *vgpu, unsigned int offset,
>  		void *p_data, unsigned int bytes)
>  {
>  	struct intel_vgpu_i2c_edid *i2c_edid = &vgpu->display.i2c_edid;
> -	u32 slave_addr;
> +	u32 target_addr;
>  	u32 wvalue = *(u32 *)p_data;
>  
>  	if (vgpu_vreg(vgpu, offset) & GMBUS_SW_CLR_INT) {
> @@ -210,21 +210,21 @@ static int gmbus1_mmio_write(struct intel_vgpu *vgpu, unsigned int offset,
>  
>  		i2c_edid->gmbus.total_byte_count =
>  			gmbus1_total_byte_count(wvalue);
> -		slave_addr = gmbus1_slave_addr(wvalue);
> +		target_addr = gmbus1_target_addr(wvalue);
>  
>  		/* vgpu gmbus only support EDID */
> -		if (slave_addr == EDID_ADDR) {
> -			i2c_edid->slave_selected = true;
> -		} else if (slave_addr != 0) {
> +		if (target_addr == EDID_ADDR) {
> +			i2c_edid->target_selected = true;
> +		} else if (target_addr != 0) {
>  			gvt_dbg_dpy(
> -				"vgpu%d: unsupported gmbus slave addr(0x%x)\n"
> +				"vgpu%d: unsupported gmbus target addr(0x%x)\n"
>  				"	gmbus operations will be ignored.\n",
> -					vgpu->id, slave_addr);
> +					vgpu->id, target_addr);
>  		}
>  
>  		if (wvalue & GMBUS_CYCLE_INDEX)
>  			i2c_edid->current_edid_read =
> -				gmbus1_slave_index(wvalue);
> +				gmbus1_target_index(wvalue);
>  
>  		i2c_edid->gmbus.cycle_type = gmbus1_bus_cycle(wvalue);
>  		switch (gmbus1_bus_cycle(wvalue)) {
> @@ -523,7 +523,7 @@ void intel_gvt_i2c_handle_aux_ch_write(struct intel_vgpu *vgpu,
>  			} else if (addr == EDID_ADDR) {
>  				i2c_edid->state = I2C_AUX_CH;
>  				i2c_edid->port = port_idx;
> -				i2c_edid->slave_selected = true;
> +				i2c_edid->target_selected = true;
>  				if (intel_vgpu_has_monitor_on_port(vgpu,
>  					port_idx) &&
>  					intel_vgpu_port_is_dp(vgpu, port_idx))
> @@ -542,7 +542,7 @@ void intel_gvt_i2c_handle_aux_ch_write(struct intel_vgpu *vgpu,
>  			return;
>  		if (drm_WARN_ON(&i915->drm, msg_length != 4))
>  			return;
> -		if (i2c_edid->edid_available && i2c_edid->slave_selected) {
> +		if (i2c_edid->edid_available && i2c_edid->target_selected) {
>  			unsigned char val = edid_get_byte(vgpu);
>  
>  			aux_data_for_write = (val << 16);
> @@ -571,7 +571,7 @@ void intel_vgpu_init_i2c_edid(struct intel_vgpu *vgpu)
>  	edid->state = I2C_NOT_SPECIFIED;
>  
>  	edid->port = -1;
> -	edid->slave_selected = false;
> +	edid->target_selected = false;
>  	edid->edid_available = false;
>  	edid->current_edid_read = 0;
>  
> diff --git a/drivers/gpu/drm/i915/gvt/edid.h b/drivers/gpu/drm/i915/gvt/edid.h
> index dfe0cbc6aad8..c3b5a55aecb3 100644
> --- a/drivers/gpu/drm/i915/gvt/edid.h
> +++ b/drivers/gpu/drm/i915/gvt/edid.h
> @@ -80,7 +80,7 @@ enum gmbus_cycle_type {
>   *      R/W Protect
>   *      Command and Status.
>   *      bit0 is the direction bit: 1 is read; 0 is write.
> - *      bit1 - bit7 is slave 7-bit address.
> + *      bit1 - bit7 is target 7-bit address.
>   *      bit16 - bit24 total byte count (ignore?)
>   *
>   * GMBUS2:
> @@ -130,7 +130,7 @@ struct intel_vgpu_i2c_edid {
>  	enum i2c_state state;
>  
>  	unsigned int port;
> -	bool slave_selected;
> +	bool target_selected;
>  	bool edid_available;
>  	unsigned int current_edid_read;
>  
> diff --git a/drivers/gpu/drm/i915/gvt/opregion.c b/drivers/gpu/drm/i915/gvt/opregion.c
> index d2bed466540a..908f910420c2 100644
> --- a/drivers/gpu/drm/i915/gvt/opregion.c
> +++ b/drivers/gpu/drm/i915/gvt/opregion.c
> @@ -86,7 +86,7 @@ struct efp_child_device_config {
>  	u8 skip2;
>  	u8 dvo_port;
>  	u8 i2c_pin; /* for add-in card */
> -	u8 slave_addr; /* for add-in card */
> +	u8 target_addr; /* for add-in card */
>  	u8 ddc_pin;
>  	u16 edid_ptr;
>  	u8 dvo_config;
> -- 
> 2.34.1
>