Hey Rob could I get your review input on this patch ?

Bryan O'Donoghue bod at nxsw.ie
Wed Jul 8 15:06:11 PDT 2026


Hi Rob.

You gave me some review feedback here:

https://lore.kernel.org/linux-media/20250710230846.GA44483-robh@kernel.org/

Through subsequent iterations of this patch we have added a 
port/remote-endpoint.

Current thread is here:

https://lore.kernel.org/linux-media/3ab676ac-919b-4a80-8dd6-71dd7f6fe06c@kernel.org

Vlad's point is that linux-media bus-type should be used to define the 
mode of the PHY. I've been sticking to your original suggestion that the 
consumer defines the mode.

https://lore.kernel.org/linux-media/6b6492a4-7610-4dce-a81c-8dc0387a4061@linaro.org

I've been citing you but, I should have _checked_in_with_you_ given 
evolution of this binding.

Perhaps its something Krzysztof or Conor can resolve from a DT schema 
perspective ? I trust either of you guys to "do the right thing" - I'd 
really just like to get an official schema maintainer steer on this one, 
so I can close it and move on.

So: Stick with #phy-cells = 1 and the consumer selecting DPHY/CPHY mode 
or #phy-cells = 0 and inferring the mode from endpoint bus-type ?

---
bod




More information about the linux-phy mailing list