Re: [PATCH] am35xx-emac: move generic EMAC init to separate file

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 21 Dec 2011, Ilya Yanok wrote:

> On 16.12.2011 10:13, Paul Walmsley wrote:
> >> Ok, let's wait for Paul's answer and then I'll prepare a separate patch.
> > 
> > Unfortunately the AM3517 TRM (SPRUGR0) here doesn't really have the same 
> > level of clock integration information that the WBU TRMs have, so it's 
> > kind of hard to tell what's going on :-(
> > 
> > Looking at Figure 22-1 "EMAC and MDIO Block Diagram", it appears that what 
> > we call "emac_fck" is really just an optional functional clock for the 
> > MDIO PHY?
> 
> Hm. I have to admit I'm completely lost here.
> CONTROL_IPSS_CLK_CTRL register has bit controlling "Func clock of
> CPGMAC". 

Yeah, I don't think that's correct.  Or rather, it's imprecise.  It might 
be true from a strict integration perspective, but it's misleading.

> But there is no mention of this clock anywhere else in the
> manual. I can't find neither where it comes from nor how it's used.
> According to the current sources it comes from external 50MHz RMII
> clock, do you think this is correct?

I think that part is correct.

> > And it sounds like the AM35xx clock that we call "emac_ick" is actually a 
> > combined interface and functional clock for the EMAC and MDIO IP blocks?  
> > I guess a combined interface/functional clock would make sense, since the 
> > EMAC seems to contain a DMA controller.
> >   
> > Maybe Ranjith can provide some more information; he's cc'ed.
> > 
> > In any case, your changes sound reasonable to me, so a patch to the clock 
> > file sounds good.  I'd suggest both changing the clkdev aliases and 
> > renaming emac_fck - that's a confusing name and I don't think it's in the 
> > TRM as such.
> 
> Actually it's called Func clock in the manual (see
> CONTROL_IPSS_CLK_CTRL). Do you still think we have to rename it?

I guess we have other optional functional clocks with _fck suffixes, so 
it's probably not such a big deal.

> I have another concern: changing clkdev aliases make things work for me
> but now I'm not sure if this change is really correct...

Well, sounds like it's better than what we currently have.


- Paul
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux