<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jan 26, 2023 at 10:40 AM Krzysztof Kozlowski <<a href="mailto:krzysztof.kozlowski@linaro.org">krzysztof.kozlowski@linaro.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 25/01/2023 22:09, Jonathan Cormier wrote:<br>
> Add a i2c example with HDMI connector<br>
<br>
Why? It's the same - but more on this below.<br></blockquote><div>The existing example is for the previous setup where it was configured as its own device.  It seemed necessary now that the driver is going to support being connected to an i2c bus to show it being used as such. <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> <br>
> Signed-off-by: Jonathan Cormier <<a href="mailto:jcormier@criticallink.com" target="_blank">jcormier@criticallink.com</a>><br>
<br>
You need to test the bindings before sending and fix the errors.<br></blockquote><div>Will do <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> ---<br>
>  .../bindings/display/bridge/ti,tfp410.yaml         | 42 ++++++++++++++++++++++<br>
>  1 file changed, 42 insertions(+)<br>
> <br>
> diff --git a/Documentation/devicetree/bindings/display/bridge/ti,tfp410.yaml b/Documentation/devicetree/bindings/display/bridge/ti,tfp410.yaml<br>
> index 4c5dd8ec2951..456214f14b47 100644<br>
> --- a/Documentation/devicetree/bindings/display/bridge/ti,tfp410.yaml<br>
> +++ b/Documentation/devicetree/bindings/display/bridge/ti,tfp410.yaml<br>
> @@ -116,4 +116,46 @@ examples:<br>
>          };<br>
>      };<br>
>  <br>
> +  - |<br>
> +    i2c {<br>
> +      #address-cells = <1>;<br>
> +      #size-cells = <0>;<br>
> +<br>
> +      hdmi_encoder: tfp410@38 {<br>
<br>
Node names should be generic.<br>
<a href="https://devicetree-specification.readthedocs.io/en/latest/chapter2-devicetree-basics.html#generic-names-recommendation" rel="noreferrer" target="_blank">https://devicetree-specification.readthedocs.io/en/latest/chapter2-devicetree-basics.html#generic-names-recommendation</a><br><br></blockquote><div>Can do <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> +        compatible = "ti,tfp410";<br>
> +        reg = <0x38>;<br>
> +<br>
> +        ports {<br>
> +          address-cells = <1>;<br>
> +          size-cells = <0>;<br>
> +<br>
> +          port@0 {<br>
> +            reg = <0>;<br>
> +            tfp410_in: endpoint {<br>
> +              remote-endpoint = <&dpi1_out>;<br>
> +            };<br>
> +          };<br>
> +<br>
> +          port@1 {<br>
> +            reg = <1>;<br>
> +            tfp410_out: endpoint {<br>
> +              remote-endpoint = <&hdmi_connector_in>;<br>
> +            };<br>
<br>
That's the same example as existing one, so it looks useless. I don't<br>
see benefits of this example.<br></blockquote><div>It's mostly the same, except defined inside an i2c bus, with the reg value set. Without the powerdown-gpios or ti,deskew.  <br></div><div>And without the pclk-sample and bus-width (these are now read from i2c) <br></div><div>And I included the hdmi_connector so it would be a more complete and useful example of how it could be used.  The TFP410 doesn't handle the ddc i2c bus on its own so a separate connector node is needed.  I'll drop it if that's preferred.<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> +          };<br>
> +        };<br>
> +      };<br>
> +    };<br>
> +<br>
> +    hdmi: hdmi_connector {<br>
<br>
Drop. Incorrect name and not really related.<br>
<br>
> +      compatible = "hdmi-connector";<br>
> +      label = "hdmi";<br>
> +      type = "a";<br>
> +      ddc-i2c-bus = <&i2c1>;<br>
> +      port {<br>
> +        hdmi_connector_in: endpoint {<br>
> +          remote-endpoint = <&tfp410_out>;<br>
> +        };<br>
> +      };<br>
> +    };<br>
> +<br>
>  ...<br>
> <br>
<br>
Best regards,<br>
Krzysztof<br>
<br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div>Jonathan Cormier<br>Software Engineer<div><div><div><div><br></div><div><div><font color="#000000">Voice:  <a value="+13154254045">315.425.4045</a> x222</font></div><div><font color="#000000"><br></font></div><div><font color="#000000"><img src="https://ci3.googleusercontent.com/mail-sig/AIorK4xml75fJPKYF7M9X5Dux4IrbtzZk0RB6jf8iBYuPQKArrkBkOkU4HJ2rP7eH6d2h7_ituMWEBc"><br></font></div></div></div><div><div><font color="#1155cc"><br></font></div><div><a href="http://www.criticallink.com/" target="_blank"><font color="#1155cc">http://www.CriticalLink.com</font></a><br></div><div>6712 Brooklawn Parkway, Syracuse, NY 13211 </div></div></div><div><div><br></div><div><font color="#1155cc"><font color="#1155cc"><a href="https://www.linkedin.com/company/critical-link-llc" target="_blank"><img src="https://docs.google.com/uc?export=download&id=0B2vNSnu-aYu6OEhHRW9BUFV5WnM&revid=0B2vNSnu-aYu6RHNZUnhNbFpER1l3emNQY2VoaHA0RDdudWlFPQ"></a>  <a href="https://twitter.com/Critical_Link" target="_blank"><img alt="" src="https://docs.google.com/uc?export=download&id=0B2vNSnu-aYu6cU1yWERrLXE0SnM&revid=0B2vNSnu-aYu6b1YrZW1SM0hueVhVS0pPWm1IOXFSc0I3ay9jPQ"></a></font></font></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div>