<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 20, 2017 at 10:30 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 Mon, Mar 20, 2017 at 10:24:20AM -0700, Maxim Sloyko wrote:<br>
> On Sun, Mar 19, 2017 at 9:42 AM, Tom Rini <<a href="mailto:trini@konsulko.com">trini@konsulko.com</a>> wrote:<br>
><br>
> > 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>
> > ++++++++++++++++++++++---<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<br>
> > 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>
> > 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>
><br>
> There is no clock driver for this part in mainline Linux Kernel yet and I<br>
> don't know how it will end up being configured. I suspect that they might<br>
> not use the same bindings though.<br>
><br>
> Should I put this into board specific dts?<br>
<br>
</div></div>So this applies to a lot of parts of the series here.  What we don't<br>
want to do is have places where the DTS here diverges from the Linux<br>
kernel DTS and we don't reconcile them.  If the relevant Linux drivers<br>
are not in mainline, are they at least in linux-next or otherwise<br>
submitted to the relevant subtrees?<br></blockquote><div><br></div><div>No, as far as I know, maybe Rick (cc'ed) knows what is the plan there.</div><div><br></div><div>I'm not really working on the linux driver and it's outside of my control.</div><div><br></div><div>I can change network driver, so that it does not use this DT configuration and either hard code clock config into it, or configure it's clocks in board specific file -- would that be more acceptable?</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>