All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH 1/2] Revert "imx: mx6: ddr correct tRFC and tXS"
@ 2015-09-01  3:03 Peng Fan
  2015-09-01  3:03 ` [U-Boot] [PATCH 2/2] imx: discard duplicated MXC_OCOTP and CMD_FUSE Peng Fan
  2015-09-13  8:27 ` [U-Boot] [PATCH 1/2] Revert "imx: mx6: ddr correct tRFC and tXS" Stefano Babic
  0 siblings, 2 replies; 4+ messages in thread
From: Peng Fan @ 2015-09-01  3:03 UTC (permalink / raw)
  To: u-boot

This reverts commit 059323fb6a8f21637bb617919715c2427f24777c.

This commit 059323fb6a8f21637bb617919715c2427f24777c use JESD79-3E which
is not the newest spec. Should use JESD79-3F in which tRFC is 260ns for
4Gb chip.

Signed-off-by: Peng Fan <Peng.Fan@freescale.com>
Cc: Stefano Babic <sbabic@denx.de>
Cc: Tim Harvey <tharvey@gateworks.com>
---
 arch/arm/cpu/armv7/mx6/ddr.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/cpu/armv7/mx6/ddr.c b/arch/arm/cpu/armv7/mx6/ddr.c
index cf5587b..6b039e4 100644
--- a/arch/arm/cpu/armv7/mx6/ddr.c
+++ b/arch/arm/cpu/armv7/mx6/ddr.c
@@ -702,8 +702,8 @@ void mx6_ddr3_cfg(const struct mx6_ddr_sysinfo *sysinfo,
 		txs = DIV_ROUND_UP(170000, clkper) - 1;
 		break;
 	case 4: /* 4Gb per chip */
-		trfc = DIV_ROUND_UP(300000, clkper) - 1;
-		txs = DIV_ROUND_UP(310000, clkper) - 1;
+		trfc = DIV_ROUND_UP(260000, clkper) - 1;
+		txs = DIV_ROUND_UP(270000, clkper) - 1;
 		break;
 	case 8: /* 8Gb per chip */
 		trfc = DIV_ROUND_UP(350000, clkper) - 1;
-- 
1.8.4

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH 2/2] imx: discard duplicated MXC_OCOTP and CMD_FUSE
  2015-09-01  3:03 [U-Boot] [PATCH 1/2] Revert "imx: mx6: ddr correct tRFC and tXS" Peng Fan
@ 2015-09-01  3:03 ` Peng Fan
  2015-09-13  8:31   ` Stefano Babic
  2015-09-13  8:27 ` [U-Boot] [PATCH 1/2] Revert "imx: mx6: ddr correct tRFC and tXS" Stefano Babic
  1 sibling, 1 reply; 4+ messages in thread
From: Peng Fan @ 2015-09-01  3:03 UTC (permalink / raw)
  To: u-boot

We have CONFIG_MXC_OCOTP and CONFIG_CMD_FUSE in mx6_common.h,
discard duplicated macro definitions in board header files.

Signed-off-by: Peng Fan <Peng.Fan@freescale.com>
Cc: Stefano Babic <sbabic@denx.de>
Cc: Peter Robinson <pbrobinson@gmail.com>
Cc: Otavio Salvador <otavio@ossystems.com.br>
---
 include/configs/cgtqmx6eval.h     | 5 -----
 include/configs/mx6ul_14x14_evk.h | 5 -----
 2 files changed, 10 deletions(-)

diff --git a/include/configs/cgtqmx6eval.h b/include/configs/cgtqmx6eval.h
index 92930c8..0979728 100644
--- a/include/configs/cgtqmx6eval.h
+++ b/include/configs/cgtqmx6eval.h
@@ -35,11 +35,6 @@
 /* Thermal support */
 #define CONFIG_IMX6_THERMAL
 
-#define CONFIG_CMD_FUSE
-#if defined(CONFIG_CMD_FUSE) || defined(CONFIG_IMX6_THERMAL)
-#define CONFIG_MXC_OCOTP
-#endif
-
 /* I2C Configs */
 #define CONFIG_CMD_I2C
 #define CONFIG_SYS_I2C
diff --git a/include/configs/mx6ul_14x14_evk.h b/include/configs/mx6ul_14x14_evk.h
index f643e4d..89e6028 100644
--- a/include/configs/mx6ul_14x14_evk.h
+++ b/include/configs/mx6ul_14x14_evk.h
@@ -34,11 +34,6 @@
 #define CONFIG_MXC_UART
 #define CONFIG_MXC_UART_BASE		UART1_BASE
 
-#define CONFIG_CMD_FUSE
-#ifdef CONFIG_CMD_FUSE
-#define CONFIG_MXC_OCOTP
-#endif
-
 /* MMC Configs */
 #ifdef CONFIG_FSL_USDHC
 #define CONFIG_SYS_FSL_ESDHC_ADDR	USDHC2_BASE_ADDR
-- 
1.8.4

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH 1/2] Revert "imx: mx6: ddr correct tRFC and tXS"
  2015-09-01  3:03 [U-Boot] [PATCH 1/2] Revert "imx: mx6: ddr correct tRFC and tXS" Peng Fan
  2015-09-01  3:03 ` [U-Boot] [PATCH 2/2] imx: discard duplicated MXC_OCOTP and CMD_FUSE Peng Fan
@ 2015-09-13  8:27 ` Stefano Babic
  1 sibling, 0 replies; 4+ messages in thread
From: Stefano Babic @ 2015-09-13  8:27 UTC (permalink / raw)
  To: u-boot



On 01/09/2015 05:03, Peng Fan wrote:
> This reverts commit 059323fb6a8f21637bb617919715c2427f24777c.
> 
> This commit 059323fb6a8f21637bb617919715c2427f24777c use JESD79-3E which
> is not the newest spec. Should use JESD79-3F in which tRFC is 260ns for
> 4Gb chip.
> 
> Signed-off-by: Peng Fan <Peng.Fan@freescale.com>
> Cc: Stefano Babic <sbabic@denx.de>
> Cc: Tim Harvey <tharvey@gateworks.com>
> ---
>  arch/arm/cpu/armv7/mx6/ddr.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/cpu/armv7/mx6/ddr.c b/arch/arm/cpu/armv7/mx6/ddr.c
> index cf5587b..6b039e4 100644
> --- a/arch/arm/cpu/armv7/mx6/ddr.c
> +++ b/arch/arm/cpu/armv7/mx6/ddr.c
> @@ -702,8 +702,8 @@ void mx6_ddr3_cfg(const struct mx6_ddr_sysinfo *sysinfo,
>  		txs = DIV_ROUND_UP(170000, clkper) - 1;
>  		break;
>  	case 4: /* 4Gb per chip */
> -		trfc = DIV_ROUND_UP(300000, clkper) - 1;
> -		txs = DIV_ROUND_UP(310000, clkper) - 1;
> +		trfc = DIV_ROUND_UP(260000, clkper) - 1;
> +		txs = DIV_ROUND_UP(270000, clkper) - 1;
>  		break;
>  	case 8: /* 8Gb per chip */
>  		trfc = DIV_ROUND_UP(350000, clkper) - 1;
> 


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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH 2/2] imx: discard duplicated MXC_OCOTP and CMD_FUSE
  2015-09-01  3:03 ` [U-Boot] [PATCH 2/2] imx: discard duplicated MXC_OCOTP and CMD_FUSE Peng Fan
@ 2015-09-13  8:31   ` Stefano Babic
  0 siblings, 0 replies; 4+ messages in thread
From: Stefano Babic @ 2015-09-13  8:31 UTC (permalink / raw)
  To: u-boot



On 01/09/2015 05:03, Peng Fan wrote:
> We have CONFIG_MXC_OCOTP and CONFIG_CMD_FUSE in mx6_common.h,
> discard duplicated macro definitions in board header files.
> 
> Signed-off-by: Peng Fan <Peng.Fan@freescale.com>
> Cc: Stefano Babic <sbabic@denx.de>
> Cc: Peter Robinson <pbrobinson@gmail.com>
> Cc: Otavio Salvador <otavio@ossystems.com.br>
> ---
>  include/configs/cgtqmx6eval.h     | 5 -----
>  include/configs/mx6ul_14x14_evk.h | 5 -----
>  2 files changed, 10 deletions(-)
> 
> diff --git a/include/configs/cgtqmx6eval.h b/include/configs/cgtqmx6eval.h
> index 92930c8..0979728 100644
> --- a/include/configs/cgtqmx6eval.h
> +++ b/include/configs/cgtqmx6eval.h
> @@ -35,11 +35,6 @@
>  /* Thermal support */
>  #define CONFIG_IMX6_THERMAL
>  
> -#define CONFIG_CMD_FUSE
> -#if defined(CONFIG_CMD_FUSE) || defined(CONFIG_IMX6_THERMAL)
> -#define CONFIG_MXC_OCOTP
> -#endif
> -
>  /* I2C Configs */
>  #define CONFIG_CMD_I2C
>  #define CONFIG_SYS_I2C
> diff --git a/include/configs/mx6ul_14x14_evk.h b/include/configs/mx6ul_14x14_evk.h
> index f643e4d..89e6028 100644
> --- a/include/configs/mx6ul_14x14_evk.h
> +++ b/include/configs/mx6ul_14x14_evk.h
> @@ -34,11 +34,6 @@
>  #define CONFIG_MXC_UART
>  #define CONFIG_MXC_UART_BASE		UART1_BASE
>  
> -#define CONFIG_CMD_FUSE
> -#ifdef CONFIG_CMD_FUSE
> -#define CONFIG_MXC_OCOTP
> -#endif
> -
>  /* MMC Configs */
>  #ifdef CONFIG_FSL_USDHC
>  #define CONFIG_SYS_FSL_ESDHC_ADDR	USDHC2_BASE_ADDR
> 

for the 6ul part:

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2015-09-13  8:31 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-09-01  3:03 [U-Boot] [PATCH 1/2] Revert "imx: mx6: ddr correct tRFC and tXS" Peng Fan
2015-09-01  3:03 ` [U-Boot] [PATCH 2/2] imx: discard duplicated MXC_OCOTP and CMD_FUSE Peng Fan
2015-09-13  8:31   ` Stefano Babic
2015-09-13  8:27 ` [U-Boot] [PATCH 1/2] Revert "imx: mx6: ddr correct tRFC and tXS" Stefano Babic

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.