Re: [PATCH v4 0/3] media: subdev: Add v4l2_subdev_get_frame_desc_passthrough and use it

[Tomi Valkeinen <tomi.valkeinen+renesas@ideasonboard.com>]

Hi,

On 24/03/2025 10:04, Tomi Valkeinen wrote:
> When writing the exact same code the third time for Renesas, I thought
> we probably need a helper.
> 
>  Tomi

It's been a while, but the need for this series hasn't gone anywhere. I
will need to send a rebased version and re-test, but any comments before
I do that?

 Tomi

> Signed-off-by: Tomi Valkeinen <tomi.valkeinen+renesas@ideasonboard.com>
> ---
> Changes in v4:
> - Update the helper name in commit descriptions too
> - Link to v3: https://lore.kernel.org/r/20250324-frame-desc-passthrough-v3-0-993839a1f9b3@ideasonboard.com
> 
> Changes in v3:
> - Rename the helper to v4l2_subdev_get_frame_desc_passthrough()
> - Check for fd->entry[] array overflow
> - Add error prints (with dev_dbg)
> - Link to v2: https://lore.kernel.org/r/20250219-frame-desc-passthrough-v2-0-5135d57ecd6a@ideasonboard.com
> 
> Changes in v2:
> - Fix "uninitialized symbol 'ret'"
> - Reorder local variables
> - Link to v1: https://lore.kernel.org/r/20250218-frame-desc-passthrough-v1-0-a32458f53714@ideasonboard.com
> 
> ---
> Tomi Valkeinen (3):
>       media: subdev: Add v4l2_subdev_get_frame_desc_passthrough helper
>       media: i2c: ds90ub953: Use v4l2_subdev_get_frame_desc_passthrough
>       media: i2c: ds90ub913: Use v4l2_subdev_get_frame_desc_passthrough
> 
>  drivers/media/i2c/ds90ub913.c         |  59 +-----------------
>  drivers/media/i2c/ds90ub953.c         |  61 +-----------------
>  drivers/media/v4l2-core/v4l2-subdev.c | 113 ++++++++++++++++++++++++++++++++++
>  include/media/v4l2-subdev.h           |  22 +++++++
>  4 files changed, 137 insertions(+), 118 deletions(-)
> ---
> base-commit: f2151613e040973c868d28c8b00885dfab69eb75
> change-id: 20250218-frame-desc-passthrough-66805e413974
> 
> Best regards,