[U-Boot] [PATCH] armv8: configs: ls1012a: correct the generic timer frequency issue
Prabhakar Kushwaha
prabhakar.kushwaha at nxp.com
Tue Oct 10 04:42:22 UTC 2017
Hi York,
> -----Original Message-----
> From: U-Boot [mailto:u-boot-bounces at lists.denx.de] On Behalf Of Andy Tang
> Sent: Tuesday, October 10, 2017 6:36 AM
> To: York Sun <york.sun at nxp.com>
> Cc: u-boot at lists.denx.de
> Subject: Re: [U-Boot] [PATCH] armv8: configs: ls1012a: correct the generic timer
> frequency issue
>
> Hi York,
>
> Ls1012a uses the fixed 25Mhz clock, I will update the commit message and
> resend the patch.
>
> Thanks,
> Andy
>
> > -----Original Message-----
> > From: York Sun
> > Sent: Tuesday, October 10, 2017 12:00 AM
> > To: Andy Tang <andy.tang at nxp.com>
> > Cc: sjg at chromium.org; u-boot at lists.denx.de
> > Subject: Re: [PATCH] armv8: configs: ls1012a: correct the generic timer
> > frequency issue
> >
> > On 10/08/2017 11:48 PM, andy.tang at nxp.com wrote:
> > > From: Yuantian Tang <andy.tang at nxp.com>
> > >
> > > Generic Timer frequency should be 25Mhz. Current setting is
> > > CONFIG_SYS_CLK_FREQ/4 which is about 31Mhz, which is not correct.
> > > So correct it.
> > >
> > > Signed-off-by: Tang Yuantian <andy.tang at nxp.com>
> > > ---
> > > include/configs/ls1012a_common.h | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/include/configs/ls1012a_common.h
> > > b/include/configs/ls1012a_common.h
> > > index 096799eb64..a4e78f335f 100644
> > > --- a/include/configs/ls1012a_common.h
> > > +++ b/include/configs/ls1012a_common.h
> > > @@ -32,7 +32,7 @@
> > > #define CONFIG_SYS_DDR_BLOCK2_BASE 0x880000000ULL
> > >
> > > /* Generic Timer Definitions */
> > > -#define COUNTER_FREQUENCY CONFIG_SYS_CLK_FREQ/4
> > /* 25MHz */
> > > +#define COUNTER_FREQUENCY 25000000 /* 25MHz */
> > >
> >
> >
> > Yuantian,
> >
> > LS1012A use fixed 25MHz clock, doesn't it? If so, that's the reason a fixed
> > value should be used, not because CONFIG_SYS_CLK_FREQ/4 isn't correct. It
> > is correct for many other platform. Please update the commit message.
> >
LS1012A has fixed external clock 25MHz which generates following 2 internal clocks
- Core clock (100MHz)
- Platform clock (133MHz)
System counter run with Core clock i.e. 100MHz.
So patch description should mention reference of Core Clock instead of external 25MHz fix clock.
--pk
More information about the U-Boot
mailing list