[PATCH] dt-bindings: phy: qcom,sc8280xp-qmp-usb43dp: Add mode-switch

Neil Armstrong posted 1 patch 1 month, 1 week ago
.../devicetree/bindings/phy/qcom,sc8280xp-qmp-usb43dp-phy.yaml       | 5 +++++
1 file changed, 5 insertions(+)
[PATCH] dt-bindings: phy: qcom,sc8280xp-qmp-usb43dp: Add mode-switch
Posted by Neil Armstrong 1 month, 1 week ago
The QMP USB3/DP Combo PHY can work in 3 modes:
- DisplayPort Only
- USB3 Only
- USB3 + DisplayPort Combo mode

In order to switch between those modes, the PHY needs to receive
Type-C events, allow marking to the phy with the mode-switch
property in order to allow the PHY to receive Type-C events.

Referencing usb-switch.yaml lookkied like as a simpler way to allow
the mode-switch property instead of duplicating the property
definition but it causes some issues with the ports definitions.

Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
---
Depends on:
[1] https://lore.kernel.org/all/20251006-topic-sm8x50-fix-qmp-usb43dp-usb-switch-v2-1-3249e511013b@linaro.org/
---
 .../devicetree/bindings/phy/qcom,sc8280xp-qmp-usb43dp-phy.yaml       | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-usb43dp-phy.yaml b/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-usb43dp-phy.yaml
index 38ce04c35d945d0d8d319191c241920810ee9005..bdf7894ff84869030252f64f20acb4f920fd19e9 100644
--- a/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-usb43dp-phy.yaml
+++ b/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-usb43dp-phy.yaml
@@ -73,6 +73,11 @@ properties:
     description:
       See include/dt-bindings/phy/phy-qcom-qmp.h
 
+  mode-switch:
+    description:
+      Flag the PHY as possible handler of USB Type-C altmode switching
+    type: boolean
+
   orientation-switch:
     description:
       Flag the PHY as possible handler of USB Type-C orientation switching

---
base-commit: 9cba8ad3e538c0d955a9844fbe26a887dbb04f4a
change-id: 20251006-topic-sm8x50-qmp-combo-allow-mode-switch-a75e5b55b7df

Best regards,
-- 
Neil Armstrong <neil.armstrong@linaro.org>
Re: [PATCH] dt-bindings: phy: qcom,sc8280xp-qmp-usb43dp: Add mode-switch
Posted by Rob Herring 1 month ago
On Mon, Oct 06, 2025 at 03:51:53PM +0200, Neil Armstrong wrote:
> The QMP USB3/DP Combo PHY can work in 3 modes:
> - DisplayPort Only
> - USB3 Only
> - USB3 + DisplayPort Combo mode
> 
> In order to switch between those modes, the PHY needs to receive
> Type-C events, allow marking to the phy with the mode-switch
> property in order to allow the PHY to receive Type-C events.
> 
> Referencing usb-switch.yaml lookkied like as a simpler way to allow
> the mode-switch property instead of duplicating the property
> definition but it causes some issues with the ports definitions.
> 
> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
> ---
> Depends on:
> [1] https://lore.kernel.org/all/20251006-topic-sm8x50-fix-qmp-usb43dp-usb-switch-v2-1-3249e511013b@linaro.org/

That's a series then...
Re: [PATCH] dt-bindings: phy: qcom,sc8280xp-qmp-usb43dp: Add mode-switch
Posted by Rob Herring (Arm) 1 month ago
On Mon, 06 Oct 2025 15:51:53 +0200, Neil Armstrong wrote:
> The QMP USB3/DP Combo PHY can work in 3 modes:
> - DisplayPort Only
> - USB3 Only
> - USB3 + DisplayPort Combo mode
> 
> In order to switch between those modes, the PHY needs to receive
> Type-C events, allow marking to the phy with the mode-switch
> property in order to allow the PHY to receive Type-C events.
> 
> Referencing usb-switch.yaml lookkied like as a simpler way to allow
> the mode-switch property instead of duplicating the property
> definition but it causes some issues with the ports definitions.
> 
> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
> ---
> Depends on:
> [1] https://lore.kernel.org/all/20251006-topic-sm8x50-fix-qmp-usb43dp-usb-switch-v2-1-3249e511013b@linaro.org/
> ---
>  .../devicetree/bindings/phy/qcom,sc8280xp-qmp-usb43dp-phy.yaml       | 5 +++++
>  1 file changed, 5 insertions(+)
> 

Acked-by: Rob Herring (Arm) <robh@kernel.org>