Re: [U-Boot] [RFC PATCH] tools/imagetool: remove linker generated list

2015-02-08 Thread Guilherme Maciel Ferreira
er functions but > we also can not use the linker script. So use another approach copied from > linux kernel scripts/mod/file2alias.c. > > Signed-off-by: Andreas Bießmann > Cc: Guilherme Maciel Ferreira > --- > > tools/Makefile |2 -- > tools/imagetoo

Re: [U-Boot] recent tools on FreeBSD

2015-02-05 Thread Guilherme Maciel Ferreira
,17 @@ SECTIONS >> __u_boot_sandbox_option_end = .; >> >> __bss_start = .; >> + >> + . = ALIGN(32 / 8); >> + PROVIDE (__preinit_array_start = .); >> + .preinit_array : { *(.preinit_array) } >> + PROVIDE (_

Re: [U-Boot] recent tools on FreeBSD

2015-02-05 Thread Guilherme Maciel Ferreira
Hi Jeroen, My apologies, I didn't test the tools on BSD. The answers are inline. Best regards, Guilherme Am 04.02.2015 19:37 schrieb "Jeroen Hofstee" : > > Hello Guilherme / Simon, > > It seems that commit f86ed6a8d52c99bb2d17d3cac1647edca0c4399c, > "tools: moved code common to all image tools t

Re: [U-Boot] [PATCH 3/3] imagetool: replace image registration function by linker_lists feature

2015-01-14 Thread Guilherme Maciel Ferreira
Hi Simon, this patch is the suggestion you've gave to me on September 18, 2013 (long time ago). But I haven't worked on u-boot since then. Best Regards, -- Guilherme Maciel Ferreira Mobile Brazil: +55 48 9917 3969 Site: http://guilhermemacielferreira.com/ Skype: guilherme.macie

[U-Boot] [PATCH 2/3] imagetool: make the image_save_datafile() available to all image types

2015-01-14 Thread Guilherme Maciel Ferreira
Move the image_save_datafile() function from an U-Multi specific file (default_image.c) to a file common to all image types (image.c). And rename it to genimg_save_datafile(), to make clear it is useful for any image type. Signed-off-by: Guilherme Maciel Ferreira --- common/image.c

[U-Boot] [PATCH 4/4] tools: do not print error messages in verify_header() functions

2015-01-14 Thread Guilherme Maciel Ferreira
error. Otherwise we pollute the screen with errors messages until we find the image format that handle a given image file. Signed-off-by: Guilherme Maciel Ferreira --- tools/default_image.c | 17 - tools/socfpgaimage.c |6 -- 2 files changed, 12 insertions(+), 11 deletions(-)

[U-Boot] [PATCH 3/4] dumpimage: fit: extract FIT images

2015-01-14 Thread Guilherme Maciel Ferreira
age Compression: gzip compressed Data Size:4040128 Bytes = 3945.44 kB = 3.85 MB Architecture: PowerPC OS: Linux Load Address: 0x Entry Point: 0x Hash algo:md5 Hash value: 22352ad39bdc03e2e50f9cc28c1c3652 Which results in the file 'kerne

[U-Boot] [PATCH 1/3] imagetool: move common code to imagetool module

2015-01-14 Thread Guilherme Maciel Ferreira
The get_type() and verify_print_header() functions have the same code on both dumpimage.c and mkimage.c modules. Signed-off-by: Guilherme Maciel Ferreira --- tools/dumpimage.c | 66 ++-- tools/imagetool.c | 51

[U-Boot] [PATCH 3/3] imagetool: replace image registration function by linker_lists feature

2015-01-14 Thread Guilherme Maciel Ferreira
The registration was introduced in commit f86ed6a8d52c99bb2d17d3cac1647edca0c4399c This commit also removes all registration functions, and the member "next" from image_type_params struct Signed-off-by: Guilherme Maciel Ferreira --- tools/Makefile|2 + tools/aisimage.c

[U-Boot] [PATCH 1/4] dumpimage: add 'T' option to explicitly set the image type

2015-01-14 Thread Guilherme Maciel Ferreira
From: Guilherme Maciel Ferreira Some image types, like "KeyStone GP", do not have magic numbers to distinguish them from other image types. Thus, the automatic image type discovery does not work correctly. This patch also fix some integer type mismatches. Signed-off-by: Guilhe

[U-Boot] [PATCH] linker_lists: fix misspellings

2015-01-14 Thread Guilherme Maciel Ferreira
Signed-off-by: Guilherme Maciel Ferreira --- include/linker_lists.h |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/include/linker_lists.h b/include/linker_lists.h index d37fba4..940c871 100644 --- a/include/linker_lists.h +++ b/include/linker_lists.h @@ -23,7 +23,7

[U-Boot] [PATCH 2/4] dumpimage: replace the term "datafile" by "subimage"

2015-01-14 Thread Guilherme Maciel Ferreira
Signed-off-by: Guilherme Maciel Ferreira --- tools/default_image.c |6 +++--- tools/dumpimage.c | 12 ++-- tools/imagetool.c |2 +- tools/imagetool.h | 17 + 4 files changed, 19 insertions(+), 18 deletions(-) diff --git a/tools/default_image.c b

[U-Boot] [PATCH 1/2] doc: "os" is also mandatory for "ramdisk" FIT image components

2015-01-14 Thread Guilherme Maciel Ferreira
According to fit_image_print(), the "os" property from "image" node is required also when "type=ramdisk". Signed-off-by: Guilherme Maciel Ferreira --- doc/uImage.FIT/source_file_format.txt |8 1 files changed, 4 insertions(+), 4 deletions(-

[U-Boot] [PATCH 2/2] doc: fix misspellings

2015-01-14 Thread Guilherme Maciel Ferreira
Signed-off-by: Guilherme Maciel Ferreira --- doc/uImage.FIT/source_file_format.txt |2 +- doc/uImage.FIT/verified-boot.txt |4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/doc/uImage.FIT/source_file_format.txt b/doc/uImage.FIT/source_file_format.txt index

Re: [U-Boot] [PATCH 3/4] dumpimage: Added a tool to extract "data files" from U-Boot multifile images

2013-09-19 Thread Guilherme Maciel Ferreira
Humm, you are right, Bash expands this. Anyway, I'll remove the {env,} (along with Simon's suggestions) once there is no such dumpenvimage tool. My mistake. Kind regards, 2013/9/18 Otavio Salvador : > On Tue, Sep 17, 2013 at 9:42 PM, wrote: >> From: Guilherme Maciel Ferr

[U-Boot] [PATCH 2/4] tools: moved code common to all image tools to a separated module.

2013-09-17 Thread guilherme . maciel . ferreira
From: Guilherme Maciel Ferreira In order to avoid duplicating code and keep only one point of modification, the functions, structs and defines useful for "dumpimage" were moved from "mkimage" to a common module called "imagetool". This modification also weakens th

[U-Boot] [PATCH 4/4] sandbox: dumpimage: Test dumpimage

2013-09-17 Thread guilherme . maciel . ferreira
From: Guilherme Maciel Ferreira Signed-off-by: Guilherme Maciel Ferreira --- test/image/test-imagetools.sh | 107 + 1 files changed, 107 insertions(+), 0 deletions(-) create mode 100755 test/image/test-imagetools.sh diff --git a/test/image/test

[U-Boot] [PATCH 3/4] dumpimage: Added a tool to extract "data files" from U-Boot multifile images

2013-09-17 Thread guilherme . maciel . ferreira
From: Guilherme Maciel Ferreira Given a multifile image created through the mkimage's -d option: $ mkimage -A x86 -O linux -T multi -n x86 -d vmlinuz:initrd.img:System.map \ multi.img Image Name: x86 Created: Thu Jul 25 10:29:13 2013 Image Type: Intel x86 Linux Multi

[U-Boot] [PATCH 1/4] mkimage: added 'static' specifier to match function's prototype.

2013-09-17 Thread guilherme . maciel . ferreira
From: Guilherme Maciel Ferreira Signed-off-by: Guilherme Maciel Ferreira --- tools/mkimage.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/mkimage.c b/tools/mkimage.c index 7f22101..08aa634 100644 --- a/tools/mkimage.c +++ b/tools/mkimage.c @@ -632,8 +632,8

Re: [U-Boot] Extract data file from U-Boot image

2013-07-26 Thread Guilherme Maciel Ferreira
Dear Wolfgang, 2013/7/26 Wolfgang Denk : > Dear Guilherme Maciel Ferreira, > > In message > you > wrote: >> >> > But I fear that mkimage explodes from creaping featurism, so if you >> > go and implement something like this, please do not add it to kmi

Re: [U-Boot] Extract data file from U-Boot image

2013-07-25 Thread Guilherme Maciel Ferreira
Dear Wolfgang, 2013/7/25 Wolfgang Denk : > Dear Guilherme Maciel Ferreira, > > In message > you > wrote: >> >> I made a patch to allow mkimage to extract files from images, which >> just works for multi-file image. But, the fact that nobody has ever >> d

[U-Boot] Extract data file from U-Boot image

2013-07-25 Thread Guilherme Maciel Ferreira
uires 'dd' to extract and another tool to parse file's offsets inside the image. Comments? -- Guilherme Maciel Ferreira Mobile Brazil: +55 48 9904 3728 e +55 48 9134 4651 Site: http://guilhermemacielferreira.com/ Skype: guilherme.maciel.ferreira __