[U-Boot] [PATCH 2/3] arm: imx-common: introduce back usec2ticks

Eric Nelson eric at nelint.com
Wed Aug 24 02:30:36 CEST 2016


Hi Gary and Peng,

On 08/23/2016 02:55 PM, Gary Bisson wrote:
> From: Peng Fan <van.freenix at gmail.com>
> 
> This commit "2bb014820c49a63902103bac710bc86b5772e843"
> do some clean up to use the code in lib/time.c.
> But usec2ticks is still being used by security related job ring code.
> Bring back the function to avoid build break when CONFIG_FSL_CAAM
> is defined.
> The computation logic has been changed, using 64-bit variable
> to ease the process, making it work on older (MX5) platforms.
> 
> Signed-off-by: Peng Fan <van.freenix at gmail.com>
> Signed-off-by: Troy Kisky <troy.kisky at boundarydevices.com>
> Signed-off-by: Gary Bisson <gary.bisson at boundarydevices.com>
> ---
>  arch/arm/imx-common/timer.c | 16 ++++++++++++++++
>  1 file changed, 16 insertions(+)
> 
> diff --git a/arch/arm/imx-common/timer.c b/arch/arm/imx-common/timer.c
> index a01590c..a04c7ae 100644
> --- a/arch/arm/imx-common/timer.c
> +++ b/arch/arm/imx-common/timer.c
> @@ -119,3 +119,19 @@ ulong get_tbclk(void)
>  {
>  	return gpt_get_clk();
>  }
> +
> +/*
> + * This function is intended for SHORT delays only.
> + * It will overflow at around 10 seconds @ 400MHz,
> + * or 20 seconds @ 200MHz.
> + */
> +unsigned long usec2ticks(unsigned long _usec)
> +{
> +	unsigned long long usec = _usec;
> +
> +	usec *= get_tbclk();
> +	usec += 999999;
> +	do_div(usec, 1000000);
> +
> +	return usec;
> +}
> 

What about the version in imx-common/syscounter.c?

It seems that only one should be needed.


More information about the U-Boot mailing list