Re: [PATCH v3] media: i2c: ov5640: use scoped fwnode_handle endpoint cleanup
From: Sakari Ailus
Date: Wed Jul 08 2026 - 09:28:06 EST
Hi Biren,
On Wed, Jul 08, 2026 at 06:19:55PM +0530, Biren Pandya wrote:
> Utilize the __free(fwnode_handle) scoped guard macro from
> <linux/cleanup.h> to automate the lifecycle management of the endpoint
> fwnode in ov5640_probe().
>
> This eliminates the need for manual fwnode_handle_put() calls.
> Additionally, drop the redundant !endpoint check before
> v4l2_fwnode_endpoint_parse(), as the parse function already handles
> NULL endpoints safely.
>
> Signed-off-by: Biren Pandya <birenpandya@xxxxxxxxx>
> ---
> v3: adopted __free scoped guard (Laurent); explicitly dropped redundant endpoint check (Sakari).
Please don't include git tags in changelog; git am will pick the lines as
tags to the changelog.
The threading appears to be broken in the set; please resend v3 (three
patches) as you sent v2.
--
Kind regards,
Sakari Ailus