linux-kernel - Re: [PATCH 4/4] i3c: drop i3c_priv_xfer and i3c_device_do_priv

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <2025120223152613f1586a@mail.local>
Date: Wed, 3 Dec 2025 00:15:26 +0100
From: Alexandre Belloni <alexandre.belloni@...tlin.com>
To: Frank Li <Frank.Li@....com>
Cc: Guenter Roeck <linux@...ck-us.net>,
	Jeremy Kerr <jk@...econstruct.com.au>,
	Matt Johnston <matt@...econstruct.com.au>,
	Andrew Lunn <andrew+netdev@...n.ch>,
	"David S. Miller" <davem@...emloft.net>,
	Eric Dumazet <edumazet@...gle.com>,
	Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>,
	Mark Brown <broonie@...nel.org>,
	Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	"Rafael J. Wysocki" <rafael@...nel.org>,
	Danilo Krummrich <dakr@...nel.org>, linux-hwmon@...r.kernel.org,
	linux-kernel@...r.kernel.org, netdev@...r.kernel.org,
	linux-i3c@...ts.infradead.org
Subject: Re: [PATCH 4/4] i3c: drop i3c_priv_xfer and
 i3c_device_do_priv_xfers()

Hello Frank,

You'll have to rebase this patch.

On 28/10/2025 10:57:55-0400, Frank Li wrote:
> Drop i3c_priv_xfer and i3c_device_do_priv_xfers() after all driver switch
> to use new API.
> 
> Signed-off-by: Frank Li <Frank.Li@....com>
> ---
> This patch need be applied after all other patches applied.
> ---
>  include/linux/i3c/device.h | 12 +-----------
>  1 file changed, 1 insertion(+), 11 deletions(-)
> 
> diff --git a/include/linux/i3c/device.h b/include/linux/i3c/device.h
> index ae0662d9d77eb3fa0c976de1803e9c2ff9547451..47e6c95d87f9494d48c5b0463544916f26923501 100644
> --- a/include/linux/i3c/device.h
> +++ b/include/linux/i3c/device.h
> @@ -25,7 +25,7 @@
>   * @I3C_ERROR_M2: M2 error
>   *
>   * These are the standard error codes as defined by the I3C specification.
> - * When -EIO is returned by the i3c_device_do_priv_xfers() or
> + * When -EIO is returned by the i3c_device_do_i3c_xfers() or
>   * i3c_device_send_hdr_cmds() one can check the error code in
>   * &struct_i3c_xfer.err or &struct i3c_hdr_cmd.err to get a better idea of
>   * what went wrong.
> @@ -79,9 +79,6 @@ struct i3c_xfer {
>  	enum i3c_error_code err;
>  };
>  
> -/* keep back compatible */
> -#define i3c_priv_xfer i3c_xfer
> -
>  /**
>   * enum i3c_dcr - I3C DCR values
>   * @I3C_DCR_GENERIC_DEVICE: generic I3C device
> @@ -311,13 +308,6 @@ static __always_inline void i3c_i2c_driver_unregister(struct i3c_driver *i3cdrv,
>  int i3c_device_do_xfers(struct i3c_device *dev, struct i3c_xfer *xfers,
>  			int nxfers, enum i3c_xfer_mode mode);
>  
> -static inline int i3c_device_do_priv_xfers(struct i3c_device *dev,
> -					   struct i3c_xfer *xfers,
> -					   int nxfers)
> -{
> -	return i3c_device_do_xfers(dev, xfers, nxfers, I3C_SDR);
> -}
> -
>  int i3c_device_do_setdasa(struct i3c_device *dev);
>  
>  void i3c_device_get_info(const struct i3c_device *dev, struct i3c_device_info *info);
> 
> -- 
> 2.34.1
> 

-- 
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com