<!DOCTYPE html>
<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
  </head>
  <body>
    <p><br>
    </p>
    <div class="moz-cite-prefix">On 3/8/24 09:28, Krzysztof Kozlowski
      wrote:<br>
    </div>
    <blockquote type="cite"
      cite="mid:578f421c-ca06-45d4-8380-8b2b423d4d47@linaro.org">
      <pre class="moz-quote-pre" wrap="">On 07/03/2024 14:59, Christophe Roullier wrote:
</pre>
      <blockquote type="cite">
        <pre class="moz-quote-pre" wrap="">Add property st,ext-phyclk to manage cases when PHY have no cristal/quartz
This property can be used with RMII phy without cristal 50Mhz and when we
want to select RCC clock instead of ETH_REF_CLK
Can be used also with RGMII phy with no cristal and we select RCC clock
instead of ETH_CLK125

</pre>
      </blockquote>
      <pre class="moz-quote-pre" wrap="">
Nothing improved here. You say you add new property (wrote it explicitly
in the subject), but where is it? Where is the user?

I think we talked about this. Rob also asked quite clear:

</pre>
      <blockquote type="cite">
        <pre class="moz-quote-pre" wrap="">That is obvious from the diff. What is not obvious is why we need a new
property and what is the problem with the existing ones.
</pre>
      </blockquote>
      <pre class="moz-quote-pre" wrap="">
How did you solve it?</pre>
    </blockquote>
    <p>Hi,</p>
    <p>I do not understand your questions.</p>
    <p>That I would like to do, it is property "<span
      style="white-space: pre-wrap">st,ext-phyclk" was introduced in driver </span></p>
    <p><span style="white-space: pre-wrap">"drivers/net/ethernet/stmicro/stmmac/dwmac-stm32.c" in 2020, and YAML was not updated at the time.</span></p>
    <p><span style="white-space: pre-wrap">Goal of this patch it is to update YAML to avoid dtbs check issue if someone use this property :</span></p>
    <pre class="moz-quote-pre" wrap=""> dtbs check issue : views/kernel/upstream/net-next/arch/arm/boot/dts/st/stm32mp157c-dk2.dtb: 
ethernet@5800a000: Unevaluated properties are not allowed 
('st,ext-phyclk' was unexpected) </pre>
    <p></p>
    <p><span style="white-space: pre-wrap">Regards,</span></p>
    <p><span style="white-space: pre-wrap">Christophe
</span></p>
    <blockquote type="cite"
      cite="mid:578f421c-ca06-45d4-8380-8b2b423d4d47@linaro.org">
      <pre class="moz-quote-pre" wrap="">

Best regards,
Krzysztof

</pre>
    </blockquote>
  </body>
</html>