From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marex@denx.de>
Date: Thu, 19 Apr 2018 04:59:00 +0200
Subject: [U-Boot] [PATCH v1 13/16] arm: socfpga: stratix10: Add timer
 support for Stratix10 SoC
In-Reply-To: <1524131457-19234-14-git-send-email-ley.foon.tan@intel.com>
References: <1524131457-19234-1-git-send-email-ley.foon.tan@intel.com>
 <1524131457-19234-14-git-send-email-ley.foon.tan@intel.com>
Message-ID: <348cf511-cdfd-bb02-9c44-5a166e75ea38@denx.de>
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 04/19/2018 11:50 AM, Ley Foon Tan wrote:
> Add timer support for Stratix SoC

Is this really custom timer or is that some armv8 thing you're adding
here ? Don't we already have a generic implementation for that ? If not,
that's what we should do here.

> Signed-off-by: Chin Liang See <chin.liang.see@intel.com>
> Signed-off-by: Ley Foon Tan <ley.foon.tan@intel.com>
> ---
>  arch/arm/mach-socfpga/timer.c |   17 ++++++++++++++++-
>  1 files changed, 16 insertions(+), 1 deletions(-)
> 
> diff --git a/arch/arm/mach-socfpga/timer.c b/arch/arm/mach-socfpga/timer.c
> index 253cde3..4c90b57 100644
> --- a/arch/arm/mach-socfpga/timer.c
> +++ b/arch/arm/mach-socfpga/timer.c
> @@ -1,5 +1,6 @@
>  /*
> - *  Copyright (C) 2012 Altera Corporation <www.altera.com>
> + * Copyright (C) 2016-2018 Intel Corporation <www.intel.com>
> + * Copyright (C) 2012-2016 Altera Corporation <www.altera.com>
>   *
>   * SPDX-License-Identifier:	GPL-2.0+
>   */
> @@ -10,15 +11,29 @@
>  
>  #define TIMER_LOAD_VAL		0xFFFFFFFF
>  
> +#if !defined(CONFIG_TARGET_SOCFPGA_STRATIX10)
>  static const struct socfpga_timer *timer_base = (void *)CONFIG_SYS_TIMERBASE;
> +#endif
>  
>  /*
>   * Timer initialization
>   */
>  int timer_init(void)
>  {
> +#if defined(CONFIG_TARGET_SOCFPGA_STRATIX10)
> +	int enable = 0x3;	/* timer enable + output signal masked */
> +	int loadval = ~0;
> +
> +	/* enable system counter */
> +	writel(enable, SOCFPGA_GTIMER_SEC_ADDRESS);
> +	/* enable processor pysical counter */
> +	asm volatile("msr cntp_ctl_el0, %0" : : "r" (enable));
> +	asm volatile("msr cntp_tval_el0, %0" : : "r" (loadval));
> +
> +#else
>  	writel(TIMER_LOAD_VAL, &timer_base->load_val);
>  	writel(TIMER_LOAD_VAL, &timer_base->curr_val);
>  	writel(readl(&timer_base->ctrl) | 0x3, &timer_base->ctrl);
> +#endif
>  	return 0;
>  }
> 


-- 
Best regards,
Marek Vasut