[PATCH v5] powerpc/mpc85xx: Update the clock nodes in device tree

[Scott Wood]

On Mon, 2013-10-14 at 20:59 -0500, Tang Yuantian-B29983 wrote:
> > -----Original Message-----
> > From: Wood Scott-B07421
> > Sent: 2013年10月15日 星期二 6:13
> > To: Tang Yuantian-B29983
> > Cc: Wood Scott-B07421; Mark Rutland; devicetree at vger.kernel.org;
> > linuxppc-dev at lists.ozlabs.org; Li Yang-Leo-R58472
> > Subject: Re: [PATCH v5] powerpc/mpc85xx: Update the clock nodes in device
> > tree
> > 
> > On Fri, 2013-10-11 at 21:52 -0500, Tang Yuantian-B29983 wrote:
> > > Thanks for your review.
> > >
> > > > -----Original Message-----
> > > > From: Wood Scott-B07421
> > > > Sent: 2013年10月12日 星期六 3:07
> > > > To: Mark Rutland
> > > > Cc: Tang Yuantian-B29983; devicetree at vger.kernel.org; linuxppc-
> > > > dev at lists.ozlabs.org; Li Yang-Leo-R58472
> > > > Subject: Re: [PATCH v5] powerpc/mpc85xx: Update the clock nodes in
> > > > device tree
> > > >
> > > > I'm not sure I understand the "_0"/"_1" part, though.  Doesn't each
> > > > PLL just have one output, which the consumer may choose to divide by
> > > > 2 (or in some cases 4)?  Why does that division need to be exposed
> > > > in the device tree as separate connections to the parent clock?
> > > >
> > > The device tree makes that quite clear.
> > 
> > You chose to model it that way in the device tree; that doesn't make it
> > clear that the hardware works that way or that it's a good way to model
> > it.
> > 
> > > Each PLL has several output which MUX node can take from.
> > 
> > Point out where in the hardware documentation it says this.  What I see
> > is a PLL that has one output, and a MUX register that can choose from
> > multiple PLL and divider options.
> > 
> Take T4240 for example: see section 4.6.5.1 , (Page 141) in T4240RM Rev. D, 09/2012.

That shows the dividers as being somewhere in between the PLL and the
MUX.  The MUX is where the divider is selected.  There's nothing in the
PLL's programming interface that relates to the dividers.  As such it's
simpler to model it as being part of the MUX.

-Scott