Re: [U-Boot] [PATCH] Add chip select region to galaxy5200

2009-08-28 Thread Detlev Zundel
Hi Eric,

in case you didn't notice, you misspelled Grants email
grant.lik...@secretlib.ca, while it should read
grant.lik...@secretlab.ca.  But hey, maybe he's also working on secret
libs that we don't know about *lol*

Cheers
  Detlev

-- 
Alisa Sherer (AMD) suggested that consumer demand will not follow
faster clock speeds. Marketing might help with this problem, she
added.-- 2004/02/20
--
DENX Software Engineering GmbH,  MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: d...@denx.de
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] Add chip select region to galaxy5200

2009-08-28 Thread Eric Millbrandt
On Thu, 27 Aug 2009 17:46:03 -0600, Grant Likely wrote
 On Thu, Aug 27, 2009 at 4:11 PM, Eric
 Millbrandtemillbra...@coldhaus.com wrote:
  Add chip select region for an Epson S1D15313.
 
  Signed-off-by: Eric Millbrandt emillbra...@dekaresearch.com
  ---
   include/configs/galaxy5200.h |    5 +
   1 files changed, 5 insertions(+), 0 deletions(-)
 
  diff --git a/include/configs/galaxy5200.h b/include/configs/galaxy5200.h
  index eb2cae6..54c6b80 100644
  --- a/include/configs/galaxy5200.h
  +++ b/include/configs/galaxy5200.h
  @@ -212,6 +212,11 @@
   #define CONFIG_SYS_CS1_SIZE            0x9
   #define CONFIG_SYS_CS1_CFG             0x0002d900
 
  +/* Chip Select configuration for Epson S1D13513 */
  +#define CONFIG_SYS_CS3_START           0x1000
  +#define CONFIG_SYS_CS3_SIZE            0x103f
  +#define CONFIG_SYS_CS3_CFG             0x3d10
 
 This doesn't look right.  Shouldn't CONFIG_SYS_CS3_SIZE be 0x0040?
 (Compare with SYS_CS1 above)
 
Yes, you are correct.  I must have been thinking of the STOP location instead
of the size of the region.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] Add chip select region to galaxy5200

2009-08-27 Thread Eric Millbrandt
Add chip select region for an Epson S1D15313.

Signed-off-by: Eric Millbrandt emillbra...@dekaresearch.com
---
 include/configs/galaxy5200.h |5 +
 1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/include/configs/galaxy5200.h b/include/configs/galaxy5200.h
index eb2cae6..54c6b80 100644
--- a/include/configs/galaxy5200.h
+++ b/include/configs/galaxy5200.h
@@ -212,6 +212,11 @@
 #define CONFIG_SYS_CS1_SIZE0x9
 #define CONFIG_SYS_CS1_CFG 0x0002d900
 
+/* Chip Select configuration for Epson S1D13513 */
+#define CONFIG_SYS_CS3_START   0x1000
+#define CONFIG_SYS_CS3_SIZE0x103f
+#define CONFIG_SYS_CS3_CFG 0x3d10
+
 /*
  * Ethernet configuration
  */
-- 
1.6.3.1


http://openwebmail.org - eem

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


Re: [U-Boot] [PATCH] Add chip select region to galaxy5200

2009-08-27 Thread Grant Likely
On Thu, Aug 27, 2009 at 4:11 PM, Eric
Millbrandtemillbra...@coldhaus.com wrote:
 Add chip select region for an Epson S1D15313.

 Signed-off-by: Eric Millbrandt emillbra...@dekaresearch.com
 ---
  include/configs/galaxy5200.h |    5 +
  1 files changed, 5 insertions(+), 0 deletions(-)

 diff --git a/include/configs/galaxy5200.h b/include/configs/galaxy5200.h
 index eb2cae6..54c6b80 100644
 --- a/include/configs/galaxy5200.h
 +++ b/include/configs/galaxy5200.h
 @@ -212,6 +212,11 @@
  #define CONFIG_SYS_CS1_SIZE            0x9
  #define CONFIG_SYS_CS1_CFG             0x0002d900

 +/* Chip Select configuration for Epson S1D13513 */
 +#define CONFIG_SYS_CS3_START           0x1000
 +#define CONFIG_SYS_CS3_SIZE            0x103f
 +#define CONFIG_SYS_CS3_CFG             0x3d10

This doesn't look right.  Shouldn't CONFIG_SYS_CS3_SIZE be 0x0040?
(Compare with SYS_CS1 above)

g.

-- 
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot