[PATCH] ARM: shmobile: r8a7779: Correct TMU clock support again

Simon Horman horms at verge.net.au
Wed Feb 13 23:42:22 EST 2013


On Thu, Feb 14, 2013 at 01:36:49PM +0900, Simon Horman wrote:
> On Wed, Feb 13, 2013 at 08:25:45PM -0800, Kuninori Morimoto wrote:
> > 
> > Hi Simon
> > 
> > > > >>       CLKDEV_DEV_ID("sh_tmu.0", &mstp_clks[MSTP016]), /* TMU00 */
> > > > >> -     CLKDEV_DEV_ID("sh_tmu.1", &mstp_clks[MSTP015]), /* TMU01 */
> > > > >> +     CLKDEV_DEV_ID("sh_tmu.1", &mstp_clks[MSTP016]), /* TMU01 */
> > > > >>       CLKDEV_DEV_ID("sh_tmu.2", &mstp_clks[MSTP014]), /* TMU02 */
> > (snip)
> > > > This means that current TMU02 numbering seems doubtful too ?
> > > > How about just rever 58079fa7d54a0929d304054ee759187a2ccd3cdf ?
> > > 
> > > Perhaps that is a good idea.
> > > 
> > > The original motivation for this patch was to add the TMU02 line.
> > > And "fixing" TMU01 was an afterthought. However, I am also
> > > now doubtful about the correctness of the TMU02 line and thus
> > > the usefulness of the original patch.
> > 
> > I think this comment out is creating confusion ?
> > 
> > /* TMU00 */ -> /* TMU0 channel 0 */
> > /* TMU01 */ -> /* TMU0 channel 1 */
> > /* TMU02 */ -> /* TMU0 channel 2 */
> 
> That does make things a little clearer,
> but I for one wasn't confused by the existing comments.

Ok, now I think I see what you are getting at.

MSTP016 is for channel 0..n of TMU0
MSTP015 is for channel 1..n of TMU1
MSTP014 is for channel 2..n of TMU2

And all the TMU values set by CLKDEV_DEV_ID statements above
relate to channels of TMU0.

That does seem to be a valid interpretation of the documentation
now that I read it again.




More information about the linux-arm-kernel mailing list