In read_memory() we transfer at maximum 64 bytes, so the returned number
of actually sent bytes will never be greater than 64 and we can drop the
corresponding checks. Then it is checked if there are really 64 bytes
transferred. We don't do this elsewhere, so drop it here as well.
Signed-off-by: Sascha Hauer <s.hauer@xxxxxxxxxxxxxx>
---
scripts/imx/imx-usb-loader.c | 13 +++----------
1 file changed, 3 insertions(+), 10 deletions(-)
diff --git a/scripts/imx/imx-usb-loader.c b/scripts/imx/imx-usb-loader.c
index f4f13d5cc7..44ad7e3f0f 100644
--- a/scripts/imx/imx-usb-loader.c
+++ b/scripts/imx/imx-usb-loader.c
@@ -619,17 +619,10 @@ static int read_memory(unsigned addr, void *dest, unsigned cnt)
err, last_trans, tmp[0], tmp[1], tmp[2], tmp[3], cnt, rem);
break;
}
- if ((last_trans > rem) || (last_trans > 64)) {
- if ((last_trans == 64) && (rem < 64)) {
- /* Last transfer is expected to be too large for HID */
- } else {
- printf("err: %02x %02x %02x %02x cnt=%u rem=%d last_trans=%i\n",
- tmp[0], tmp[1], tmp[2], tmp[3], cnt, rem, last_trans);
- }
+
+ if (last_trans > rem)
last_trans = rem;
- if (last_trans > 64)
- last_trans = 64;
- }
+
memcpy(dest, tmp, last_trans);
dest += last_trans;
rem -= last_trans;
--
2.30.2
