[v5] media: subdev: Add v4l2_subdev_get_frame_desc_passthrough and use it

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

Posted by Tomi Valkeinen 3 weeks, 6 days ago

When writing the exact same code the third time for Renesas, I thought
we probably need a helper.

 Tomi

Signed-off-by: Tomi Valkeinen <tomi.valkeinen+renesas@ideasonboard.com>
---
Changes in v5:
- Minor code cleanups which do not change functionality
- Rewrote the kernel doc
- Fix mentions of 'v4l2_get_frame_desc' op, as it's 'get_frame_desc'
- Rebase on v7.0-rc2
- Link to v4: https://lore.kernel.org/r/20250324-frame-desc-passthrough-v4-0-dbe2412297cc@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 | 111 ++++++++++++++++++++++++++++++++++
 include/media/v4l2-subdev.h           |  28 +++++++++
 4 files changed, 141 insertions(+), 118 deletions(-)
---
base-commit: 11439c4635edd669ae435eec308f4ab8a0804808
change-id: 20250218-frame-desc-passthrough-66805e413974

Best regards,
-- 
Tomi Valkeinen <tomi.valkeinen+renesas@ideasonboard.com>

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

Posted by Sakari Ailus 3 weeks, 6 days ago

Moi,

On Wed, Mar 11, 2026 at 02:17:18PM +0200, Tomi Valkeinen wrote:
> When writing the exact same code the third time for Renesas, I thought
> we probably need a helper.
> 
>  Tomi
> 
> Signed-off-by: Tomi Valkeinen <tomi.valkeinen+renesas@ideasonboard.com>

Can you send follow-ups to v4? V4 is already in my PR to Hans.

-- 
Terveisin,

Sakari Ailus

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

Posted by Tomi Valkeinen 3 weeks, 5 days ago

Hi,

On 11/03/2026 20:11, Sakari Ailus wrote:
> Moi,
> 
> On Wed, Mar 11, 2026 at 02:17:18PM +0200, Tomi Valkeinen wrote:
>> When writing the exact same code the third time for Renesas, I thought
>> we probably need a helper.
>>
>>  Tomi
>>
>> Signed-off-by: Tomi Valkeinen <tomi.valkeinen+renesas@ideasonboard.com>
> 
> Can you send follow-ups to v4? V4 is already in my PR to Hans.
> 

I have sent
https://lore.kernel.org/all/20260312-frame-desc-passthrough-impro-v1-0-30f64d637a3a%40ideasonboard.com/

 Tomi