
8 May
2013
8 May
'13
1:18 a.m.
On 05/07/2013 04:04 PM, Andy Fleming wrote:
On Mon, Mar 25, 2013 at 12:33 PM, York Sun <yorksun@freescale.com mailto:yorksun@freescale.com> wrote:
From: Shaohui Xie <Shaohui.Xie@freescale.com <mailto:Shaohui.Xie@freescale.com>> When CONFIG_SYS_FSL_QORIQ_CHASSIS2 is not defined, QMAN frequency will not be initialized, and QMAN will have a wrong frequency display. Signed-off-by: Shaohui Xie <Shaohui.Xie@freescale.com <mailto:Shaohui.Xie@freescale.com>> --- arch/powerpc/cpu/mpc85xx/speed.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/arch/powerpc/cpu/mpc85xx/speed.c b/arch/powerpc/cpu/mpc85xx/speed.c index 9fc7b54..f00b1ab 100644 --- a/arch/powerpc/cpu/mpc85xx/speed.c +++ b/arch/powerpc/cpu/mpc85xx/speed.c @@ -293,6 +293,10 @@ void get_sys_info (sys_info_t * sysInfo) #endif #endif +#ifdef CONFIG_SYS_DPAA_QBMAN + sysInfo->freqQMAN = sysInfo->freqSystemBus / 2; +#endif +
Can we just move the original copy of the above lines out of the #ifdef? I don't see any reason to do it the same way in two places.
You mean delete #ifdef? It will have a compiling error, won't it? See include/e500.h, the freqQMAN is withing #ifdef.
York