From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [213.167.242.64]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4EA8346AA60; Tue, 21 Jul 2026 15:32:15 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=213.167.242.64 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784647937; cv=none; b=HvfpWtzEWk0thwfF+ebgNQP7KAN8peHPoHNLrzHdC4hznox3HmngUYZlu9p4HfTekArnsKlbfMytNxBKekEWrgihwy4NljU1PdymOTd/xl1VdLAG238kcUgdMP/4JrZ5i22hO0SwH+8DN6QYo8uIUbQwJ+p+uqrqA3/8sEtEeEI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784647937; c=relaxed/simple; bh=t6Ai+E5koIEVC/lAWoZNJddRID1TUkuPv9C1ZyMtBV0=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=GsuZHDIBRjy5pVjXB16fbEX3vTxTs1fKH+3TTdHRuxwFA8Tbd1gm9CNvf5IRBkhl9GRYzIQALXCnPdNYuXsjF2pUPj9NIW9tBkGnoR+Q7UqNPEOOKfRFnHvyW+nJ3nPAE3S4A4LouzjmWUtSANMKNYYq4FEqb5oniYK6wOdCBO0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=ideasonboard.com; spf=pass smtp.mailfrom=ideasonboard.com; dkim=pass (1024-bit key) header.d=ideasonboard.com header.i=@ideasonboard.com header.b=vFjW2CFH; arc=none smtp.client-ip=213.167.242.64 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=ideasonboard.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=ideasonboard.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=ideasonboard.com header.i=@ideasonboard.com header.b="vFjW2CFH" Received: from killaraus.ideasonboard.com (2001-14ba-70f3-e800--a06.rev.dnainternet.fi [IPv6:2001:14ba:70f3:e800::a06]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id A4120C1; Tue, 21 Jul 2026 17:31:14 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1784647874; bh=t6Ai+E5koIEVC/lAWoZNJddRID1TUkuPv9C1ZyMtBV0=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=vFjW2CFHDvg2TteGV1oKIn8/7nnQIacd6MNodk2KXvRLWHwURkUsvtrRxo9xZieQ6 lh6Mg3jqEkz9pl0aUkEk+u6aIGPUfb6WGXZNxxQxnFZkpzyvDJqyyQuqcf07Byn9BL N0gLF3mHvVDFl0mO1ujLo5OSgDmDqTJjVpA9bo/0= Date: Tue, 21 Jul 2026 18:32:12 +0300 From: Laurent Pinchart To: Richard Leitner Cc: Alexander Stein , Mauro Carvalho Chehab , Sakari Ailus , Dave Stevenson , Hans de Goede , linux-media@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 2/2] media: ov9282: enable single-read I2C transactions Message-ID: <20260721153212.GP50424@killaraus.ideasonboard.com> References: <20260721-cci-single-v1-0-8485171e1393@linux.dev> <20260721-cci-single-v1-2-8485171e1393@linux.dev> <2422477.ElGaqSPkdT@steina-w> Precedence: bulk X-Mailing-List: linux-media@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: Hi Richard, On Tue, Jul 21, 2026 at 05:15:06PM +0200, Richard Leitner wrote: > On Tue, Jul 21, 2026 at 03:38:26PM +0200, Alexander Stein wrote: > > Am Dienstag, 21. Juli 2026, 15:25:29 CEST schrieb Richard Leitner: > > > Some OV9282 camera modules, such as those from Vision Components, fail > > > multi-byte register reads with the default CCI I2C regmap access pattern > > > and only work when single-register read transactions are used. This > > > results in the driver being unable to probe due to a chip id mismatch. > > > > > > As the driver has no reliable way to identify affected modules enable > > > use_single_read for OV9282 unconditionally when creating the CCI regmap. > > > > What about a property in the DT instead? In DT it should be clear if a > > Vision Components camera is used. > > I've talked to Laurent about this topic (I guess a year ago or so...) > and he pointed me to his downstream work regarding the vc-mipi driver > (which I am currently also using): > > https://gitlab.com/ideasonboard/nxp/linux/-/commits/v7.0/sensors/vcmipi > > There is a new driver (and therefore DT compatible) for the controller > (without the sensor) on the Vision Components board implemented. This > separates basically the "regulator" part of the VC microprocessor from > the "forward I2C traffic to the sensor" part. AFAICT... > > Talking about that separation I vaguely remember he argued against a > new device-tree property or compatible for the vcmipi variants of the > ov9282. Therefore I went for the "easy" approach of just enabling single > register reads. > > Nonetheless if I understood this wrong or "things" changed, please feel > free to correct me. > > Just to be clear: I'm also perfectly fine with introducing a new > "vcmipi,ov9282" compatible or something like a "quirk-vision-components" > property if that's preferred. My main concern is that I don't want to see this being handled manually in drivers. Whatever solution we decide to implement (I proposed two in a separate e-mail in this thread) should not require modifying image sensor drivers to implement logic specific to the Vision Components module. I'm fine relying on drivers using specific helpers (for instance requiring drivers to be ported to regmap or v4l2-cci in order to support the quirk). > > > Tested on an i.M8MP system with OV9282 modules from Vision Components. > > > > > > Signed-off-by: Richard Leitner > > > --- > > > drivers/media/i2c/ov9282.c | 7 ++++++- > > > 1 file changed, 6 insertions(+), 1 deletion(-) > > > > > > diff --git a/drivers/media/i2c/ov9282.c b/drivers/media/i2c/ov9282.c > > > index 5b6f897a74fcd..8267a0a9c6f2d 100644 > > > --- a/drivers/media/i2c/ov9282.c > > > +++ b/drivers/media/i2c/ov9282.c > > > @@ -1283,6 +1283,11 @@ static int ov9282_probe(struct i2c_client *client) > > > struct ov9282 *ov9282; > > > int ret; > > > > > > + const struct cci_regmap_config config = { > > > + .reg_addr_bits = 16, > > > + .use_single_read = true, > > > + }; > > > + > > > ov9282 = devm_kzalloc(&client->dev, sizeof(*ov9282), GFP_KERNEL); > > > if (!ov9282) > > > return -ENOMEM; > > > @@ -1301,7 +1306,7 @@ static int ov9282_probe(struct i2c_client *client) > > > return ret; > > > } > > > > > > - ov9282->regmap = devm_cci_regmap_init_i2c(client, 16); > > > + ov9282->regmap = devm_cci_regmap_init_i2c_cfg(client, &config); > > > if (IS_ERR(ov9282->regmap)) > > > return dev_err_probe(ov9282->dev, PTR_ERR(ov9282->regmap), > > > "Failed to init CCI\n"); -- Regards, Laurent Pinchart