Re: [PATCH v13 1/2] dt-bindings: phy: qcom: Add CSI2 C-PHY/DPHY schema

[Bryan O'Donoghue <bod@kernel.org>]

On 22/07/2026 02:17, Frank Li wrote:
>> Such design has a number of flaws and in general it looks overly
>> complicated. From the beginning it's unclear how to distinguish
>> a clock lane from a data lane, how to specify lane frequencies etc.
> Thanks, but phy-cell support more than one argument. like
> 
> 	<&phy 0 mode_cphy link_freq>
> 
> I ask these because simiar other phys may connect to display panel, some
> phys combine csi and dsi together.

For a combined CSI/DSI PHY that's a good model, and it re-establishes
the principle that the consumer chooses the mode.

For our simpler camera-only case the cell has less work to do. The
series carries #phy-cells = 1 per Rob's feedback and an RB from Krzysztof.

That's reviewed, tested and mergeable.

The one alternative under discussion with Vinod is bus-type in the CSID 
controller's endpoint with the mode conveyed via phy_configure(). Either 
way the consumer decides.

---
bod