[U-Boot] [PATCH] MX31: Adjust default environment for QONG module

2010-04-28 Thread Stefano Babic 
Because the the size of u-boot increased after adding
new features (mainly the support for ubi/ubifs), storing
u-boot requires an additional sector on the flash. The patch
adjusts the kernel_addr and mtdparts variables giving 128KB
more space for u-boot code.

Signed-off-by: Stefano Babic sba...@denx.de
---
 include/configs/qong.h |4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/configs/qong.h b/include/configs/qong.h
index 1a2f19f..067dc79 100644
--- a/include/configs/qong.h
+++ b/include/configs/qong.h
@@ -144,7 +144,7 @@
addmtd=setenv bootargs ${bootargs} ${mtdparts}\0  \
addmisc=setenv bootargs ${bootargs}\0 \
uboot_addr=a000\0 \
-   kernel_addr=a008\0\
+   kernel_addr=a00a\0\
ramdisk_addr=a030\0   \
u-boot=qong/u-boot.bin\0  \
kernel_addr_r=8080\0  \
@@ -274,7 +274,7 @@ extern int qong_nand_rdy(void *chip);
 #define CONFIG_FLASH_CFI_MTD
 #define MTDIDS_DEFAULT nor0=physmap-flash.0
 #define MTDPARTS_DEFAULT   \
-   mtdparts=physmap-flash.0:256k(U-Boot),128k(env1), \
+   mtdparts=physmap-flash.0:384k(U-Boot),128k(env1), \
128k(env2),2560k(kernel),13m(ramdisk),-(user)
 
 #endif /* __CONFIG_H */
-- 
1.6.3.3

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

Re: [U-Boot] [PATCH] MX31: Adjust default environment for QONG module

2010-04-28 Thread Wolfgang Denk 
Dear Stefano Babic,

In message 1272455915-1143-1-git-send-email-sba...@denx.de you wrote:
 Because the the size of u-boot increased after adding
 new features (mainly the support for ubi/ubifs), storing
 u-boot requires an additional sector on the flash. The patch
 adjusts the kernel_addr and mtdparts variables giving 128KB
 more space for u-boot code.
...
 @@ -144,7 +144,7 @@
   addmtd=setenv bootargs ${bootargs} ${mtdparts}\0  \
   addmisc=setenv bootargs ${bootargs}\0 \
   uboot_addr=a000\0 \
 - kernel_addr=a008\0\
 + kernel_addr=a00a\0\
   ramdisk_addr=a030\0   \
=^

   u-boot=qong/u-boot.bin\0  \
   kernel_addr_r=8080\0  \
 @@ -274,7 +274,7 @@ extern int qong_nand_rdy(void *chip);
  #define CONFIG_FLASH_CFI_MTD
  #define MTDIDS_DEFAULT   nor0=physmap-flash.0
  #define MTDPARTS_DEFAULT \
 - mtdparts=physmap-flash.0:256k(U-Boot),128k(env1), \
 + mtdparts=physmap-flash.0:384k(U-Boot),128k(env1), \
   128k(env2),2560k(kernel),13m(ramdisk),-(user)

This shifts the start address of the ramdisk' and user partitions
to some odd addresses; also, this makes the definition of
ramdisk_addr (see above) incorrct.

I recommend to do this instead:

-   mtdparts=physmap-flash.0:256k(U-Boot),128k(env1), \
-   128k(env2),2560k(kernel),13m(ramdisk),-(user)
+   mtdparts=physmap-flash.0:384k(U-Boot),128k(env1), \
+   128k(env2),2432k(kernel),13m(ramdisk),-(user)
 

What do you think?

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
People seldom know what they want until you give them what  they  ask
for.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] MX31: Adjust default environment for QONG module

2010-04-28 Thread Stefano Babic 
Wolfgang Denk wrote:
  ramdisk_addr=a030\0   \
 =^
 
  u-boot=qong/u-boot.bin\0  \
  kernel_addr_r=8080\0  \
 @@ -274,7 +274,7 @@ extern int qong_nand_rdy(void *chip);
  #define CONFIG_FLASH_CFI_MTD
  #define MTDIDS_DEFAULT  nor0=physmap-flash.0
  #define MTDPARTS_DEFAULT\
 -mtdparts=physmap-flash.0:256k(U-Boot),128k(env1), \
 +mtdparts=physmap-flash.0:384k(U-Boot),128k(env1), \
  128k(env2),2560k(kernel),13m(ramdisk),-(user)
 
 This shifts the start address of the ramdisk' and user partitions
 to some odd addresses; also, this makes the definition of
 ramdisk_addr (see above) incorrct.

You are right. The ramdisk_addr must be shifted, or the kernel size must
be adjusted.

 I recommend to do this instead:
 
 - mtdparts=physmap-flash.0:256k(U-Boot),128k(env1), \
 - 128k(env2),2560k(kernel),13m(ramdisk),-(user)
 + mtdparts=physmap-flash.0:384k(U-Boot),128k(env1), \
 + 128k(env2),2432k(kernel),13m(ramdisk),-(user)
  
 
 What do you think?

Agree. I will send a new version of the patch with your proposed values.

Best regards,
Stefano Babic

-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: off...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot