From: Antoine Pichot <antoine.pichot@gmail.com>
To: Dave Stevenson <dave.stevenson@raspberrypi.com>
Cc: linux-media@vger.kernel.org, sakari.ailus@linux.intel.com,
mchehab@kernel.org, djrscally@gmail.com,
dan.scally@ideasonboard.com, hansg@kernel.org,
andriy.shevchenko@linux.intel.com, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 5/7] media: i2c: ov7251: Move the strobe registers out of the mode tables
Date: Fri, 30 Oct 2026 23:26:33 +0900 [thread overview]
Message-ID: <20261030142633.2235231-1-antoine.pichot@gmail.com> (raw)
In-Reply-To: <CAPY8ntAzJ=jA-c9LoLW5tQS=bpQZ3SfepcGVuvhq4cRzXvRK=A@mail.gmail.com>
Hi Dave,
On Thu, Oct 29, 2026 at 04:17:29PM +0000, Dave Stevenson wrote:
> Is there a real need to add a new array of registers for these, or can
> they be added to ov7251_global_init_setting? As long as it's after the
> {0x0103, 0x01} reset, then order shouldn't matter.
No need. v3 drops this patch and takes your "Move the common registers
into the global init table" in its place, under your authorship, so
the strobe registers go into ov7251_global_init_setting with the rest,
and the strobe control patch only names the three it writes. Details
on the v2 1/9 thread, where you posted the branch.
Thanks,
Antoine
next prev parent reply other threads:[~2026-10-30 14:26 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-26 11:10 [PATCH 0/7] media: i2c: ov7251: fixes and infrared strobe for Surface Book 2 Antoine Pichot
2026-10-26 11:10 ` [PATCH 1/7] media: i2c: ov7251: Fix link frequency index Antoine Pichot
2026-10-27 10:26 ` D. Manresa
2026-10-27 13:26 ` Antoine Pichot
2026-10-27 18:11 ` D. Manresa
2026-10-27 16:34 ` Dan Scally
2026-10-27 16:50 ` Dave Stevenson
2026-10-27 18:15 ` D. Manresa
2026-10-27 16:05 ` Dan Scally
2026-10-26 11:10 ` [PATCH 2/7] media: i2c: ov7251: Reset vblank when the frame interval changes Antoine Pichot
2026-10-26 11:10 ` [PATCH 3/7] media: i2c: ov7251: Add control event support Antoine Pichot
2026-10-26 11:34 ` Sakari Ailus
2026-10-26 13:03 ` Antoine Pichot
2026-10-26 11:10 ` [PATCH 4/7] media: i2c: ov7251: Add orientation and rotation controls Antoine Pichot
2026-10-26 11:10 ` [PATCH 5/7] media: i2c: ov7251: Move the strobe registers out of the mode tables Antoine Pichot
2026-10-29 16:17 ` Dave Stevenson
2026-10-30 14:26 ` Antoine Pichot [this message]
2026-10-26 11:10 ` [PATCH 6/7] media: i2c: ov7251: Add strobe output enable control Antoine Pichot
2026-10-26 15:14 ` Andy Shevchenko
2026-10-27 2:44 ` Antoine Pichot
2026-10-26 11:10 ` [PATCH 7/7] media: ipu-bridge: Add a 90 degree rotation quirk for Surface Book 2 Antoine Pichot
2026-10-27 10:26 ` [PATCH 0/7] media: i2c: ov7251: fixes and infrared strobe " D. Manresa
2026-10-27 13:26 ` Antoine Pichot
2026-10-27 18:45 ` D. Manresa
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=20261030142633.2235231-1-antoine.pichot@gmail.com \
--to=antoine.pichot@gmail.com \
--cc=andriy.shevchenko@linux.intel.com \
--cc=dan.scally@ideasonboard.com \
--cc=dave.stevenson@raspberrypi.com \
--cc=djrscally@gmail.com \
--cc=hansg@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@kernel.org \
--cc=sakari.ailus@linux.intel.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®