* [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 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
* [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
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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox