[U-Boot] [PATCH] image/FIT: Ramdisk Load, Entry address and OS tag inclusion
From: Jagannadha Sutradharudu Teki 402jagan@gmail.com
The current code in image.c doesn't include the Load, Entry address and OS tag to FIT image through mkimage tool.
Due to this while extracting the ramdisk from FIT blob, bootm showing an error like... Can't get ramdisk subimage load address! Ramdisk image is corrupt or invalid
So, I was added Load, Entry address and also OS tag on image.c. Now mkimage is able to include these details and it's works fine.
Regards, Jagan.
Jagannadha Sutradharudu Teki (1): image/FIT: Add ramdisk load, entry address and OS tag inclusion
common/image.c | 7 ++++--- 1 files changed, 4 insertions(+), 3 deletions(-)
From: Jagannadha Sutradharudu Teki 402jagan@gmail.com
This patch adds support to include Load, Entry address and OS tag of ramdisk on to FIT image through mkimage tool.
Signed-off-by: Jagannadha Sutradharudu Teki 402jagan@gmail.com --- common/image.c | 7 ++++--- 1 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/common/image.c b/common/image.c index 91954ac..70a112d 100644 --- a/common/image.c +++ b/common/image.c @@ -2042,13 +2042,13 @@ void fit_image_print(const void *fit, int image_noffset, const char *p) printf("%s Architecture: %s\n", p, genimg_get_arch_name(arch)); }
- if (type == IH_TYPE_KERNEL) { + if ((type == IH_TYPE_KERNEL) || (type == IH_TYPE_RAMDISK)) { fit_image_get_os(fit, image_noffset, &os); printf("%s OS: %s\n", p, genimg_get_os_name(os)); }
if ((type == IH_TYPE_KERNEL) || (type == IH_TYPE_STANDALONE) || - (type == IH_TYPE_FIRMWARE)) { + (type == IH_TYPE_FIRMWARE) || (type == IH_TYPE_RAMDISK)) { ret = fit_image_get_load(fit, image_noffset, &load); printf("%s Load Address: ", p); if (ret) @@ -2057,7 +2057,8 @@ void fit_image_print(const void *fit, int image_noffset, const char *p) printf("0x%08lx\n", load); }
- if ((type == IH_TYPE_KERNEL) || (type == IH_TYPE_STANDALONE)) { + if ((type == IH_TYPE_KERNEL) || (type == IH_TYPE_STANDALONE) || + (type == IH_TYPE_RAMDISK)) { fit_image_get_entry(fit, image_noffset, &entry); printf("%s Entry Point: ", p); if (ret)
Any point to consider this..or we have any solution that may not require this change?
Regards, Jagan.
On Sat, Jul 28, 2012 at 1:32 PM, Jagan 402jagan@gmail.com wrote:
From: Jagannadha Sutradharudu Teki 402jagan@gmail.com
This patch adds support to include Load, Entry address and OS tag of ramdisk on to FIT image through mkimage tool.
Signed-off-by: Jagannadha Sutradharudu Teki 402jagan@gmail.com
common/image.c | 7 ++++--- 1 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/common/image.c b/common/image.c index 91954ac..70a112d 100644 --- a/common/image.c +++ b/common/image.c @@ -2042,13 +2042,13 @@ void fit_image_print(const void *fit, int image_noffset, const char *p) printf("%s Architecture: %s\n", p, genimg_get_arch_name(arch)); }
if (type == IH_TYPE_KERNEL) {
if ((type == IH_TYPE_KERNEL) || (type == IH_TYPE_RAMDISK)) { fit_image_get_os(fit, image_noffset, &os); printf("%s OS: %s\n", p,genimg_get_os_name(os)); }
if ((type == IH_TYPE_KERNEL) || (type == IH_TYPE_STANDALONE) ||
(type == IH_TYPE_FIRMWARE)) {
(type == IH_TYPE_FIRMWARE) || (type == IH_TYPE_RAMDISK)) { ret = fit_image_get_load(fit, image_noffset, &load); printf("%s Load Address: ", p); if (ret)@@ -2057,7 +2057,8 @@ void fit_image_print(const void *fit, int image_noffset, const char *p) printf("0x%08lx\n", load); }
if ((type == IH_TYPE_KERNEL) || (type == IH_TYPE_STANDALONE)) {
if ((type == IH_TYPE_KERNEL) || (type == IH_TYPE_STANDALONE) ||(type == IH_TYPE_RAMDISK)) { fit_image_get_entry(fit, image_noffset, &entry); printf("%s Entry Point: ", p); if (ret)-- 1.7.0.4
Dear Jagan,
In message 1343462542-1556-2-git-send-email-jagan@techved.org you wrote:
From: Jagannadha Sutradharudu Teki 402jagan@gmail.com
This patch adds support to include Load, Entry address and OS tag of ramdisk on to FIT image through mkimage tool.
Signed-off-by: Jagannadha Sutradharudu Teki 402jagan@gmail.com
common/image.c | 7 ++++--- 1 files changed, 4 insertions(+), 3 deletions(-)
Applied, thanks.
Best regards,
Wolfgang Denk
participants (3)
-
Jagan -
jagan
-
Wolfgang Denk