Re: [U-Boot-Users] [PATCH] [FIT] Delay FIT format check on sector based devices

2008-06-09 Thread Scott Wood
On Fri, Jun 06, 2008 at 11:07:40PM +0200, Marian Balakowicz wrote:
 Global FIT image operations like format check cannot be performed on
 a first sector data, defer them to the point when whole FIT image was
 uploaded to a system RAM.
 
 Signed-off-by: Marian Balakowicz [EMAIL PROTECTED]
 ---
 
 Grant,
 
 Please give this patch a try on your system and let me know if it helped to
 solve the nand booting issue.
 
 Thanks,
 m.
 
  common/cmd_doc.c  |   19 ++-
  common/cmd_fdc.c  |   15 ---
  common/cmd_ide.c  |   19 ++-
  common/cmd_nand.c |   38 --
  common/cmd_scsi.c |   15 ---
  common/cmd_usb.c  |   15 ---
  6 files changed, 64 insertions(+), 57 deletions(-)

NAND and DOC bits Acked-by: Scott Wood [EMAIL PROTECTED]

-Scott

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] [PATCH] [FIT] Delay FIT format check on sector based devices

2008-06-06 Thread Marian Balakowicz
Global FIT image operations like format check cannot be performed on
a first sector data, defer them to the point when whole FIT image was
uploaded to a system RAM.

Signed-off-by: Marian Balakowicz [EMAIL PROTECTED]
---

Grant,

Please give this patch a try on your system and let me know if it helped to
solve the nand booting issue.

Thanks,
m.

 common/cmd_doc.c  |   19 ++-
 common/cmd_fdc.c  |   15 ---
 common/cmd_ide.c  |   19 ++-
 common/cmd_nand.c |   38 --
 common/cmd_scsi.c |   15 ---
 common/cmd_usb.c  |   15 ---
 6 files changed, 64 insertions(+), 57 deletions(-)


diff --git a/common/cmd_doc.c b/common/cmd_doc.c
index 83aba37..d7b2f53 100644
--- a/common/cmd_doc.c
+++ b/common/cmd_doc.c
@@ -206,7 +206,7 @@ int do_docboot (cmd_tbl_t *cmdtp, int flag, int argc, char 
*argv[])
image_header_t *hdr;
int rcode = 0;
 #if defined(CONFIG_FIT)
-   const void *fit_hdr;
+   const void *fit_hdr = NULL;
 #endif
 
show_boot_progress (34);
@@ -275,12 +275,6 @@ int do_docboot (cmd_tbl_t *cmdtp, int flag, int argc, char 
*argv[])
 #if defined(CONFIG_FIT)
case IMAGE_FORMAT_FIT:
fit_hdr = (const void *)addr;
-   if (!fit_check_format (fit_hdr)) {
-   show_boot_progress (-130);
-   puts (** Bad FIT image format\n);
-   return 1;
-   }
-   show_boot_progress (131);
puts (Fit image detected...\n);
 
cnt = fit_get_size (fit_hdr);
@@ -304,8 +298,15 @@ int do_docboot (cmd_tbl_t *cmdtp, int flag, int argc, char 
*argv[])
 
 #if defined(CONFIG_FIT)
/* This cannot be done earlier, we need complete FIT image in RAM first 
*/
-   if (genimg_get_format ((void *)addr) == IMAGE_FORMAT_FIT)
-   fit_print_contents ((const void *)addr);
+   if (genimg_get_format ((void *)addr) == IMAGE_FORMAT_FIT) {
+   if (!fit_check_format (fit_hdr)) {
+   show_boot_progress (-130);
+   puts (** Bad FIT image format\n);
+   return 1;
+   }
+   show_boot_progress (131);
+   fit_print_contents (fit_hdr);
+   }
 #endif
 
/* Loading ok, update default load address */
diff --git a/common/cmd_fdc.c b/common/cmd_fdc.c
index 0293d18..e4fbf29 100644
--- a/common/cmd_fdc.c
+++ b/common/cmd_fdc.c
@@ -787,7 +787,7 @@ int do_fdcboot (cmd_tbl_t *cmdtp, int flag, int argc, char 
*argv[])
char *ep;
int rcode = 0;
 #if defined(CONFIG_FIT)
-   const void *fit_hdr;
+   const void *fit_hdr = NULL;
 #endif
 
switch (argc) {
@@ -847,10 +847,6 @@ int do_fdcboot (cmd_tbl_t *cmdtp, int flag, int argc, char 
*argv[])
 #if defined(CONFIG_FIT)
case IMAGE_FORMAT_FIT:
fit_hdr = (const void *)addr;
-   if (!fit_check_format (fit_hdr)) {
-   puts (** Bad FIT image format\n);
-   return 1;
-   }
puts (Fit image detected...\n);
 
imsize = fit_get_size (fit_hdr);
@@ -879,8 +875,13 @@ int do_fdcboot (cmd_tbl_t *cmdtp, int flag, int argc, char 
*argv[])
 
 #if defined(CONFIG_FIT)
/* This cannot be done earlier, we need complete FIT image in RAM first 
*/
-   if (genimg_get_format ((void *)addr) == IMAGE_FORMAT_FIT)
-   fit_print_contents ((const void *)addr);
+   if (genimg_get_format ((void *)addr) == IMAGE_FORMAT_FIT) {
+   if (!fit_check_format (fit_hdr)) {
+   puts (** Bad FIT image format\n);
+   return 1;
+   }
+   fit_print_contents (fit_hdr);
+   }
 #endif
 
/* Loading ok, update default load address */
diff --git a/common/cmd_ide.c b/common/cmd_ide.c
index 6560702..97a873d 100644
--- a/common/cmd_ide.c
+++ b/common/cmd_ide.c
@@ -367,7 +367,7 @@ int do_diskboot (cmd_tbl_t *cmdtp, int flag, int argc, char 
*argv[])
image_header_t *hdr;
int rcode = 0;
 #if defined(CONFIG_FIT)
-   const void *fit_hdr;
+   const void *fit_hdr = NULL;
 #endif
 
show_boot_progress (41);
@@ -465,12 +465,6 @@ int do_diskboot (cmd_tbl_t *cmdtp, int flag, int argc, 
char *argv[])
 #if defined(CONFIG_FIT)
case IMAGE_FORMAT_FIT:
fit_hdr = (const void *)addr;
-   if (!fit_check_format (fit_hdr)) {
-   show_boot_progress (-140);
-   puts (** Bad FIT image format\n);
-   return 1;
-   }
-   show_boot_progress (141);
puts (Fit image detected...\n);
 
cnt = fit_get_size (fit_hdr);
@@ -496,8 +490,15 @@ int do_diskboot (cmd_tbl_t *cmdtp, int flag, int argc, 
char *argv[])
 
 #if defined(CONFIG_FIT)
/* This cannot be done earlier, 

Re: [U-Boot-Users] [PATCH] [FIT] Delay FIT format check on sector based devices

2008-06-06 Thread Grant Erickson
On 6/6/08 2:07 PM, Marian Balakowicz wrote:
 Global FIT image operations like format check cannot be performed on
 a first sector data, defer them to the point when whole FIT image was
 uploaded to a system RAM.
 
 Signed-off-by: Marian Balakowicz [EMAIL PROTECTED]
 ---
 
 Grant,
 
 Please give this patch a try on your system and let me know if it helped to
 solve the nand booting issue.

Marian:

Thanks for the prompt response on the patch. This matches what I mocked up
as a fix and testing your solution works for the 'cmd_nand' case on my AMCC
board. I did not test and cannot comment on the others so I'll partially ACK
it.

Partial Acked-by: Grant Erickson [EMAIL PROTECTED]

Thanks again,

Grant



-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users