fastboot: Improve error reporting on 'getvar partition-{size, type}'
authorEugeniu Rosca <erosca@de.adit-jv.com>
Thu, 28 Mar 2019 13:31:33 +0000 (14:31 +0100)
committerMarek Vasut <marex@denx.de>
Sun, 21 Apr 2019 08:26:51 +0000 (10:26 +0200)
Currently U-Boot reports the same error message in all below cases:
[A] host> fastboot getvar partition-type
[B] host> fastboot getvar partition-size
[C] host> fastboot getvar partition-type:
[D] host> fastboot getvar partition-size:
[E] host> fastboot getvar partition-type:<invalid-part>
[F] host> fastboot getvar partition-size:<invalid-part>

The message looks like:
host> fastboot getvar partition-size:
getvar:partition-size: FAILED (remote: partition not found)
Finished. Total time: 0.003s

Be more user friendly and output:
 - "partition not given" for [A-D]
 - "partition not found" for [E-F]

Fixes: f73a7df984a9 ("net: fastboot: Merge AOSP UDP fastboot")
Signed-off-by: Eugeniu Rosca <erosca@de.adit-jv.com>
Acked-by: Alex Kiernan <alex.kiernan@gmail.com>
drivers/fastboot/fb_mmc.c

index 4c1c7fd2cd8d08d1c90817beadb2341827d4cb24..d04d8a448ccf036f847daa911c47f6857454cd5c 100644 (file)
@@ -308,8 +308,8 @@ int fastboot_mmc_get_part_info(char *part_name, struct blk_desc **dev_desc,
                fastboot_fail("block device not found", response);
                return -ENOENT;
        }
-       if (!part_name) {
-               fastboot_fail("partition not found", response);
+       if (!part_name || !strcmp(part_name, "")) {
+               fastboot_fail("partition not given", response);
                return -ENOENT;
        }