Re: [PATCH v8 1/2] dt-bindings: phy: qcom: Add CSI2 C-PHY/DPHY schema
From: Vladimir Zapolskiy
Date: Thu Jul 02 2026 - 04:41:16 EST
On 7/2/26 03:08, Bryan O'Donoghue wrote:
On 02/07/2026 00:37, Vladimir Zapolskiy wrote:
Inserting bus-type into the PHY layer is borrowing from one domain
linux-media and pushing the concept into PHYs.
It's the other way around, "bus-type" is an endpoint property of media
devices, reference to video-interfaces.yaml to get more details about it.
phy-cells = 1 with CPHY/DPHY specified in the consumer, as was Rob's
suggestion will specify the mode.
This cell is just not needed, and unneeded complexity should not be added.
I'm going to stick to Rob's initial guidance on this.
https://lore.kernel.org/linux-media/20250710230846.GA44483-robh@xxxxxxxxxx/
Sure, the discussion will be continued abouve v9 of the series, it's
just a chance to do it now. Rob properly pointed out, that the invented
"phy-type" is not needed at all.
Ther fact is that the whole placement of the CSIPHY to phy subsystem
looks execessive, since you add a driver for a media device, there will
be nothing PHY driver specific left in the CSIPHY driver but name.
--
Best wishes,
Vladimir