[U-Boot] [PATCH 5/5] 85xx: enable the auto self refresh for wake up ARP

The wake up ARP feature need use the memory to process wake up packet, we enable auto self refresh to support it.
Signed-off-by: Dave Liu daveliu@freescale.com --- board/freescale/mpc8536ds/ddr.c | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-)
diff --git a/board/freescale/mpc8536ds/ddr.c b/board/freescale/mpc8536ds/ddr.c index 3135d6d..2bad787 100644 --- a/board/freescale/mpc8536ds/ddr.c +++ b/board/freescale/mpc8536ds/ddr.c @@ -79,4 +79,10 @@ void fsl_ddr_board_options(memctl_options_t *popts, * - number of DIMMs installed */ popts->half_strength_driver_enable = 0; + + /* + * For wake up arp feature, we need enable auto self refresh + */ + popts->auto_self_refresh_en = 1; + popts->sr_it = 0x6; }

On Nov 21, 2008, at 2:31 AM, Dave Liu wrote:
The wake up ARP feature need use the memory to process wake up packet, we enable auto self refresh to support it.
Signed-off-by: Dave Liu daveliu@freescale.com
board/freescale/mpc8536ds/ddr.c | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-)
diff --git a/board/freescale/mpc8536ds/ddr.c b/board/freescale/ mpc8536ds/ddr.c index 3135d6d..2bad787 100644 --- a/board/freescale/mpc8536ds/ddr.c +++ b/board/freescale/mpc8536ds/ddr.c @@ -79,4 +79,10 @@ void fsl_ddr_board_options(memctl_options_t *popts, * - number of DIMMs installed */ popts->half_strength_driver_enable = 0;
- /*
* For wake up arp feature, we need enable auto self refresh
*/
- popts->auto_self_refresh_en = 1;
- popts->sr_it = 0x6;
any reason we picked 6?
- k

-----Original Message----- From: Kumar Gala [mailto:galak@kernel.crashing.org] Sent: Friday, November 21, 2008 7:43 PM To: Liu Dave-R63238 Cc: u-boot@lists.denx.de Subject: Re: [U-Boot] [PATCH 5/5] 85xx: enable the auto self refresh for wake up ARP
On Nov 21, 2008, at 2:31 AM, Dave Liu wrote:
The wake up ARP feature need use the memory to process wake up packet, we enable auto self refresh to support it.
Signed-off-by: Dave Liu daveliu@freescale.com
board/freescale/mpc8536ds/ddr.c | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-)
diff --git a/board/freescale/mpc8536ds/ddr.c b/board/freescale/ mpc8536ds/ddr.c index 3135d6d..2bad787 100644 --- a/board/freescale/mpc8536ds/ddr.c +++ b/board/freescale/mpc8536ds/ddr.c @@ -79,4 +79,10 @@ void
fsl_ddr_board_options(memctl_options_t *popts,
* - number of DIMMs installed */
popts->half_strength_driver_enable = 0;
- /*
* For wake up arp feature, we need enable auto self refresh
*/
- popts->auto_self_refresh_en = 1;
- popts->sr_it = 0x6;
any reason we picked 6?
That is balance for performance and respond time. the 6 stand for 3ms time (3ns MCLK)

On Nov 21, 2008, at 6:20 AM, Liu Dave wrote:
-----Original Message----- From: Kumar Gala [mailto:galak@kernel.crashing.org] Sent: Friday, November 21, 2008 7:43 PM To: Liu Dave-R63238 Cc: u-boot@lists.denx.de Subject: Re: [U-Boot] [PATCH 5/5] 85xx: enable the auto self refresh for wake up ARP
On Nov 21, 2008, at 2:31 AM, Dave Liu wrote:
The wake up ARP feature need use the memory to process wake up packet, we enable auto self refresh to support it.
Signed-off-by: Dave Liu daveliu@freescale.com
board/freescale/mpc8536ds/ddr.c | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-)
diff --git a/board/freescale/mpc8536ds/ddr.c b/board/freescale/ mpc8536ds/ddr.c index 3135d6d..2bad787 100644 --- a/board/freescale/mpc8536ds/ddr.c +++ b/board/freescale/mpc8536ds/ddr.c @@ -79,4 +79,10 @@ void
fsl_ddr_board_options(memctl_options_t *popts,
* - number of DIMMs installed */
popts->half_strength_driver_enable = 0;
- /*
* For wake up arp feature, we need enable auto self refresh
*/
- popts->auto_self_refresh_en = 1;
- popts->sr_it = 0x6;
any reason we picked 6?
That is balance for performance and respond time. the 6 stand for 3ms time (3ns MCLK)
Probably good to say that in the commit message or as a comment in the code.
- k

On Fri, Nov 21, 2008 at 2:31 AM, Dave Liu daveliu@freescale.com wrote:
The wake up ARP feature need use the memory to process wake up packet, we enable auto self refresh to support it.
Signed-off-by: Dave Liu daveliu@freescale.com
Applied to 85xx-next, thanks
participants (4)
-
Andy Fleming
-
Dave Liu
-
Kumar Gala
-
Liu Dave