Re: [PATCH 1/6] OMAP4: Add missing clock divider for OCP_ABE_ICLK

2011-09-14 Thread Jon Hunter
Hi Paul On 8/28/2011 23:09, Paul Walmsley wrote: Just doublechecking on this patch. Does anything need to be changed before we merge this? I can take a look at the autogeneration script if that's the only issue left. Sorry, I have been out on holiday. I believe that was the only problem

Re: [PATCH 1/6] OMAP4: Add missing clock divider for OCP_ABE_ICLK

2011-08-28 Thread Paul Walmsley
Hi Benoît, Jon, On Wed, 27 Jul 2011, Cousson, Benoit wrote: On 7/15/2011 4:34 PM, Hunter, Jon wrote: On 7/15/2011 3:21, Paul Walmsley wrote: On Thu, 14 Jul 2011, Jon Hunter wrote: From: Jon Hunterjon-hun...@ti.com The parent clock of the OCP_ABE_ICLK is the AESS_FCLK and

Re: [PATCH 1/6] OMAP4: Add missing clock divider for OCP_ABE_ICLK

2011-07-26 Thread Cousson, Benoit
Hi Jon, On 7/15/2011 4:34 PM, Hunter, Jon wrote: Hi Paul, On 7/15/2011 3:21, Paul Walmsley wrote: cc'ing Benoît Hi Jon On Thu, 14 Jul 2011, Jon Hunter wrote: From: Jon Hunterjon-hun...@ti.com The parent clock of the OCP_ABE_ICLK is the AESS_FCLK and the parent clock of the AESS_FCLK is

Re: [PATCH 1/6] OMAP4: Add missing clock divider for OCP_ABE_ICLK

2011-07-18 Thread Jon Hunter
On 7/15/2011 9:34, Jon Hunter wrote: 2). The ocp_abe_iclk is an interface clock and is not a parent to any other functional clock and therefore, is not driving any internal logic in a peripheral which would have a direct impact of the speed of that peripheral. However, there does appear to be

Re: [PATCH 1/6] OMAP4: Add missing clock divider for OCP_ABE_ICLK

2011-07-18 Thread Jon Hunter
On 7/18/2011 15:57, Jon Hunter wrote: On further inspection of the clock44xx_data.c, it appears that all interface clocks are called xxx_fck and not xxx_ick. I will ask Benoit about this. However, this particular clock we are dealing with here is an interface clock. Actually, the above it not

Re: [PATCH 1/6] OMAP4: Add missing clock divider for OCP_ABE_ICLK

2011-07-15 Thread Paul Walmsley
cc'ing Benoît Hi Jon On Thu, 14 Jul 2011, Jon Hunter wrote: From: Jon Hunter jon-hun...@ti.com The parent clock of the OCP_ABE_ICLK is the AESS_FCLK and the parent clock of the AESS_FCLK is the ABE_FCLK... ABE_FCLK -- AESS_FCLK -- OCP_ABE_ICLK The AESS_FCLK and OCP_ABE_ICLK clocks

Re: [PATCH 1/6] OMAP4: Add missing clock divider for OCP_ABE_ICLK

2011-07-15 Thread Jon Hunter
Hi Paul, On 7/15/2011 3:21, Paul Walmsley wrote: cc'ing Benoît Hi Jon On Thu, 14 Jul 2011, Jon Hunter wrote: From: Jon Hunterjon-hun...@ti.com The parent clock of the OCP_ABE_ICLK is the AESS_FCLK and the parent clock of the AESS_FCLK is the ABE_FCLK... ABE_FCLK -- AESS_FCLK --

[PATCH 1/6] OMAP4: Add missing clock divider for OCP_ABE_ICLK

2011-07-14 Thread Jon Hunter
From: Jon Hunter jon-hun...@ti.com The parent clock of the OCP_ABE_ICLK is the AESS_FCLK and the parent clock of the AESS_FCLK is the ABE_FCLK... ABE_FCLK -- AESS_FCLK -- OCP_ABE_ICLK The AESS_FCLK and OCP_ABE_ICLK clocks both have dividers which determine their operational frequency. However,