usb: rockchip: be quiet on serial port while transferring data
authorAlberto Panizzo <alberto@amarulasolutions.com>
Thu, 12 Jul 2018 11:05:46 +0000 (13:05 +0200)
committerMarek Vasut <marex@denx.de>
Wed, 8 Aug 2018 20:22:07 +0000 (22:22 +0200)
While downloading or uploading megabytes of data we had thousands of
printf in console like:

transfer 0x10000 bytes done
OR
Uploading 0x1000 bytes

This because transfers are chunked and there is no way on target
side to know the overall transfer size (to print one string per
overall transfer).

All these prints on serial console do slow down significantly the
transfer and does not offer a significant information to the
developer: rkdeveloptool and Rockchip original tool do use small
chunks read/writes on big transfers. This allows on workstation
to print percentage of transfer complete and as well offers to
developer the information about: transfer is running OK.
On error, either the percentage will stop or an error will be shown
on workstation console.

Signed-off-by: Alberto Panizzo <alberto@amarulasolutions.com>
drivers/usb/gadget/f_rockusb.c

index bcdc2bac868fce3d002a44936c39a4c0cefef1af..0959b8c3a0a8ab1a6ca1417f6bbac3bfffd99960 100644 (file)
@@ -474,7 +474,7 @@ static void tx_handler_ul_image(struct usb_ep *ep, struct usb_request *req)
        memcpy(in_req->buf, rbuffer, transfer_size);
        in_req->length = transfer_size;
        in_req->complete = tx_handler_ul_image;
-       printf("Uploading 0x%x bytes\n", transfer_size);
+       debug("Uploading 0x%x bytes\n", transfer_size);
        usb_ep_dequeue(rockusb_func->in_ep, in_req);
        ret = usb_ep_queue(rockusb_func->in_ep, in_req, 0);
        if (ret)
@@ -536,7 +536,7 @@ static void rx_handler_dl_image(struct usb_ep *ep, struct usb_request *req)
                req->complete = rx_handler_command;
                req->length = EP_BUFFER_SIZE;
                f_rkusb->buf = f_rkusb->buf_head;
-               printf("transfer 0x%x bytes done\n", f_rkusb->dl_size);
+               debug("transfer 0x%x bytes done\n", f_rkusb->dl_size);
                f_rkusb->dl_size = 0;
                rockusb_tx_write_csw(f_rkusb->tag, 0, CSW_GOOD,
                                     USB_BULK_CS_WRAP_LEN);