From mboxrd@z Thu Jan  1 00:00:00 1970
From: York Sun <yorksun@freescale.com>
Date: Tue, 7 May 2013 16:18:21 -0700
Subject: [U-Boot] [Patch v2,
 batch 2 17/23] powerpc/85xx: add missing QMAN frequency calculation
In-Reply-To: <CAKWjMd5t4rE=rvxiETDQ-4deBRt-sc1Mf1wXPh8t_KmL0D2Log@mail.gmail.com>
References: <1364232811-30856-1-git-send-email-yorksun@freescale.com>
	<1364232811-30856-17-git-send-email-yorksun@freescale.com>
	<CAKWjMd5t4rE=rvxiETDQ-4deBRt-sc1Mf1wXPh8t_KmL0D2Log@mail.gmail.com>
Message-ID: <51898BBD.5010906@freescale.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

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