<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Mar 19, 2017 at 9:42 AM, Tom Rini <span dir="ltr"><<a href="mailto:trini@konsulko.com" target="_blank">trini@konsulko.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Thu, Mar 16, 2017 at 02:36:20PM -0700, Maxim Sloyko wrote:<br>
> Add support for clocks needed by MACs to ast2500 clock driver.<br>
> The clocks are D2-PLL, which is used by both MACs and PCLK_MAC1 and<br>
> PCLK_MAC2 for MAC1 and MAC2 respectively.<br>
><br>
> The rate of D2-PLL is hardcoded to 250MHz -- the value used in Aspeed<br>
> SDK. It is not entirely clear from the datasheet how this clock is used<br>
> by MACs, so not clear if the rate would ever need to be different. So,<br>
> for now, hardcoding it is probably safer.<br>
><br>
> The rate of PCLK_MAC{1,2} is chosen based on MAC speed selected through<br>
> hardware strapping.<br>
><br>
> So, the network driver would only need to enable these clocks, no need<br>
> to configure the rate.<br>
><br>
> Signed-off-by: Maxim Sloyko <<a href="mailto:maxims@google.com">maxims@google.com</a>><br>
> ---<br>
><br>
>  arch/arm/dts/ast2500-u-boot.<wbr>dtsi               |   8 +<br>
>  arch/arm/include/asm/arch-<wbr>aspeed/scu_ast2500.h |  62 +++++-<br>
>  drivers/clk/aspeed/clk_<wbr>ast2500.c               | 265 ++++++++++++++++++++++---<br>
>  include/dt-bindings/clock/<wbr>ast2500-scu.h        |   2 +<br>
>  4 files changed, 304 insertions(+), 33 deletions(-)<br>
><br>
> diff --git a/arch/arm/dts/ast2500-u-boot.<wbr>dtsi b/arch/arm/dts/ast2500-u-boot.<wbr>dtsi<br>
> index faeeec1be4..f826646095 100644<br>
> --- a/arch/arm/dts/ast2500-u-boot.<wbr>dtsi<br>
> +++ b/arch/arm/dts/ast2500-u-boot.<wbr>dtsi<br>
> @@ -61,3 +61,11 @@<br>
>               };<br>
>       };<br>
>  };<br>
> +<br>
> +&mac0 {<br>
> +     clocks = <&scu PCLK_MAC1>, <&scu PLL_D2PLL>;<br>
> +};<br>
> +<br>
> +&mac1 {<br>
> +     clocks = <&scu PCLK_MAC2>, <&scu PLL_D2PLL>;<br>
> +};<br>
<br>
</div></div>Why is this here and not in the main dts file?  The -u-boot.dtsi is for<br>
stuff that's not appropriate in the upstream dts file.  Thanks!<br></blockquote><div><br></div><div>There is no clock driver for this part in mainline Linux Kernel yet and I don't know how it will end up being configured. I suspect that they might not use the same bindings though.</div><div><br></div><div>Should I put this into board specific dts?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Tom<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div><b>M</b>axim <b>S</b>loyko</div></div>
</div></div>