diff options
| author | Ahmad Fatoum <a.fatoum@pengutronix.de> | 2024-05-03 12:08:56 +0200 |
|---|---|---|
| committer | Sascha Hauer <s.hauer@pengutronix.de> | 2024-05-06 08:16:17 +0200 |
| commit | 302db826b67c44a925a25a7b7508bc82a0fcd6a8 (patch) | |
| tree | f1ba346bcdc974ff4eeb6af018e20ad724d7975e /scripts | |
| parent | f4cc035245c6b2ec761ac9292190504a6426de6b (diff) | |
| download | barebox-302db826b67c44a925a25a7b7508bc82a0fcd6a8.tar.gz barebox-302db826b67c44a925a25a7b7508bc82a0fcd6a8.tar.xz | |
pblimage: suppress stdout output
Any output, stdout or stderr, during normal operation looks jarring when
interleaved with Kbuild, so utilities should just be always silent on
stdout and print errors to stderr if they occur.
The pblimage tool violated this and had a stdout debug print left-over and
printed part of an error message to stdout instead of stderr. Fix both
instances.
Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
Link: https://lore.barebox.org/20240503100856.1222507-1-a.fatoum@pengutronix.de
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'scripts')
| -rw-r--r-- | scripts/pblimage.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/scripts/pblimage.c b/scripts/pblimage.c index ef09b0f960..610d93b368 100644 --- a/scripts/pblimage.c +++ b/scripts/pblimage.c @@ -258,7 +258,6 @@ static void pbl_load_image(void) read(in_fd, mem_buf + 0x1000, image_size); pbl_size = 0x1000 + image_size; - printf("%s imagesize: %d\n", rcwfile, image_size); } else { exit(EXIT_FAILURE); } @@ -419,7 +418,7 @@ int main(int argc, char *argv[]) if (soc_type == SOC_TYPE_INVALID) { fprintf(stderr, "Invalid CPU type %s. Valid types are:\n", cputypestr); for (i = 0; i < ARRAY_SIZE(socs); i++) - printf(" %s\n", socs[i].name); + fprintf(stderr, " %s\n", socs[i].name); exit(EXIT_FAILURE); } |