Hi, York, > On 09/23/2016 01:15 AM, Alison Wang wrote: > > GENERIC_TIMER_CLK and CONFIG_TIMER_CLK_FREQ are both used to define > > Generic Timer frequency. It is reduplicate. This patch will remove > > GENERIC_TIMER_CLK macro. > > > > Signed-off-by: Alison Wang <alison.w...@nxp.com> > > --- > > arch/arm/cpu/armv7/ls102xa/psci.S | 2 +- > > arch/arm/cpu/armv7/ls102xa/timer.c | 2 +- > > include/configs/ls1021aqds.h | 5 ----- > > include/configs/ls1021atwr.h | 5 ----- > > 4 files changed, 2 insertions(+), 12 deletions(-) > > > > diff --git a/arch/arm/cpu/armv7/ls102xa/psci.S > > b/arch/arm/cpu/armv7/ls102xa/psci.S > > index 8f38680..9efb6d8 100644 > > --- a/arch/arm/cpu/armv7/ls102xa/psci.S > > +++ b/arch/arm/cpu/armv7/ls102xa/psci.S > > @@ -36,7 +36,7 @@ > > > > .align 5 > > > > -#define ONE_MS (GENERIC_TIMER_CLK / 1000) > > +#define ONE_MS (CONFIG_TIMER_CLK_FREQ / 1000) > > #define RESET_WAIT (30 * ONE_MS) > > > > Alison, > > Can you use GENERIC_TIMER_CLK? Recent change in U-Boot doesn't favor > using CONFIG_* macros. [Alison Wang] If GENERIC_TIMER_CLK is used and CONFIG_TIMER_CLK_FREQ is removed, I need to modify the generic codes which use CONFIG_TIMER_CLK_FREQ in arch/arm/cpu/armv7/nonsec_virt.S. Some other platforms may be affected. So I remove GENERIC_TIMER_CLK in v1.
What's your idea? Best Regards, Alison Wang _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot