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

From: Bryan O'Donoghue

Date: Tue Jul 21 2026 - 21:53:30 EST


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