From nobody Thu Apr 9 15:50:48 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 E96E321B191; Sun, 1 Mar 2026 00:51:23 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772326284; cv=none; b=TTuCfZ9nZwwB3bKC+/hAPZutXCejyDUx3LmdOzfVUfTKHfFD8bdQZPovnu7xdP91hl32oeoP/ciMrAC0cK7VAtfeQ9ZpaDHgMdN2U0ljFimd44uSUVn+Cc/wZYrhodkLJ+yg1k035q0oNo8IqX6IF+mrL7rzf//hkLJbBBHI4KI= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772326284; c=relaxed/simple; bh=rhGfNydoUOkdRXybUMyg+yWuVRmt64y1v5Ezv+TIJqk=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=QeULTK3SMI6vQ7Oh7e/dOrkayHbo0H83AhOdtC7yg7wwzvJ8Ff6W56JQZHfrSj8EUVj4Tbx1gMkRX5J0zv/y8OyNMFZA2zxhEDMZJKG8yIO1ihMWQeENg2wHB5CY2pyfaCgkVGds4+Isppc7IUyPvsACCJftJZrBKaPhVNPcPWc= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=k8wUC1hE; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="k8wUC1hE" Received: by smtp.kernel.org (Postfix) with ESMTPS id BE441C19421; Sun, 1 Mar 2026 00:51:23 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1772326283; bh=rhGfNydoUOkdRXybUMyg+yWuVRmt64y1v5Ezv+TIJqk=; h=From:Date:Subject:References:In-Reply-To:To:Cc:Reply-To:From; b=k8wUC1hE2CGt/6iET7MjaHg6hFW+IRGpA4oDadJ19QwcpkzdX0FzewRamQVEIFHS+ 4vIlqbS7cbNQWRi8+J1CdMtWDGGDWufNTsuWhowEVg5f3WhX+7VSZmfu+/5QZD7D9m BwfKGa9kTfMFEu46vOeK4Pm34rabLT4+PW/d4+f7unJ3ABEEzkDT0VrhmrCPPzkVi+ VgLNuTce6E+goXsUxUyyBfHx4RQ7H4fak8uHE9qhkglMYWZ3/HF3XdFR8BR5p6RNIL NcueZTUwc784E3xWMlIp6hNahJ4X3ks699uj+eWuD2VnjUscmZXdNkOcv39Zbp8/Hd zeq5p9WuBVrqA== Received: from aws-us-west-2-korg-lkml-1.web.codeaurora.org (localhost.localdomain [127.0.0.1]) by smtp.lore.kernel.org (Postfix) with ESMTP id B3B2DFD004A; Sun, 1 Mar 2026 00:51:23 +0000 (UTC) From: David Heidelberg via B4 Relay Date: Sun, 01 Mar 2026 01:51:27 +0100 Subject: [PATCH WIP v4 8/9] media: qcom: camss: csiphy-3ph: C-PHY needs own lane configuration Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20260301-qcom-cphy-v4-8-e53316d2cc65@ixit.cz> References: <20260301-qcom-cphy-v4-0-e53316d2cc65@ixit.cz> In-Reply-To: <20260301-qcom-cphy-v4-0-e53316d2cc65@ixit.cz> To: Robert Foss , Todor Tomov , Bryan O'Donoghue , Vladimir Zapolskiy , Mauro Carvalho Chehab , Luca Weiss , Petr Hodina , Casey Connolly , "Dr. Git" Cc: Konrad Dybcio , Joel Selvaraj , Kieran Bingham , Sakari Ailus , linux-media@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, phone-devel@vger.kernel.org, David Heidelberg X-Mailer: b4 0.14.3 X-Developer-Signature: v=1; a=openpgp-sha256; l=1623; i=david@ixit.cz; h=from:subject:message-id; bh=xJUXUtwBIhekrOdLKRvTP/J2Ey1JhDpeic7sPTFnqSI=; b=owEBbQKS/ZANAwAIAWACP8TTSSByAcsmYgBpo42JBwEFoFHmRtNt2RnACQuIFO0B25Bk1KcHL LCMYuudgouJAjMEAAEIAB0WIQTXegnP7twrvVOnBHRgAj/E00kgcgUCaaONiQAKCRBgAj/E00kg ckb2D/9QD2HGudr+0YjoFLjEVf53a3ZwgZGfpLQBVrLFg4AdFWKoLg8EjGd2FpVxlSPdyQ+ObPz 2Uj/1Xz5yUgtRSuBQYjcgNkOSr6ZPQvDTSxLIQ7Wicu18wZRl3aWb3QlH7IfOpFnpLTKU78z5MO CB+zs7PcSfmQFwc8zdru/6n7xSjlptL1QjSLb5Jcy9lbel4vnM64Jv8sOJLK0pYJdGZbyxvzbIN IOj0sBQ7Ll2H3l1nwC6+wBNKf7ySLyaN5t1lS1A074AlKfvvtpTLXw6Vf4RQpefdX9raTB4ujD3 9BlGZQzqwgr4c7+Xaab1xhkYOTq43qoIlbohsSi1DKlIHvU8kGhBflQiDlZnNNzsLiX27BwuqGB S1mVgkcWXF2D0h4PBY2cPjkEPDXE3d3M7pqpMZzNnhZI1woevOOl6R5FVpHEwMGxueN+USnptPa ygHA17qun12HUtZfptjtyY59Vm6AztwRPc1fHUMlytTTkaaabkDIdvQmkG4aMR/AcccZG1RUcd+ A35YulDU26Tz/DHrQN6Y1bIdHDxGN5btZCmHLJXesPpuMIQIaYaVNKFTYeafjCNFnnpVOocbsA3 FdgaaSclrd0bQBWJBHOm7u9q7cmX82vkZ85V5eVN3XL9R0UoS5h+k47EvaRWe4NtUHp4d2oFReb czcWyDXMz2zV6zA== X-Developer-Key: i=david@ixit.cz; a=openpgp; fpr=D77A09CFEEDC2BBD53A7047460023FC4D3492072 X-Endpoint-Received: by B4 Relay for david@ixit.cz/default with auth_id=355 X-Original-From: David Heidelberg Reply-To: david@ixit.cz From: David Heidelberg Catch when C-PHY configuration gets used on SoC with CAMSS missing C-PHY configuration lane registers. Hopefully this check will disappear as these lane regs gets populated. -- @bod Proliferating special cases in switch statements on a per-SoC basis is verboten. Please find another way to do this, you already have a bool to indicate cphy in struct csid_phy_config {} so at some level CAMSS already has a bool to indicate what to do. Please make that logic accessible to logical consumers throughout, in this case the CPHY code. -- Signed-off-by: David Heidelberg --- drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c | 16 ++++++++++++= ++++ 1 file changed, 16 insertions(+) diff --git a/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c b/dri= vers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c index e48f5c1290173..2164647211c8d 100644 --- a/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c +++ b/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c @@ -1221,6 +1221,22 @@ static int csiphy_lanes_enable(struct csiphy_device = *csiphy, u8 val; int i; =20 + if (c->phy_cfg =3D=3D V4L2_MBUS_CSI2_CPHY) { + switch (csiphy->camss->res->version) { + case CAMSS_2290: + case CAMSS_8280XP: + case CAMSS_X1E80100: + case CAMSS_8550: + case CAMSS_8650: + case CAMSS_8300: + case CAMSS_8775P: + dev_err(dev, "Missing lane_regs definition for C-PHY\n"); + return -EINVAL; + default: + break; + } + } + switch (csiphy->camss->res->version) { case CAMSS_845: if (c->phy_cfg =3D=3D V4L2_MBUS_CSI2_CPHY) { --=20 2.51.0