From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 5726AC624DB for ; Thu, 3 Sep 2026 20:54:39 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:MIME-Version:Reply-To:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:Message-Id:Date: References:In-Reply-To:Cc:To:Subject:From:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=8sBve3oHvl2z+rF4GSUZZRGumPzPdiGfKcrRQNlVmQk=; b=U0WwctyTq38mqJ RYtZWH0O+VXJfY66hHhHTO24h46RXHBM4Qd0PmN4h9zXDgnZp91XSljn95F317BcjIYY5CjOaHWM7 uFmfyrlgXX+9jD5NGCZbGgRFh7+zIStH+gk+KYhYvy+KgO/SUERCXYtdTHIJ2hKz8EnqX+Dyis3+C QF124nfY1OXkBsAYWDoA0Zxlj9W2VZDPnGlDacdxGLNS+7x8/jHCkziZc6wOkyzDioRFVnXp9fazu QooAk6yGwF5xi+Lop7KSXcEKR3+H2bIGWAzNR5mxQat1yvQj+RyFRW40sUcfs4xjmmTzpm5v02xpH ZOAi9tYfp9GQ9MOtxv+g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1x2ESE-00000000au5-2jgE; Thu, 03 Sep 2026 20:54:38 +0000 Received: from sea.source.kernel.org ([2600:3c0a:e001:78e:0:1991:8:25]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1x2ESD-00000000atx-3BWS for linux-phy@lists.infradead.org; Thu, 03 Sep 2026 20:54:37 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by sea.source.kernel.org (Postfix) with ESMTP id 848E341AAC; Thu, 3 Sep 2026 20:54:37 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id DEA4E1F000E9; Thu, 3 Sep 2026 20:54:36 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1788468877; bh=aE9v9B5eRnSI6xpicfymScibzPae9lIzGDw+sAemqpg=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date; b=TryPiCaPrsWHDjSCEm5lBmFNFCzdD1HeunMKNdbljIo+xe7JRwuDnLBHb40wYpulO zb9bHJQuHmt5LLv9jBApe4hRTMOaAvyLNrmcEJKc8eKDmXhU1QrSAAZQcfYDZoc6Ug WvT2IYNkSYS4NGBl8/Xemia+cFvda+zgXj2MzLbF2o5B7GMyHZUF351rVxDRRkJ+lV 1buVae6tXyMoSm/7RxQYTZaIduiCoe4WYGxqvJ7GGnXGwO+dlx6ejUT4zw3ThtwBct BJvTJ24QgIqp7hKYBYnKVbtsz0lw3h4Zt3elI7FqDW8wos/vRqKzSeDNkbiySyZyfL xzOXRLGEnB+iQ== From: sashiko-bot@kernel.org Subject: Re: [PATCH v17 2/2] phy: qcom-mipi-csi2: Add a CSI2 MIPI DPHY driver To: =?utf-8?b?QnJ5YW4gTydEb25vZ2h1ZQ==?= Cc: conor+dt@kernel.org, neil.armstrong@linaro.org, olteanv@gmail.com, robh@kernel.org, vkoul@kernel.org, linux-phy@lists.infradead.org, media-ci@linuxtv.org, devicetree@vger.kernel.org In-Reply-To: <20260903-x1e-csi2-phy-v17-2-26606fa9a039@linaro.org> References: <20260903-x1e-csi2-phy-v17-0-26606fa9a039@linaro.org> <20260903-x1e-csi2-phy-v17-2-26606fa9a039@linaro.org> Date: Thu, 03 Sep 2026 20:54:36 +0000 Message-Id: <20260903205436.DEA4E1F000E9@smtp.kernel.org> X-BeenThere: linux-phy@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux Phy Mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: sashiko-reviews@lists.linux.dev MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-phy" Errors-To: linux-phy-bounces+linux-phy=archiver.kernel.org@lists.infradead.org VGhhbmsgeW91IGZvciB5b3VyIGNvbnRyaWJ1dGlvbiEgU2FzaGlrbyBBSSByZXZpZXcgZm91bmQg MSBwb3RlbnRpYWwgaXNzdWUocykgdG8gY29uc2lkZXI6Ci0gW01lZGl1bV0gVGhlIE1JUEkgRC1Q SFkgY29uZmlndXJhdGlvbiBpZ25vcmVzIHRoZSByZXF1ZXN0ZWQgbnVtYmVyIG9mIGFjdGl2ZSBs YW5lcywgZW5hYmxpbmcgYWxsIHBoeXNpY2FsbHkgbWFwcGVkIGxhbmVzIGluc3RlYWQuCi0tCgpj b21taXQgODAxODQxOTRkNDM2YjRlYTY0YzllYmM3OTkwNzA3MGI1OWM5ZGU1MwpBdXRob3I6IEJy eWFuIE8nRG9ub2dodWUgPGJyeWFuLm9kb25vZ2h1ZUBsaW5hcm8ub3JnPgoKcGh5OiBxY29tLW1p cGktY3NpMjogQWRkIGEgQ1NJMiBNSVBJIERQSFkgZHJpdmVyCgpBZGRzIGEgbmV3IE1JUEkgQ1NJ MiBkcml2ZXIgaW4gRFBIWSBtb2RlIGJ5IGltcG9ydGluZyBleGlzdGluZyBDQU1TUyBDU0kKUEhZ IGluaXQgc2VxdWVuY2VzLiBTdXBwb3J0cyB0aGUgcWNvbSx4MWU4MDEwMC1jc2kyLXBoeSBkZXZp Y2UgYW5kIGluY2x1ZGVzCmRhdGEvY2xvY2sgbGFuZSBwb3NpdGlvbmluZyBhbmQgcG9sYXJpdHkg c2VsZWN0aW9uLgoKPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9waHkvcXVhbGNvbW0vcGh5LXFjb20t bWlwaS1jc2kyLWNvcmUuYyBiL2RyaXZlcnMvcGh5L3F1YWxjb21tL3BoeS1xY29tLW1pcGktY3Np Mi1jb3JlLmMKPiAtLS0gL2Rldi9udWxsCj4gKysrIGIvZHJpdmVycy9waHkvcXVhbGNvbW0vcGh5 LXFjb20tbWlwaS1jc2kyLWNvcmUuYwpbIC4uLiBdCj4gK3N0YXRpYyBpbnQgcGh5X3Fjb21fbWlw aV9jc2kyX2NvbmZpZ3VyZShzdHJ1Y3QgcGh5ICpwaHksCj4gKwkJCQkJdW5pb24gcGh5X2NvbmZp Z3VyZV9vcHRzICpvcHRzKQo+ICt7Cj4gKwlzdHJ1Y3QgbWlwaV9jc2kycGh5X2RldmljZSAqY3Np MnBoeSA9IHBoeV9nZXRfZHJ2ZGF0YShwaHkpOwo+ICsJc3RydWN0IHBoeV9jb25maWd1cmVfb3B0 c19taXBpX2RwaHkgKmRwaHlfY2ZnID0gJm9wdHMtPm1pcGlfZHBoeTsKPiArCXN0cnVjdCBtaXBp X2NzaTJwaHlfc3RyZWFtX2NmZyAqc3RyZWFtX2NmZyA9ICZjc2kycGh5LT5zdHJlYW1fY2ZnOwo+ ICsJaW50IHJldDsKPiArCj4gKwlyZXQgPSBwaHlfbWlwaV9kcGh5X2NvbmZpZ192YWxpZGF0ZShk cGh5X2NmZyk7Cj4gKwlpZiAocmV0KQo+ICsJCXJldHVybiByZXQ7Cj4gKwo+ICsJaWYgKGRwaHlf Y2ZnLT5sYW5lcyA8IDEgfHwgZHBoeV9jZmctPmxhbmVzID4gQ1NJMl9NQVhfREFUQV9MQU5FUykK PiArCQlyZXR1cm4gLUVJTlZBTDsKPiArCj4gKwlzdHJlYW1fY2ZnLT5saW5rX2ZyZXEgPSBkcGh5 X2NmZy0+aHNfY2xrX3JhdGU7CgpbU2V2ZXJpdHk6IE1lZGl1bV0KU2hvdWxkIHRoZSByZXF1ZXN0 ZWQgYWN0aXZlIGxhbmUgY291bnQgZnJvbSBvcHRzLT5taXBpX2RwaHkubGFuZXMgYmUgc3RvcmVk CmluIHN0cmVhbV9jZmcgaGVyZT8gQ3VycmVudGx5LCBkcGh5X2NmZy0+bGFuZXMgaXMgdmFsaWRh dGVkIGJ1dCB0aGUgdmFsdWUKZG9lcyBub3QgYXBwZWFyIHRvIGJlIHNhdmVkIGZvciBsYXRlciB1 c2UgYnkgdGhlIFBIWSBlbmFibGUgcm91dGluZXMuCgo+ICsKPiArCXJldHVybiAwOwo+ICt9Clsg Li4uIF0KPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9waHkvcXVhbGNvbW0vcGh5LXFjb20tbWlwaS1j c2kyLTNwaC1kcGh5LmMgYi9kcml2ZXJzL3BoeS9xdWFsY29tbS9waHktcWNvbS1taXBpLWNzaTIt M3BoLWRwaHkuYwo+IC0tLSAvZGV2L251bGwKPiArKysgYi9kcml2ZXJzL3BoeS9xdWFsY29tbS9w aHktcWNvbS1taXBpLWNzaTItM3BoLWRwaHkuYwpbIC4uLiBdCj4gK3N0YXRpYyBpbnQgcGh5X3Fj b21fbWlwaV9jc2kyX2xhbmVzX2VuYWJsZShzdHJ1Y3QgbWlwaV9jc2kycGh5X2RldmljZSAqY3Np MnBoeSwKPiArCQkJCQkgICBzdHJ1Y3QgbWlwaV9jc2kycGh5X3N0cmVhbV9jZmcgKmNmZykKPiAr ewo+ICsJY29uc3Qgc3RydWN0IG1pcGlfY3NpMnBoeV9kZXZpY2VfcmVncyAqcmVncyA9IGNzaTJw aHlfZGV2X3RvX3JlZ3MoY3NpMnBoeSk7Cj4gKwlzdHJ1Y3QgbWlwaV9jc2kycGh5X2xhbmVzX2Nm ZyAqbGFuZV9jZmcgPSAmY2ZnLT5sYW5lX2NmZzsKPiArCXU4IHNldHRsZV9jbnQ7Cj4gKwl1OCB2 YWw7Cj4gKwlpbnQgaTsKWyAuLi4gXQo+ICsJLyoKPiArCSAqIENTSV9DT01NT05fQ1RSTDUgaXMg YSBwaHlzaWNhbCBsYW5lIHBvd2VyLXVwIGJpdG1hcDoKPiArCSAqIC0gQml0cyBbMCwyLDQsNl0g 4oaSIEQtUEhZIGRhdGEgbGFuZXMoTE4wLCBMTjIsIExONCwgTE42KQo+ICsJICogLSBCaXRzIFsx LDMsNV0g4oaSIEMtUEhZIHRyaW8gbGFuZXMoTE4xLCBMTjMsIExONSkKPiArCSAqIC0gQml0IFs3 XSDihpIgRC1QSFkgY2xvY2sgbGFuZShMTkNLKSBkZWRpY2F0ZWQgY2xvY2sgZW5hYmxlCj4gKwkg Ki8KPiArCXZhbCA9IEJJVChsYW5lX2NmZy0+Y2xrLnBvcyk7Cj4gKwlmb3IgKGkgPSAwOyBpIDwg Y2ZnLT5udW1fZGF0YV9sYW5lczsgaSsrKQo+ICsJCXZhbCB8PSBCSVQobGFuZV9jZmctPmRhdGFb aV0ucG9zICogMik7CgpbU2V2ZXJpdHk6IE1lZGl1bV0KRG9lcyB0aGlzIGxvb3AgdW5jb25kaXRp b25hbGx5IGVuYWJsZSBhbGwgcGh5c2ljYWxseSBtYXBwZWQgbGFuZXMgaW5zdGVhZApvZiB0aGUg ZHluYW1pY2FsbHkgcmVxdWVzdGVkIGxhbmUgY291bnQ/CgpJZiBhIE1JUEkgQ1NJLTIgY29uc3Vt ZXIgY2FsbHMgcGh5X2NvbmZpZ3VyZSgpIHJlcXVlc3RpbmcgZmV3ZXIgYWN0aXZlIGxhbmVzCnRo YW4gdGhlIGhhcmR3YXJlIG1heGltdW0gbWFwcGVkIGluIHRoZSBkZXZpY2UgdHJlZSwgdGhlIFBI WSB3aWxsIHN0aWxsCnBvd2VyIHVwIGFsbCBvZiB0aGVtIGhlcmUgc2luY2UgY2ZnLT5udW1fZGF0 YV9sYW5lcyBpcyBkZXJpdmVkIGZyb20gdGhlCmRldmljZSB0cmVlIHBhcnNpbmcuIFNob3VsZCB0 aGlzIHVzZSBhIGR5bmFtaWNhbGx5IGNvbmZpZ3VyZWQgbGFuZSBjb3VudCwKb3IgYXQgbGVhc3Qg cmVqZWN0IHRoZSBjb25maWd1cmF0aW9uIGluIHBoeV9xY29tX21pcGlfY3NpMl9jb25maWd1cmUg aWYgaXQKZG9lc24ndCBtYXRjaCB0aGUgaGFyZHdhcmU/Cgo+ICsKPiArCXdyaXRlbCh2YWwsIGNz aTJwaHktPmJhc2UgKwo+ICsJICAgICAgIENTSVBIWV8zUEhfQ01OX0NTSV9DT01NT05fQ1RSTG4o cmVncy0+Y29tbW9uX3JlZ3Nfb2Zmc2V0LCA1KSk7ClsgLi4uIF0KCi0tIApTYXNoaWtvIEFJIHJl dmlldyDCtyBodHRwczovL3Nhc2hpa28uZGV2LyMvcGF0Y2hzZXQvMjAyNjA5MDMteDFlLWNzaTIt cGh5LXYxNy0wLTI2NjA2ZmE5YTAzOUBsaW5hcm8ub3JnP3BhcnQ9MgoKLS0gCmxpbnV4LXBoeSBt YWlsaW5nIGxpc3QKbGludXgtcGh5QGxpc3RzLmluZnJhZGVhZC5vcmcKaHR0cHM6Ly9saXN0cy5p bmZyYWRlYWQub3JnL21haWxtYW4vbGlzdGluZm8vbGludXgtcGh5Cg== From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.18]) (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 52BE64F30F4 for ; Thu, 3 Sep 2026 20:54:38 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788468881; cv=none; b=gbASIgPi3AQp+lqBaIpohMoLT9mMR8PXENwQiJP00OrzV8jGcWo5cDIWlzwPmF3O1dRtpGe3lsVmz2WNbPl93wQy1ZZeS7MRPgvj1bLYRAGT0BNsIWpljWuvvZJua7YwO957IJ9PbeiEgT4jB/IB7Nt6UqP3BXtPz7jB6Uvsms0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788468881; c=relaxed/simple; bh=JkbG6bLKBAQvTXxTv3/ryN6Jpt3t0r+47SqhRX/Fp7Q=; h=From:Subject:To:Cc:In-Reply-To:References:Content-Type:Date: Message-Id; b=ZhYJBWvrVNForNEGBRV6cCAjTs7YLyT1yMoA1RVW98lY2na7JEhbp2OPTI+legG3qR9nRhIwiqwPIBKDx9HF9KTt4rCXJvdbKH4DDjGxRgPaE9+x3AIoo2+uuc0DZ+aQ9K0sdmDUYjzjBvbbEI+LDfsU3UH76dFXZ9nIvdbQvo0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=TryPiCaP; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="TryPiCaP" Received: by smtp.kernel.org (Postfix) with ESMTPSA id DEA4E1F000E9; Thu, 3 Sep 2026 20:54:36 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1788468877; bh=aE9v9B5eRnSI6xpicfymScibzPae9lIzGDw+sAemqpg=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date; b=TryPiCaPrsWHDjSCEm5lBmFNFCzdD1HeunMKNdbljIo+xe7JRwuDnLBHb40wYpulO zb9bHJQuHmt5LLv9jBApe4hRTMOaAvyLNrmcEJKc8eKDmXhU1QrSAAZQcfYDZoc6Ug WvT2IYNkSYS4NGBl8/Xemia+cFvda+zgXj2MzLbF2o5B7GMyHZUF351rVxDRRkJ+lV 1buVae6tXyMoSm/7RxQYTZaIduiCoe4WYGxqvJ7GGnXGwO+dlx6ejUT4zw3ThtwBct BJvTJ24QgIqp7hKYBYnKVbtsz0lw3h4Zt3elI7FqDW8wos/vRqKzSeDNkbiySyZyfL xzOXRLGEnB+iQ== From: sashiko-bot@kernel.org Subject: Re: [PATCH v17 2/2] phy: qcom-mipi-csi2: Add a CSI2 MIPI DPHY driver Reply-To: sashiko-reviews@lists.linux.dev To: =?utf-8?b?QnJ5YW4gTydEb25vZ2h1ZQ==?= Cc: conor+dt@kernel.org, neil.armstrong@linaro.org, olteanv@gmail.com, robh@kernel.org, vkoul@kernel.org, linux-phy@lists.infradead.org, media-ci@linuxtv.org, devicetree@vger.kernel.org In-Reply-To: <20260903-x1e-csi2-phy-v17-2-26606fa9a039@linaro.org> References: <20260903-x1e-csi2-phy-v17-0-26606fa9a039@linaro.org> <20260903-x1e-csi2-phy-v17-2-26606fa9a039@linaro.org> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Date: Thu, 03 Sep 2026 20:54:36 +0000 Message-Id: <20260903205436.DEA4E1F000E9@smtp.kernel.org> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Thank you for your contribution! Sashiko AI review found 1 potential issue(= s) to consider: - [Medium] The MIPI D-PHY configuration ignores the requested number of act= ive lanes, enabling all physically mapped lanes instead. -- commit 80184194d436b4ea64c9ebc79907070b59c9de53 Author: Bryan O'Donoghue phy: qcom-mipi-csi2: Add a CSI2 MIPI DPHY driver Adds a new MIPI CSI2 driver in DPHY mode by importing existing CAMSS CSI PHY init sequences. Supports the qcom,x1e80100-csi2-phy device and includes data/clock lane positioning and polarity selection. > diff --git a/drivers/phy/qualcomm/phy-qcom-mipi-csi2-core.c b/drivers/phy= /qualcomm/phy-qcom-mipi-csi2-core.c > --- /dev/null > +++ b/drivers/phy/qualcomm/phy-qcom-mipi-csi2-core.c [ ... ] > +static int phy_qcom_mipi_csi2_configure(struct phy *phy, > + union phy_configure_opts *opts) > +{ > + struct mipi_csi2phy_device *csi2phy =3D phy_get_drvdata(phy); > + struct phy_configure_opts_mipi_dphy *dphy_cfg =3D &opts->mipi_dphy; > + struct mipi_csi2phy_stream_cfg *stream_cfg =3D &csi2phy->stream_cfg; > + int ret; > + > + ret =3D phy_mipi_dphy_config_validate(dphy_cfg); > + if (ret) > + return ret; > + > + if (dphy_cfg->lanes < 1 || dphy_cfg->lanes > CSI2_MAX_DATA_LANES) > + return -EINVAL; > + > + stream_cfg->link_freq =3D dphy_cfg->hs_clk_rate; [Severity: Medium] Should the requested active lane count from opts->mipi_dphy.lanes be stored in stream_cfg here? Currently, dphy_cfg->lanes is validated but the value does not appear to be saved for later use by the PHY enable routines. > + > + return 0; > +} [ ... ] > diff --git a/drivers/phy/qualcomm/phy-qcom-mipi-csi2-3ph-dphy.c b/drivers= /phy/qualcomm/phy-qcom-mipi-csi2-3ph-dphy.c > --- /dev/null > +++ b/drivers/phy/qualcomm/phy-qcom-mipi-csi2-3ph-dphy.c [ ... ] > +static int phy_qcom_mipi_csi2_lanes_enable(struct mipi_csi2phy_device *c= si2phy, > + struct mipi_csi2phy_stream_cfg *cfg) > +{ > + const struct mipi_csi2phy_device_regs *regs =3D csi2phy_dev_to_regs(csi= 2phy); > + struct mipi_csi2phy_lanes_cfg *lane_cfg =3D &cfg->lane_cfg; > + u8 settle_cnt; > + u8 val; > + int i; [ ... ] > + /* > + * CSI_COMMON_CTRL5 is a physical lane power-up bitmap: > + * - Bits [0,2,4,6] =E2=86=92 D-PHY data lanes(LN0, LN2, LN4, LN6) > + * - Bits [1,3,5] =E2=86=92 C-PHY trio lanes(LN1, LN3, LN5) > + * - Bit [7] =E2=86=92 D-PHY clock lane(LNCK) dedicated clock enable > + */ > + val =3D BIT(lane_cfg->clk.pos); > + for (i =3D 0; i < cfg->num_data_lanes; i++) > + val |=3D BIT(lane_cfg->data[i].pos * 2); [Severity: Medium] Does this loop unconditionally enable all physically mapped lanes instead of the dynamically requested lane count? If a MIPI CSI-2 consumer calls phy_configure() requesting fewer active lanes than the hardware maximum mapped in the device tree, the PHY will still power up all of them here since cfg->num_data_lanes is derived from the device tree parsing. Should this use a dynamically configured lane count, or at least reject the configuration in phy_qcom_mipi_csi2_configure if it doesn't match the hardware? > + > + writel(val, csi2phy->base + > + CSIPHY_3PH_CMN_CSI_COMMON_CTRLn(regs->common_regs_offset, 5)); [ ... ] --=20 Sashiko AI review =C2=B7 https://sashiko.dev/#/patchset/20260903-x1e-csi2-p= hy-v17-0-26606fa9a039@linaro.org?part=3D2