public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stefano Babic <sbabic@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH V2] imx: vf610 add get_cpu_rev
Date: Tue, 1 Sep 2015 16:35:14 +0200	[thread overview]
Message-ID: <55E5B7A2.8060400@denx.de> (raw)
In-Reply-To: <1441098903-5917-1-git-send-email-Peng.Fan@freescale.com>

Hi Peng,

On 01/09/2015 11:15, Peng Fan wrote:
> Since we need to support runtime check for different drivers, we need
> to add get_cpu_rev for vf610, otherwise there will be build errors.
> 
> This patch introduces a dummy CPU id which is not read from chip
> silicon. Later when we can get the real id from chip, can fix the
> value of MXC_CPU_VF610 then.
> 
> Signed-off-by: Peng Fan <Peng.Fan@freescale.com>
> Suggested-by: Stefano Babic <sbabic@denx.de>
> Cc: Stefano Babic <sbabic@denx.de>
> ---
> 
> Changes v2:
>  Take suggestion from Stefano, get_cpu_rev for vf610 should return an 
>  ID saying dummy ID, but not 0.
> 
>  arch/arm/cpu/armv7/vf610/generic.c  | 6 ++++++
>  arch/arm/include/asm/arch-imx/cpu.h | 1 +
>  2 files changed, 7 insertions(+)
> 
> diff --git a/arch/arm/cpu/armv7/vf610/generic.c b/arch/arm/cpu/armv7/vf610/generic.c
> index 05c401d..08b9ef4 100644
> --- a/arch/arm/cpu/armv7/vf610/generic.c
> +++ b/arch/arm/cpu/armv7/vf610/generic.c
> @@ -9,6 +9,7 @@
>  #include <asm/arch/imx-regs.h>
>  #include <asm/arch/clock.h>
>  #include <asm/arch/crm_regs.h>
> +#include <asm/imx-common/sys_proto.h>
>  #include <netdev.h>
>  #ifdef CONFIG_FSL_ESDHC
>  #include <fsl_esdhc.h>
> @@ -266,6 +267,11 @@ void imx_get_mac_from_fuse(int dev_id, unsigned char *mac)
>  }
>  #endif
>  
> +u32 get_cpu_rev(void)
> +{
> +	return MXC_CPU_VF610 << 12;
> +}
> +
>  #if defined(CONFIG_DISPLAY_CPUINFO)
>  static char *get_reset_cause(void)
>  {
> diff --git a/arch/arm/include/asm/arch-imx/cpu.h b/arch/arm/include/asm/arch-imx/cpu.h
> index be13b96..1c8d24e 100644
> --- a/arch/arm/include/asm/arch-imx/cpu.h
> +++ b/arch/arm/include/asm/arch-imx/cpu.h
> @@ -21,6 +21,7 @@
>  #define MXC_CPU_MX6D		0x67
>  #define MXC_CPU_MX6DP		0x68
>  #define MXC_CPU_MX6QP		0x69
> +#define MXC_CPU_VF610		0xF6 /* dummy ID */
>  
>  #define CS0_128					0
>  #define CS0_64M_CS1_64M				1
> 

Acked-by: Stefano Babic <sbabic@denx.de>

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================

      reply	other threads:[~2015-09-01 14:35 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-09-01  9:15 [U-Boot] [PATCH V2] imx: vf610 add get_cpu_rev Peng Fan
2015-09-01 14:35 ` Stefano Babic [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=55E5B7A2.8060400@denx.de \
    --to=sbabic@denx.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox