From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.12]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0595B72631; Fri, 24 Oct 2025 06:13:34 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=198.175.65.12 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1761286417; cv=none; b=Pft5Wj0UYabPibG1DC33poudsskWIygwvBTbqbhWbDR+lcaeL9jEoWkg+8kiK0WA2Sow8YFpdWvO5uKBZdcoy5mjNvBnputyXqdQXbQ+LaPDlMOQjq6wchwRpTDNlj2VEVJUm7GQHsxnDd9us3fEXi4KMsSRIOWfh3uM5RSz168= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1761286417; c=relaxed/simple; bh=pPg4tGMDdSKgzOhv/19H1ClKHTpBCPpph/N7jY8PuY0=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=rNsFUrYE9H0+YqO7n5m4dCZJc334cSqyl4G8PzignbZPbovzFkJPGX9P/Td/TJtCAO1EIsuYaPViKRLxr/MWzPRIaBYoSvyaLQFxDpddy4nRYiphE1Ibef8qAQ8acsxW5WekqVahX7F/byATdC+l8jy3NhZJ7Kfv/R/HFkTaGQY= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com; spf=pass smtp.mailfrom=intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=TutqQKVW; arc=none smtp.client-ip=198.175.65.12 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=intel.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="TutqQKVW" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1761286416; x=1792822416; h=date:from:to:cc:subject:message-id:references: mime-version:content-transfer-encoding:in-reply-to; bh=pPg4tGMDdSKgzOhv/19H1ClKHTpBCPpph/N7jY8PuY0=; b=TutqQKVWPcmQDxCqpClK5Je+gO2goj86pU0iUIIfxk+xzLuVfyZqkxyg ywtdgfG/spmc4UudxXWFPQGTZ91dgGpIR8nf+NIX5IH4uG5ExGAuR4j4K IxSPQ8RPPdDlSG/bLNjEqSncn5xg4BVOdGVXrYrrDa5nj6F2Y53lOUgHO v/jasdebyZCVofgLdCTuMVhjZse8VYIuDMTRxAIKpPX3BcnP0gxI+i5Pt Xw0WquOZqMNq5UwLFEWLSb5HtB7y9CECfs94bQ3+Gp/u53Q/TQZJ44gYI ax52c3az27sUN37GNnIhxEQJeV1mSsw+I+OCHAMsNmpPN2aKKaIev6MUa w==; X-CSE-ConnectionGUID: 2lSKj0jaTcGLfDoBeealyQ== X-CSE-MsgGUID: IvKkTY/GR+yiVQUX7z32og== X-IronPort-AV: E=McAfee;i="6800,10657,11586"; a="74906837" X-IronPort-AV: E=Sophos;i="6.19,251,1754982000"; d="scan'208";a="74906837" Received: from fmviesa005.fm.intel.com ([10.60.135.145]) by orvoesa104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 23 Oct 2025 23:13:35 -0700 X-CSE-ConnectionGUID: EDZuUWOTS7iZKdMy7RFhHw== X-CSE-MsgGUID: zgPfV1+5TsCKMukpOqL70Q== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.19,251,1754982000"; d="scan'208";a="188740617" Received: from opintica-mobl1 (HELO ashevche-desk.local) ([10.245.245.60]) by fmviesa005-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 23 Oct 2025 23:13:31 -0700 Received: from andy by ashevche-desk.local with local (Exim 4.98.2) (envelope-from ) id 1vCB3H-000000024n3-3VVz; Fri, 24 Oct 2025 09:13:27 +0300 Date: Fri, 24 Oct 2025 09:13:27 +0300 From: Andy Shevchenko To: Frank Li Cc: Alexandre Belloni , Miquel Raynal , Jonathan Cameron , David Lechner , Nuno =?iso-8859-1?Q?S=E1?= , Andy Shevchenko , Rob Herring , Krzysztof Kozlowski , Conor Dooley , linux-i3c@lists.infradead.org, linux-kernel@vger.kernel.org, imx@lists.linux.dev, linux-iio@vger.kernel.org, joshua.yeong@starfivetech.com, devicetree@vger.kernel.org Subject: Re: [PATCH v6 1/5] i3c: Add HDR API support Message-ID: References: <20251014-i3c_ddr-v6-0-3afe49773107@nxp.com> <20251014-i3c_ddr-v6-1-3afe49773107@nxp.com> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Organization: Intel Finland Oy - BIC 0357606-4 - c/o Alberga Business Park, 6 krs, Bertel Jungin Aukio 5, 02600 Espoo On Thu, Oct 23, 2025 at 07:53:15PM -0400, Frank Li wrote: > On Thu, Oct 23, 2025 at 09:22:55PM +0300, Andy Shevchenko wrote: > > On Thu, Oct 23, 2025 at 11:18:37AM -0400, Frank Li wrote: > > > On Thu, Oct 23, 2025 at 11:23:39AM +0300, Andy Shevchenko wrote: > > > > On Tue, Oct 14, 2025 at 12:40:00PM -0400, Frank Li wrote: ... > > > > > +/* keep back compatible */ > > > > > +#define i3c_priv_xfer i3c_xfer > > > > > > > > How many of the current users do this? Can't we just rename treewide? > > > > > > git grep -r priv_xfer drivers/ > > > > `git grep -lw ...` is a better approach :-) > > > > > drivers/base/regmap/regmap-i3c.c: struct i3c_priv_xfer xfers[] = { > > > drivers/base/regmap/regmap-i3c.c: return i3c_device_do_priv_xfers(i3c, xfers, 1); > > > drivers/base/regmap/regmap-i3c.c: struct i3c_priv_xfer xfers[2]; > > > drivers/base/regmap/regmap-i3c.c: return i3c_device_do_priv_xfers(i3c, xfers, 2); > > > drivers/hwmon/lm75.c: struct i3c_priv_xfer xfers[] = { > > > drivers/hwmon/lm75.c: ret = i3c_device_do_priv_xfers(i3cdev, xfers, 2); > > > drivers/hwmon/lm75.c: struct i3c_priv_xfer xfers[] = { > > > drivers/hwmon/lm75.c: return i3c_device_do_priv_xfers(i3cdev, xfers, 1); > > > drivers/i3c/device.c:int i3c_device_do_xfers(struct i3c_device *dev, struct i3c_priv_xfer *xfers, > > > drivers/i3c/master.c: if (!ops->priv_xfers && !ops->i3c_xfers) > > > drivers/i3c/master.c: if (!master->ops->priv_xfers) > > > drivers/i3c/master.c: return master->ops->priv_xfers(dev, xfers, nxfers); > > > drivers/i3c/master/dw-i3c-master.c:static int dw_i3c_master_priv_xfers(struct i3c_dev_desc *dev, > > > drivers/i3c/master/dw-i3c-master.c: struct i3c_priv_xfer *i3c_xfers, > > > drivers/i3c/master/dw-i3c-master.c: .priv_xfers = dw_i3c_master_priv_xfers, > > > drivers/i3c/master/i3c-master-cdns.c:static int cdns_i3c_master_priv_xfers(struct i3c_dev_desc *dev, > > > drivers/i3c/master/i3c-master-cdns.c: struct i3c_priv_xfer *xfers, > > > drivers/i3c/master/i3c-master-cdns.c: .priv_xfers = cdns_i3c_master_priv_xfers, > > > drivers/i3c/master/mipi-i3c-hci/core.c:static int i3c_hci_priv_xfers(struct i3c_dev_desc *dev, > > > drivers/i3c/master/mipi-i3c-hci/core.c: struct i3c_priv_xfer *i3c_xfers, > > > drivers/i3c/master/mipi-i3c-hci/core.c: .priv_xfers = i3c_hci_priv_xfers, > > > drivers/i3c/master/renesas-i3c.c:static int renesas_i3c_priv_xfers(struct i3c_dev_desc *dev, struct i3c_priv_xfer *i3c_xfers, > > > drivers/i3c/master/renesas-i3c.c: .priv_xfers = renesas_i3c_priv_xfers, > > > drivers/i3c/master/svc-i3c-master.c: struct i3c_priv_xfer *xfer; > > > drivers/i3c/master/svc-i3c-master.c: * at svc_i3c_master_priv_xfers(). > > > drivers/i3c/master/svc-i3c-master.c:static int svc_i3c_master_i3c_xfers(struct i3c_dev_desc *dev, struct i3c_priv_xfer *xfers, > > > drivers/net/mctp/mctp-i3c.c: struct i3c_priv_xfer xfer = { .rnw = 1, .len = mi->mrl }; > > > drivers/net/mctp/mctp-i3c.c: rc = i3c_device_do_priv_xfers(mi->i3c, &xfer, 1); > > > drivers/net/mctp/mctp-i3c.c: struct i3c_priv_xfer xfer = { .rnw = false }; > > > drivers/net/mctp/mctp-i3c.c: rc = i3c_device_do_priv_xfers(mi->i3c, &xfer, 1); > > > > > > After this patch merged, I can clean up it at difference subsytem. After > > > all cleanup done, we can safely remove this define. > > > > I counted 9. I think it's not a big deal to convert all of them at once without > > leaving an intermediate state. But this is a call for the I³C subsystem maintaiiner. > > There also are other cleanup works. The key point is that everyone agree my > HDR solution. Cleanup these is not big deal. I am not sure how to avoid > build broken at difference subsystem. > > After this patch merge, cleanup will be easier and safer. Then leave that renaming to the cleanup series. No need to use a define, just use the old function name. -- With Best Regards, Andy Shevchenko