Skip to content

PR consisting of frame interval and compose resolution fixes#72

Merged
vgarodia merged 2 commits intoqualcomm-linux:video.qclinux.mainfrom
venvenk:video.qclinux.main
Mar 6, 2026
Merged

PR consisting of frame interval and compose resolution fixes#72
vgarodia merged 2 commits intoqualcomm-linux:video.qclinux.mainfrom
venvenk:video.qclinux.main

Conversation

@venvenk
Copy link
Copy Markdown
Contributor

@venvenk venvenk commented Mar 5, 2026

This tag brings a following updates for video driver to fix GST pipeline issues:

fix FRMIVAL stepwise denominators
Initialize compose selection values for decoder

venvenk added 2 commits March 6, 2026 03:27
The FRAME_RATE.max value in the platform data is stored in Q16 format.
The frame interval reporting logic was using this scaled value directly,
resulting in incorrect denominators being exposed to userspace.

Convert the stored frame rate to its integer form before filling the
stepwise min/step denominators. This restores correct frame interval
reporting.

Change-Id: I8945c24159d2be81c61743aa1f0a99c0a1d3a832
Signed-off-by: Venkata Siva Pavan KumarVenkatapatigari <venvenk@qti.qualcomm.com>
Initialize compose selection values to match crop values on hardware
without scaling support (e.g., IRIS2/IRIS3) to prevent GStreamer decoder
pipeline failures.

GStreamer was receiving zero-valued compose rectangles, which violated
V4L2 selection semantics and caused pipeline setup to fail. For hardware
without scaling capability, compose and crop must be identical:

- V4L2_SEL_TGT_COMPOSE_DEFAULT should equal V4L2_SEL_TGT_CROP
- V4L2_SEL_TGT_COMPOSE defaults to V4L2_SEL_TGT_COMPOSE_DEFAULT

This patch sets compose = crop:
1. During instance initialization (default resolution)
2. After stream parsing (firmware‑reported visible resolution)

IRIS variants with scaling support are unaffected.

Change-Id: Ibcc62405b583e8ca5ae6ebf68eb10ecea34c1b64
Signed-off-by: Venkata Siva Pavan KumarVenkatapatigari <venvenk@qti.qualcomm.com>
@vgarodia vgarodia merged commit 9bd1da4 into qualcomm-linux:video.qclinux.main Mar 6, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants