qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 4/4] usb: use DPRINTF instead of printf for some


From: Hans de Goede
Subject: Re: [Qemu-devel] [PATCH 4/4] usb: use DPRINTF instead of printf for some simple cases
Date: Wed, 13 Apr 2011 14:52:37 +0200
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.15) Gecko/20110403 Fedora/3.1.9-6.fc15 Lightning/1.0b2 Thunderbird/3.1.9

Hi,

On 04/13/2011 11:45 AM, Brad Hards wrote:
Signed-off-by: Brad Hards<address@hidden>
---
  usb-linux.c |   16 ++++++++--------
  1 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/usb-linux.c b/usb-linux.c
index 1f33c2c..b02a0f9 100644
--- a/usb-linux.c
+++ b/usb-linux.c
@@ -233,8 +233,8 @@ static void async_complete(void *opaque)
                  return;
              }
              if (errno == ENODEV&&  !s->closing) {
-                printf("husb: device %d.%d disconnected\n",
-                       s->bus_num, s->addr);
+                DPRINTF("husb: device %d.%d disconnected\n",
+                        s->bus_num, s->addr);
                  usb_host_close(s);
                  usb_host_auto_check(NULL);
                  return;

I think this one should stay a regular printf, in case the disconnect is
unintentional people may think it is a qemu problem without the printf.

@@ -320,7 +320,7 @@ static int usb_host_claim_interfaces(USBHostDevice *dev, 
int configuration)
          }
          config_descr_len = dev->descr[i];

-        printf("husb: config #%d need %d\n", dev->descr[i + 5], configuration);
+        DPRINTF("husb: config #%d need %d\n", dev->descr[i + 5], 
configuration);

          if (configuration<  0 || configuration == dev->descr[i + 5]) {
              configuration = dev->descr[i + 5];

Ack.

@@ -359,7 +359,7 @@ static int usb_host_claim_interfaces(USBHostDevice *dev, 
int configuration)
          ret = ioctl(dev->fd, USBDEVFS_CLAIMINTERFACE,&interface);
          if (ret<  0) {
              if (errno == EBUSY) {
-                printf("husb: update iface. device already grabbed\n");
+                DPRINTF("husb: update iface. device already grabbed\n");
              } else {
                  perror("husb: failed to claim interface");
              }

Nack, this is an error condition, so it should not be a DPRINTF.

@@ -368,8 +368,8 @@ static int usb_host_claim_interfaces(USBHostDevice *dev, 
int configuration)
          }
      }

-    printf("husb: %d interfaces claimed for configuration %d\n",
-           nb_interfaces, configuration);
+    DPRINTF("husb: %d interfaces claimed for configuration %d\n",
+            nb_interfaces, configuration);

      dev->ninterfaces   = nb_interfaces;
      dev->configuration = configuration;

Ack.

@@ -929,7 +929,7 @@ static int usb_host_open(USBHostDevice *dev, int bus_num,
      if (dev->fd != -1) {
          goto fail;
      }
-    printf("husb: open device %d.%d\n", bus_num, addr);
+    DPRINTF("husb: open device %d.%d\n", bus_num, addr);

      if (!usb_host_device_path) {
          perror("husb: USB Host Device Path not set");

Ack.

@@ -984,7 +984,7 @@ static int usb_host_open(USBHostDevice *dev, int bus_num,
          goto fail;
      }

-    printf("husb: grabbed usb device %d.%d\n", bus_num, addr);
+    DPRINTF("husb: grabbed usb device %d.%d\n", bus_num, addr);

      ret = usb_linux_update_endp_table(dev);
      if (ret) {

Ack.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]