u-boot.git/tools/image-host.c, branch v2016.09

tools, rsa: Further minor cleanups on top of c236ebd and 2b9ec7

2016-07-25T16:01:36+00:00

[NOTE: I took v1 of these patches in, and then v2 came out, this commit
is squashing the minor deltas from v1 -> v2 of updates to c236ebd and
2b9ec76 into this commit - trini]

- Added an additional NULL check, as suggested by Simon Glass to
  fit_image_process_sig
- Re-formatted the comment blocks

Signed-off-by: Mario Six 
Reviewed-by: Simon Glass 
[For merging the chnages from v2 back onto v1]
Signed-off-by: Tom Rini

tools: Fix return code of fit_image_process_sig()

2016-07-22T18:46:24+00:00

When signing images, we repeatedly call fit_add_file_data() with
successively increasing size values to include the keys in the DTB.

Unfortunately, if large keys are used (such as 4096 bit RSA keys), this
process fails sometimes, and mkimage needs to be called repeatedly to
integrate the keys into the DTB.

This is because fit_add_file_data actually returns the wrong error
code, and the loop terminates prematurely, instead of trying again with
a larger size value.

This patch corrects the return value and also removes a error message,
which is misleading, since we actually allow the function to fail. A
(hopefully helpful) comment is also added to explain the lack of error
message.

This is probably related to 1152a05 ("tools: Correct error handling in
fit_image_process_hash()") and the corresponding error reported here:

https://www.mail-archive.com/u-boot@lists.denx.de/msg217417.html

Signed-off-by: Mario Six

tools: Correct error handling in fit_image_process_hash()

2016-07-14T22:22:37+00:00

We should not be returning -1 as an error code. This can mask a situation
where we run out of space adding things to the FIT. By returning the correct
error in this case (-ENOSPC) it can be handled by the higher-level code.

This may fix the error reported by Tom Van Deun here:

https://www.mail-archive.com/u-boot@lists.denx.de/msg217417.html

although I am not sure as I cannot actually repeat it.

Signed-off-by: Simon Glass 
Reported-by: Tom Van Deun 
Reviewed-by: Teddy Reed

tools: fix typo in tools/image-host.c

2014-08-14T09:20:01+00:00

fix a typo in error printf. If FIT_CONFS_PATH is not found
print FIT_CONFS_PATH not FIT_IMAGES_PATH.

Signed-off-by: Heiko Schocher 
Cc: Simon Glass 
Acked-by: Simon Glass

Enhance fit_check_sign to check all images

2014-06-19T15:19:02+00:00

At present this tool only checks the configuration signing. Have it also
look at each of the images in the configuration and confirm that they
verify.

Signed-off-by: Simon Glass 
Acked-by: Heiko Schocher  (v1)

mkimage: Automatically expand FDT in more cases

2014-06-19T15:18:56+00:00

The original code did not cover every case and there was a missing negative
sign in one case. Expand the coverage and fix the bug.

Signed-off-by: Simon Glass

mkimage: Automatically make space in FDT when full

2014-06-11T20:25:46+00:00

When adding hashes or signatures, the target FDT may be full. Detect this
and automatically try again after making 1KB of space.

Signed-off-by: Simon Glass

tools, fit_check_sign: verify a signed fit image

2014-03-21T20:40:38+00:00

add host tool "fit_check_sign" which verifies, if a fit image is
signed correct.

Signed-off-by: Heiko Schocher 
Cc: Simon Glass

tools/image-host: fix sign-images bug

2014-03-21T20:39:31+00:00

property "sign-images" is never found, fix this.

Signed-off-by: Heiko Schocher 
Acked-by: Simon Glass

Add GPL-2.0+ SPDX-License-Identifier to source files

2013-07-24T13:44:38+00:00

Signed-off-by: Wolfgang Denk 
[trini: Fixup common/cmd_io.c]
Signed-off-by: Tom Rini