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

From: Bryan O'Donoghue

Date: Mon Mar 23 2026 - 11:03:43 EST


On 23/03/2026 14:22, Konrad Dybcio wrote:
Sure, FYI: Dmitry, Konrad.
I'll admit I haven't yet read up on all of the background discussions that you
guys had on LKML, but*if* we're going to put the PHY under camss, the GDSC and
CPAS_AHB/CAMNOC_AXI_RT references should be unnecessary, given they're not
related strictly to this PHY itself, rather it sitting in a specific corner of
the subsystem that needs them to be active (see related:
https://lore.kernel.org/linux-arm-msm/cb2430f2-8601-4c72- af6b-10f1ff16c188@xxxxxxxxxxxxxxxx/
)

That's fair comment with the PHYs inside of the CAMSS block. Obviously if its outside of the block we need the full gamut of clocks defined.

..

Yeah I think I'm happy enough to drop these predicated on sub-nodes.

For the other names, I*think* we won't need to rely on them (i.e. only operate
the resources through PHY APIs from the V4L2 driver) and can come up with new
ones. And hence I think we can turn "csiphy" to "core" and "csiphy_timer" to
"timer" (because we really don't need to repeat the csiphy_ prefix)

Works for me.

---
bod