From mboxrd@z Thu Jan 1 00:00:00 1970 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 3BB833C8709; Mon, 30 Mar 2026 11:41:56 +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=1774870916; cv=none; b=gsQ0ClWFfznf+kIXmQWWqyPPjPbZhM/YDVzptxeW/WQoaDu4hF7vPidLMLsXd+CaDkstiPSGEE8v1Z7sm/Sf0yCt4ubS0RmJjCq6Lamu1oAGxEXTVwJKQqvb3HJOnYYViR6ljvh1Y1ptVMgyXqk5CXHMseDywzsykbNRrORMGdk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774870916; c=relaxed/simple; bh=EOIraZfT3LaP1D8hYweBX5dLmcXzYF30cAdNPSQCC2Q=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=E4KCo+BU40bzpfLQ2fMK/pZfz0B7Co7XoM2e2BVZsx4rgL+UC8FBQd4fvE5kgEB8/HKv0UU2E7sVZXJnFMC/OHh4BtUrT9ViR9da6JtG9VLNoHKyYu/1l3rbpAvSjzo8+MLeJVMoorrPkyusuHlfCiR1a7poaB/1svIm852rDWA= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=tUtyGXBo; 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="tUtyGXBo" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 22243C2BCB2; Mon, 30 Mar 2026 11:41:52 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1774870916; bh=EOIraZfT3LaP1D8hYweBX5dLmcXzYF30cAdNPSQCC2Q=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=tUtyGXBos9E2kXQ+5HdLG8McmasamfKnl9dYARbBogmVfzz9GcyRGRe8LVdCFbKme NaoNWbLPPBMtsuWj92M/JxDDEbSiazsngt39lGpjtTyvzTZjmJoPRdvtM9mk9Aei1L WulxCoFXEuvssF7/TelFfzD9O+KfZdigxxItoygaXWAALCCoSvuSqj7APNKc4aUIhK BI5cUMYiGqFBW8rhOquKY1LR/Dfo9f3SMu+pFPlF0Rd+NW8YG6q4XSCscR/AJeRM8s 7T8mORrYyLxr9Rl0scKJ5rI7GnftQX1/wZdUjtarBud/MEy/QqQp6eaBHvYzHQKq4/ 0KRhEtzxPay0Q== Message-ID: <6ea214cf-e68e-44c6-ba0c-38aa0a644ae9@kernel.org> Date: Mon, 30 Mar 2026 12:41:51 +0100 Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v5 1/2] dt-bindings: phy: qcom: Add CSI2 C-PHY/DPHY schema To: Konrad Dybcio , Neil Armstrong , Bryan O'Donoghue , Vinod Koul , Kishon Vijay Abraham I , Rob Herring , Krzysztof Kozlowski , Conor Dooley Cc: Vladimir Zapolskiy , linux-arm-msm@vger.kernel.org, linux-phy@lists.infradead.org, linux-media@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <20260326-x1e-csi2-phy-v5-0-0c0fc7f5c01b@linaro.org> <20260326-x1e-csi2-phy-v5-1-0c0fc7f5c01b@linaro.org> <7712fbdd-a225-49f0-aeb9-ebcbb9d5abac@oss.qualcomm.com> <1f38187a-9464-4aa9-b70a-03b767349d56@linaro.org> <016c03b8-27c3-41dc-a630-8e7095db1f88@linaro.org> <456ded59-d13e-4b61-975b-97ca48b5e771@linaro.org> <76ea03d0-d41b-4880-a48c-06570eb089ed@linaro.org> <0101d8bc-1ae8-475e-bb9e-cc1e16db87ec@kernel.org> <6d5ebab6-3c4e-4a1c-9578-6e926cbd96de@oss.qualcomm.com> From: Bryan O'Donoghue Content-Language: en-US In-Reply-To: <6d5ebab6-3c4e-4a1c-9578-6e926cbd96de@oss.qualcomm.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 30/03/2026 12:34, Konrad Dybcio wrote: > Let's take a step back - since any CSIPHY can feed into any CSID (at runtime), > the resulting nodes would either look like: > > // hardcoded, m may != n > csid_n: csid@1000000 { > phys = <&csiphy_m>; > }; > Well that would be wrong they can connect to any CSID. We'd be churning the user-space ABI and imposing an artificial constraint on what the hw can do. > > // determined at runtime > csid_n: csid@1000000 { > phys = <&csiphy_0>, > [...] > <&csiphy_n-1>; > }; This I think works well and actually maps to what the hardware can do. This would be where to talk more about Neil's mux. > > or we could store them once, centrally, in the "CAMSS_TOP" node and > pass handles around as necessary: > > // camss "catalog/manager" driver/library provides CSIDn with PHYm > camss: camss@10000000 { > phys = <&csiphy_0>, > [...] > <&csiphy_n-1>; > > csid_n: csid@1000 { > // no PHY references > }; > }; That could work too. Either way I think this is to be hashed out when doing CSID. --- bod