Re: [U-Boot] [PATCH] mpc85xx/t102xqds: convert deep sleep to generic board interface

2015-01-22 Thread York Sun


On 12/17/2014 07:55 PM, Tang Yuantian wrote:
 A new deep sleep interface is introduced to support generic
 board structure. Converts it to use new interface.
 
 Signed-off-by: Tang Yuantian yuantian.t...@freescale.com
 ---

Applied to u-boot-mpc85xx master branch, awaiting upstream.

York

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] mpc85xx/t102xqds: convert deep sleep to generic board interface

2015-01-18 Thread Yuantian Tang
Hi York,

 -Original Message-
 From: Sun York-R58495
 Sent: Saturday, January 17, 2015 1:23 AM
 To: Tang Yuantian-B29983
 Cc: u-boot@lists.denx.de
 Subject: Re: [PATCH] mpc85xx/t102xqds: convert deep sleep to generic board
 interface
 
 Yuantian,
 
 On 01/15/2015 06:23 PM, Tang Yuantian-B29983 wrote:
  Hi York,
 
  Yes, I did it on purpose. T102XQDS is not supported in SDK1.7 anymore.
  So I don't want to put more efforts to add SPI/NAND/SD boot deep sleep
 support.
 
 That's not a good reason. Regardless if Freescale SDK supports a board, as 
 far as
 it benefits the community, we keep the support.
 
  That means t102xqds only supports nor-boot deep sleep and need not to
 update spl.c.
 
  It is the same case for T1040QDS.
 
 A note in readme would be nice. I am not going to push for a fix. Not having 
 the
 console silenced is no big deal in case someone uses SPL with deep sleep.

I am not sure I understand you right, but Non-nor deep sleep support is more 
than just disabling console.
We need to update board head file, like t102xqds.h, as well.
I think you are right. We should add non-nor boot support.

Please apply these patches. I will send another two patch to support QDS 
non-nor boot deep sleep on qds.

Regards,
Yuantian

 
 York
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] mpc85xx/t102xqds: convert deep sleep to generic board interface

2015-01-16 Thread York Sun
Yuantian,

On 01/15/2015 06:23 PM, Tang Yuantian-B29983 wrote:
 Hi York,
 
 Yes, I did it on purpose. T102XQDS is not supported in SDK1.7 anymore.
 So I don't want to put more efforts to add SPI/NAND/SD boot deep sleep 
 support.

That's not a good reason. Regardless if Freescale SDK supports a board, as far
as it benefits the community, we keep the support.

 That means t102xqds only supports nor-boot deep sleep and need not to update 
 spl.c. 
 
 It is the same case for T1040QDS.

A note in readme would be nice. I am not going to push for a fix. Not having the
console silenced is no big deal in case someone uses SPL with deep sleep.

York
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] mpc85xx/t102xqds: convert deep sleep to generic board interface

2015-01-15 Thread York Sun
Yuantian,

On 12/17/2014 05:55 PM, Tang Yuantian wrote:
 A new deep sleep interface is introduced to support generic
 board structure. Converts it to use new interface.
 
 Signed-off-by: Tang Yuantian yuantian.t...@freescale.com
 ---
  board/freescale/t102xqds/ddr.c  | 19 +++
  board/freescale/t102xqds/t102xqds.c | 23 +++
  include/configs/T102xQDS.h  |  3 +++
  3 files changed, 33 insertions(+), 12 deletions(-)

You disable console in spl.c for RDB, but not for QDS. Is that what you want?

York

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] mpc85xx/t102xqds: convert deep sleep to generic board interface

2015-01-15 Thread Yuantian Tang
Hi York,

Yes, I did it on purpose. T102XQDS is not supported in SDK1.7 anymore.
So I don't want to put more efforts to add SPI/NAND/SD boot deep sleep support.
That means t102xqds only supports nor-boot deep sleep and need not to update 
spl.c. 

It is the same case for T1040QDS.

Thanks,
Yuantian

 -Original Message-
 From: Sun York-R58495
 Sent: Friday, January 16, 2015 2:02 AM
 To: Tang Yuantian-B29983
 Cc: u-boot@lists.denx.de
 Subject: Re: [PATCH] mpc85xx/t102xqds: convert deep sleep to generic board
 interface
 
 Yuantian,
 
 On 12/17/2014 05:55 PM, Tang Yuantian wrote:
  A new deep sleep interface is introduced to support generic board
  structure. Converts it to use new interface.
 
  Signed-off-by: Tang Yuantian yuantian.t...@freescale.com
  ---
   board/freescale/t102xqds/ddr.c  | 19 +++
   board/freescale/t102xqds/t102xqds.c | 23 +++
   include/configs/T102xQDS.h  |  3 +++
   3 files changed, 33 insertions(+), 12 deletions(-)
 
 You disable console in spl.c for RDB, but not for QDS. Is that what you want?
 
 York

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] mpc85xx/t102xqds: convert deep sleep to generic board interface

2014-12-17 Thread Tang Yuantian
A new deep sleep interface is introduced to support generic
board structure. Converts it to use new interface.

Signed-off-by: Tang Yuantian yuantian.t...@freescale.com
---
 board/freescale/t102xqds/ddr.c  | 19 +++
 board/freescale/t102xqds/t102xqds.c | 23 +++
 include/configs/T102xQDS.h  |  3 +++
 3 files changed, 33 insertions(+), 12 deletions(-)

diff --git a/board/freescale/t102xqds/ddr.c b/board/freescale/t102xqds/ddr.c
index 46fc64e..2d4d10f 100644
--- a/board/freescale/t102xqds/ddr.c
+++ b/board/freescale/t102xqds/ddr.c
@@ -11,6 +11,7 @@
 #include fsl_ddr_sdram.h
 #include fsl_ddr_dimm_params.h
 #include asm/fsl_law.h
+#include asm/mpc85xx_gpio.h
 
 DECLARE_GLOBAL_DATA_PTR;
 
@@ -152,6 +153,19 @@ found:
 #endif
 }
 
+#if defined(CONFIG_DEEP_SLEEP)
+void board_mem_sleep_setup(void)
+{
+   void __iomem *qixis_base = (void *)QIXIS_BASE;
+
+   /* does not provide HW signals for power management */
+   clrbits_8(qixis_base + 0x21, 0x2);
+   /* Disable MCKE isolation */
+   gpio_set_value(2, 0);
+   udelay(1);
+}
+#endif
+
 phys_size_t initdram(int board_type)
 {
phys_size_t dram_size;
@@ -166,5 +180,10 @@ phys_size_t initdram(int board_type)
/* DDR has been initialised by first stage boot loader */
dram_size =  fsl_ddr_sdram_size();
 #endif
+
+#if defined(CONFIG_DEEP_SLEEP)  !defined(CONFIG_SPL_BUILD)
+   fsl_dp_resume();
+#endif
+
return dram_size;
 }
diff --git a/board/freescale/t102xqds/t102xqds.c 
b/board/freescale/t102xqds/t102xqds.c
index f3141b5..708afca 100644
--- a/board/freescale/t102xqds/t102xqds.c
+++ b/board/freescale/t102xqds/t102xqds.c
@@ -19,10 +19,10 @@
 #include asm/fsl_liodn.h
 #include fm_eth.h
 #include hwconfig.h
-#include asm/mpc85xx_gpio.h
 #include ../common/qixis.h
 #include t102xqds.h
 #include t102xqds_qixis.h
+#include ../common/sleep.h
 
 DECLARE_GLOBAL_DATA_PTR;
 
@@ -242,6 +242,16 @@ void board_retimer_ds125df111_init(void)
i2c_write(I2C_RETIMER_ADDR, 0x64, 1, reg, 1);
 }
 
+int board_early_init_f(void)
+{
+#if defined(CONFIG_DEEP_SLEEP)
+   if (is_warm_boot())
+   fsl_dp_disable_console();
+#endif
+
+   return 0;
+}
+
 int board_early_init_r(void)
 {
 #ifdef CONFIG_SYS_FLASH_BASE
@@ -395,14 +405,3 @@ void qixis_dump_switch(void)
printf(SW%d = (0x%02x)\n, i, QIXIS_READ(cms[1]));
}
 }
-
-#ifdef CONFIG_DEEP_SLEEP
-void board_mem_sleep_setup(void)
-{
-   /* does not provide HW signals for power management */
-   QIXIS_WRITE(pwr_ctl[1], (QIXIS_READ(pwr_ctl[1])  ~0x2));
-   /* Disable MCKE isolation */
-   gpio_set_value(2, 0);
-   udelay(1);
-}
-#endif
diff --git a/include/configs/T102xQDS.h b/include/configs/T102xQDS.h
index c2bdbb9..3f02ced 100644
--- a/include/configs/T102xQDS.h
+++ b/include/configs/T102xQDS.h
@@ -35,7 +35,10 @@
 #define CONFIG_ENV_OVERWRITE
 
 #define CONFIG_DEEP_SLEEP
+#if defined(CONFIG_DEEP_SLEEP)
 #define CONFIG_SILENT_CONSOLE
+#define CONFIG_BOARD_EARLY_INIT_F
+#endif
 
 #ifdef CONFIG_RAMBOOT_PBL
 #define CONFIG_SYS_FSL_PBL_PBI board/freescale/t102xqds/t1024_pbi.cfg
-- 
2.1.0.27.g96db324

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot