From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1KTlWc-0006W8-1C for qemu-devel@nongnu.org; Thu, 14 Aug 2008 18:42:34 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1KTlWb-0006Vo-Id for qemu-devel@nongnu.org; Thu, 14 Aug 2008 18:42:33 -0400 Received: from [199.232.76.173] (port=44510 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1KTlWb-0006Vl-By for qemu-devel@nongnu.org; Thu, 14 Aug 2008 18:42:33 -0400 Received: from hera.kernel.org ([140.211.167.34]:36295) by monty-python.gnu.org with esmtps (TLS-1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.60) (envelope-from ) id 1KTlWa-0005Q1-Ox for qemu-devel@nongnu.org; Thu, 14 Aug 2008 18:42:33 -0400 From: Max Krasnyansky Date: Thu, 14 Aug 2008 22:42:28 +0000 Message-Id: <1218753748-27916-1-git-send-email-maxk@kernel.org> Subject: [Qemu-devel] [PATCH] husb: fixup printfs and stuff based on the review comments Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: kvm@vger.kernel.org, Max Krasnyansky Addressing Anthony's comments regarding printf and stuff. Anthony, if you you want I can fold this commit and resend the original patch. Signed-off-by: Max Krasnyansky --- usb-linux.c | 11 ++++------- 1 files changed, 4 insertions(+), 7 deletions(-) diff --git a/usb-linux.c b/usb-linux.c index dd78bb6..c31d56a 100644 --- a/usb-linux.c +++ b/usb-linux.c @@ -272,7 +272,7 @@ static int usb_host_update_interfaces(USBHostDevice *dev, int configuration) } if (i >= dev->descr_len) { - printf("husb: update iface failed. no matching configuration\n"); + fprintf(stderr, "husb: update iface failed. no matching configuration\n"); goto fail; } nb_interfaces = dev->descr[i + 4]; @@ -815,9 +815,6 @@ static int usb_host_auto_scan(void *opaque, int bus_num, int addr, return 0; for (f = usb_auto_filter; f; f = f->next) { - // printf("Auto match: bus_num %d addr %d vid %d pid %d\n", - // bus_num, addr, vendor_id, product_id); - if (f->bus_num >= 0 && f->bus_num != bus_num) continue; @@ -860,8 +857,8 @@ static void usb_host_auto_add(int bus_num, int addr, int vendor_id, int product_ { struct USBAutoFilter *f = qemu_mallocz(sizeof(*f)); if (!f) { - printf("husb: failed to allocate auto filter\n"); - return; + fprintf(stderr, "husb: failed to allocate auto filter\n"); + return; } f->bus_num = bus_num; @@ -878,7 +875,7 @@ static void usb_host_auto_add(int bus_num, int addr, int vendor_id, int product_ */ usb_auto_timer = qemu_new_timer(rt_clock, usb_host_auto_timer, NULL); if (!usb_auto_timer) { - printf("husb: failed to allocate timer\n"); + fprintf(stderr, "husb: failed to allocate auto scan timer\n"); qemu_free(f); return; } -- 1.5.5.1