From: Chris Simpson <chris@nilk.net>
To: Marco Ziliani <rylos78@gmail.com>,
Daniel Scally <dan.scally@ideasonboard.com>,
Sakari Ailus <sakari.ailus@linux.intel.com>,
Mauro Carvalho Chehab <mchehab@kernel.org>
Cc: linux-media@vger.kernel.org, Tony Garcia <tony@loafecho.com>,
Simon Kruse <simon0x+git@proton.me>,
"D. Manresa" <dmanresa@gmail.com>,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 1/3] media: i2c: ov5693: Program the mode registers on stream start
Date: Thu, 29 Oct 2026 10:56:13 -0600 [thread overview]
Message-ID: <1dd56310-9286-4321-ad1b-e4643aeaf60e@nilk.net> (raw)
In-Reply-To: <20261029-ov5693-fixes-v2-1-2526d7b13030@gmail.com>
Tested on a Surface Pro 9 (Intel Alder Lake-U, i7-1255U; IPU6, front
OV5693) in a 7.3-based kernel, with the v2 series applied to an
ov5693.c that matches the base-commit (d7e7bca71698) apart from the
pad ops' client_info argument, against the same file unpatched, using
raw captures from the ISYS video node:
- With the sensor held powered between streams, OUTPUT_SIZE (read
over I2C during and after each stream) matched the requested size
in 32 of 32 streams requesting 2592x1944 or 1296x972, and every
stream delivered image data. Unpatched, it stayed at 2592x1944 in
all 32 and the 1296x972 streams gave no frames.
- With each stream started from runtime suspend, the path v2
changed, OUTPUT_SIZE matched the requested size in 12 of 12
streams (as it did unpatched), all with complete image data.
The test scripts, the analysis of their output and this summary were
prepared with an AI coding assistant (Claude); I ran the tests on my
Surface Pro 9.
Tested-by: Chris Simpson <chris@nilk.net>
next prev parent reply other threads:[~2026-10-29 16:56 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-29 9:27 [PATCH v2 0/3] media: i2c: ov5693: Fix mode changes, flips and autosuspend Marco Ziliani
2026-10-29 9:27 ` [PATCH v2 1/3] media: i2c: ov5693: Program the mode registers on stream start Marco Ziliani
2026-10-29 16:56 ` Chris Simpson [this message]
2026-10-29 9:27 ` [PATCH v2 2/3] media: i2c: ov5693: Keep the Bayer order when flipping at full resolution Marco Ziliani
2026-10-29 9:27 ` [PATCH v2 3/3] media: i2c: ov5693: Use autosuspend when stopping the stream Marco Ziliani
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1dd56310-9286-4321-ad1b-e4643aeaf60e@nilk.net \
--to=chris@nilk.net \
--cc=dan.scally@ideasonboard.com \
--cc=dmanresa@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@kernel.org \
--cc=rylos78@gmail.com \
--cc=sakari.ailus@linux.intel.com \
--cc=simon0x+git@proton.me \
--cc=tony@loafecho.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®