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,