[U-Boot] [Patch v2, batch 2 17/23] powerpc/85xx: add missing QMAN frequency calculation
Fleming Andy-AFLEMING
afleming at freescale.com
Thu May 9 02:40:14 CEST 2013
Ah, ok, makes sense.
On May 7, 2013, at 18:18, "sun york-R58495" <R58495 at freescale.com> wrote:
> On 05/07/2013 04:04 PM, Andy Fleming wrote:
>>
>>
>>
>> On Mon, Mar 25, 2013 at 12:33 PM, York Sun <yorksun at freescale.com
>> <mailto:yorksun at freescale.com>> wrote:
>>
>> From: Shaohui Xie <Shaohui.Xie at freescale.com
>> <mailto:Shaohui.Xie at 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 at freescale.com
>> <mailto:Shaohui.Xie at 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
>
More information about the U-Boot
mailing list