[PATCH v2 3/4] Documentation: bindings: add phy_config for Rockchip USB Type-C PHY

Hi Lin,

2018-05-11 16:43 GMT+02:00 Sean Paul [off-list ref]:

On Wed, May 09, 2018 at 06:22:43PM +0800, Lin Huang wrote:

quoted

If want to do training outside DP Firmware, need phy voltage swing
and pre_emphasis value.

Signed-off-by: Lin Huang <redacted>

Adding Rob Herring so he has a hope of seeing this.

And adding the devicetree ML, otherwise Rob will probably ignore the patch.

quoted

---
Changes in v2:
- rebase

 Documentation/devicetree/bindings/phy/phy-rockchip-typec.txt | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/phy/phy-rockchip-typec.txt b/Documentation/devicetree/bindings/phy/phy-rockchip-typec.txt
index 960da7f..eda26dd 100644
--- a/Documentation/devicetree/bindings/phy/phy-rockchip-typec.txt
+++ b/Documentation/devicetree/bindings/phy/phy-rockchip-typec.txt

@@ -17,7 +17,9 @@ Required properties:

 Optional properties:
  - extcon : extcon specifier for the Power Delivery
-
+ - rockchip,phy_config : That's phy voltage swing and pre_emphasis
+                      setting, if want to do dp training outside
+                      dp firmware, need to add these value.

What are the units?

Also, I think that will be good add an example or include the property
in the current examples to see how works. It's not clear from the
property description. I suppose it is an array of pairs <swing
pre-emphasis>?

nit:

-rockchip,phy_config: A list of voltage swing (unit) and pre-emphasis
(unit) pairs. Use this property to enable software link training
instead of hardware link training.

Best regards,
 Enric

Sean

quoted

 Required nodes : a sub-node is required for each port the phy provides.
               The sub-node name is used to identify dp or usb3 port,
               and shall be the following entries:
--
2.7.4

--
Sean Paul, Software Engineer, Google / Chromium OS