[PATCH v3 2/2] i3c: master: renesas: Use __free(kfree) for xfer cleanup in renesas_i3c_send_ccc_cmd()
Tommaso Merciai
tommaso.merciai.xr at bp.renesas.com
Tue Apr 7 01:11:39 PDT 2026
Hi Felix,
Thanks for your patch.
On Mon, Apr 06, 2026 at 08:43:17PM +0800, Felix Gu wrote:
> Use __free(kfree) for automatic cleanup, matching the pattern already
> used in other functions in this driver.
>
Tested-by: Tommaso Merciai <tommaso.merciai.xr at bp.renesas.com>
Reviewed-by: Tommaso Merciai <tommaso.merciai.xr at bp.renesas.com>
Kind Regards,
Tommaso
> Signed-off-by: Felix Gu <ustc.gu at gmail.com>
> ---
> drivers/i3c/master/renesas-i3c.c | 5 +----
> 1 file changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/drivers/i3c/master/renesas-i3c.c b/drivers/i3c/master/renesas-i3c.c
> index a8a9e89a9710..f39c449922ca 100644
> --- a/drivers/i3c/master/renesas-i3c.c
> +++ b/drivers/i3c/master/renesas-i3c.c
> @@ -748,7 +748,6 @@ static int renesas_i3c_send_ccc_cmd(struct i3c_master_controller *m,
> struct i3c_ccc_cmd *ccc)
> {
> struct renesas_i3c *i3c = to_renesas_i3c(m);
> - struct renesas_i3c_xfer *xfer;
> struct renesas_i3c_cmd *cmd;
> int ret, pos = 0;
>
> @@ -758,7 +757,7 @@ static int renesas_i3c_send_ccc_cmd(struct i3c_master_controller *m,
> return pos;
> }
>
> - xfer = renesas_i3c_alloc_xfer(i3c, 1);
> + struct renesas_i3c_xfer *xfer __free(kfree) = renesas_i3c_alloc_xfer(i3c, 1);
> if (!xfer)
> return -ENOMEM;
>
> @@ -807,8 +806,6 @@ static int renesas_i3c_send_ccc_cmd(struct i3c_master_controller *m,
> if (ret)
> ccc->err = I3C_ERROR_M2;
>
> - kfree(xfer);
> -
> return ret;
> }
>
>
> --
> 2.43.0
>
More information about the linux-i3c
mailing list