From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stefano Babic <sbabic@denx.de>
Date: Wed, 13 Jul 2016 16:18:24 +0200
Subject: [U-Boot] [PATCH] mx6: support i.MX6UL speed grading reading
 from OTP bits
In-Reply-To: <1466432092-3289-1-git-send-email-hector.palacios@digi.com>
References: <1466432092-3289-1-git-send-email-hector.palacios@digi.com>
Message-ID: <57864DB0.2010706@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

Hi Hector,

On 20/06/2016 16:14, Hector Palacios wrote:
> i.MX6UL defines speed grading in OCOTP register 0x440[17:16]
> as follows:
>     00      reserved
>     01      528MHz
>     10      700MHz
>     11      reserved
> 
> This commit removes the constants (which had the speed hardcoded
> in their names) and uses values instead.
> 
> Signed-off-by: Hector Palacios <hector.palacios@digi.com>
> ---
>  arch/arm/cpu/armv7/mx6/soc.c | 23 ++++++++++++-----------
>  1 file changed, 12 insertions(+), 11 deletions(-)
> 
> diff --git a/arch/arm/cpu/armv7/mx6/soc.c b/arch/arm/cpu/armv7/mx6/soc.c
> index d4b22ad7f315..49b5773b1b77 100644
> --- a/arch/arm/cpu/armv7/mx6/soc.c
> +++ b/arch/arm/cpu/armv7/mx6/soc.c
> @@ -103,10 +103,6 @@ u32 get_cpu_rev(void)
>   * defines a 2-bit SPEED_GRADING
>   */
>  #define OCOTP_CFG3_SPEED_SHIFT	16
> -#define OCOTP_CFG3_SPEED_800MHZ	0
> -#define OCOTP_CFG3_SPEED_850MHZ	1
> -#define OCOTP_CFG3_SPEED_1GHZ	2
> -#define OCOTP_CFG3_SPEED_1P2GHZ	3
>  
>  u32 get_cpu_speed_grade_hz(void)
>  {
> @@ -122,18 +118,23 @@ u32 get_cpu_speed_grade_hz(void)
>  
>  	switch (val) {
>  	/* Valid for IMX6DQ */
> -	case OCOTP_CFG3_SPEED_1P2GHZ:
> +	case 3:
>  		if (is_cpu_type(MXC_CPU_MX6Q) || is_cpu_type(MXC_CPU_MX6D))
>  			return 1200000000;
> -	/* Valid for IMX6SX/IMX6SDL/IMX6DQ */
> -	case OCOTP_CFG3_SPEED_1GHZ:
> -		return 996000000;
> -	/* Valid for IMX6DQ */
> -	case OCOTP_CFG3_SPEED_850MHZ:
> +	/* Valid for IMX6SX/IMX6SDL/IMX6DQ/IMX6UL */
> +	case 2:
> +		if (is_cpu_type(MXC_CPU_MX6UL))
> +			return 700000000;
> +		else
> +			return 996000000;
> +	/* Valid for IMX6DQ/IMX6UL */
> +	case 1:
>  		if (is_cpu_type(MXC_CPU_MX6Q) || is_cpu_type(MXC_CPU_MX6D))
>  			return 852000000;
> +		else if (is_cpu_type(MXC_CPU_MX6UL))
> +			return 528000000;
>  	/* Valid for IMX6SX/IMX6SDL/IMX6DQ */
> -	case OCOTP_CFG3_SPEED_800MHZ:
> +	case 0:
>  		return 792000000;
>  	}
>  	return 0;
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
> 

Applied to -u-boot-imx, thanks !

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
=====================================================================