On 3/27/24 17:23, Marek Behún wrote:
In the rng command, print
   Abort
instead of
   Reading RNG failed
if the error number is -EINTR, which can happen if the user pressed
CTRL-C.

Signed-off-by: Marek Behún <ka...@kernel.org>

Reviewed-by: Stefan Roese <s...@denx.de>

Thanks,
Stefan

---
  cmd/rng.c | 7 ++++---
  1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/cmd/rng.c b/cmd/rng.c
index 52f722c7af..48ba67061b 100644
--- a/cmd/rng.c
+++ b/cmd/rng.c
@@ -17,7 +17,7 @@ static int do_rng(struct cmd_tbl *cmdtp, int flag, int argc, 
char *const argv[])
        u8 buf[64];
        int devnum;
        struct udevice *dev;
-       int ret = CMD_RET_SUCCESS;
+       int ret = CMD_RET_SUCCESS, err;
switch (argc) {
        case 1:
@@ -46,8 +46,9 @@ static int do_rng(struct cmd_tbl *cmdtp, int flag, int argc, 
char *const argv[])
n = min(n, sizeof(buf)); - if (dm_rng_read(dev, buf, n)) {
-               printf("Reading RNG failed\n");
+       err = dm_rng_read(dev, buf, n);
+       if (err) {
+               puts(err == -EINTR ? "Abort\n" : "Reading RNG failed\n");
                ret = CMD_RET_FAILURE;
        } else {
                print_hex_dump_bytes("", DUMP_PREFIX_OFFSET, buf, n);

Viele Grüße,
Stefan Roese

--
DENX Software Engineering GmbH,      Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: s...@denx.de

Reply via email to