fwnames[idx],
                                       &hdw->usb_dev->dev);
                if (!ret) {
-                       trace_firmware("Located %s firmware: %s;"
-                                      " uploading...",
+                       trace_firmware("Located %s firmware: %s; uploading...",
                                       fwtypename,
                                       fwnames[idx]);
                        return idx;
                return ret;
        }
        pvr2_trace(PVR2_TRACE_ERROR_LEGS,
-                  "***WARNING***"
-                  " Device %s firmware"
-                  " seems to be missing.",
+                  "***WARNING*** Device %s firmware seems to be missing.",
                   fwtypename);
        pvr2_trace(PVR2_TRACE_ERROR_LEGS,
-                  "Did you install the pvrusb2 firmware files"
-                  " in their proper location?");
+                  "Did you install the pvrusb2 firmware files in their proper location?");
        if (fwcount == 1) {
                pvr2_trace(PVR2_TRACE_ERROR_LEGS,
                           "request_firmware unable to locate %s file %s",
                           fwtypename,fwnames[0]);
        } else {
                pvr2_trace(PVR2_TRACE_ERROR_LEGS,
-                          "request_firmware unable to locate"
-                          " one of the following %s files:",
+                          "request_firmware unable to locate one of the following %s files:",
                           fwtypename);
                for (idx = 0; idx < fwcount; idx++) {
                        pvr2_trace(PVR2_TRACE_ERROR_LEGS,
        if (!hdw->hdw_desc->fx2_firmware.cnt) {
                hdw->fw1_state = FW1_STATE_OK;
                pvr2_trace(PVR2_TRACE_ERROR_LEGS,
-                          "Connected device type defines"
-                          " no firmware to upload; ignoring firmware");
+                          "Connected device type defines no firmware to upload; ignoring firmware");
                return -ENOTTY;
        }
 
            (!(hdw->hdw_desc->flag_fx2_16kb && (fwsize == 0x4000)))) {
                if (hdw->hdw_desc->flag_fx2_16kb) {
                        pvr2_trace(PVR2_TRACE_ERROR_LEGS,
-                                  "Wrong fx2 firmware size"
-                                  " (expected 8192 or 16384, got %u)",
+                                  "Wrong fx2 firmware size (expected 8192 or 16384, got %u)",
                                   fwsize);
                } else {
                        pvr2_trace(PVR2_TRACE_ERROR_LEGS,
-                                  "Wrong fx2 firmware size"
-                                  " (expected 8192, got %u)",
+                                  "Wrong fx2 firmware size (expected 8192, got %u)",
                                   fwsize);
                }
                release_firmware(fw_entry);
 
        if (fw_len % sizeof(u32)) {
                pvr2_trace(PVR2_TRACE_ERROR_LEGS,
-                          "size of %s firmware"
-                          " must be a multiple of %zu bytes",
+                          "size of %s firmware must be a multiple of %zu bytes",
                           fw_files[fwidx],sizeof(u32));
                release_firmware(fw_entry);
                ret = -EINVAL;
 
        bcnt = pvr2_std_id_to_str(buf,sizeof(buf),hdw->std_mask_eeprom);
        pvr2_trace(PVR2_TRACE_STD,
-                  "Supported video standard(s) reported available"
-                  " in hardware: %.*s",
+                  "Supported video standard(s) reported available in hardware: %.*s",
                   bcnt,buf);
 
        hdw->std_mask_avail = hdw->std_mask_eeprom;
        if (std2) {
                bcnt = pvr2_std_id_to_str(buf,sizeof(buf),std2);
                pvr2_trace(PVR2_TRACE_STD,
-                          "Expanding supported video standards"
-                          " to include: %.*s",
+                          "Expanding supported video standards to include: %.*s",
                           bcnt,buf);
                hdw->std_mask_avail |= std2;
        }
        if (std3) {
                bcnt = pvr2_std_id_to_str(buf,sizeof(buf),std3);
                pvr2_trace(PVR2_TRACE_STD,
-                          "Initial video standard"
-                          " (determined by device type): %.*s",bcnt,buf);
+                          "Initial video standard (determined by device type): %.*s",
+                          bcnt, buf);
                hdw->std_mask_cur = std3;
                hdw->std_dirty = !0;
                return;
        }
 
        pvr2_trace(PVR2_TRACE_INIT,
-                  "Module ID %u:"
-                  " Executing cx25840 VBI hack",
+                  "Module ID %u: Executing cx25840 VBI hack",
                   hdw->decoder_client_id);
        memset(&fmt, 0, sizeof(fmt));
        fmt.type = V4L2_BUF_TYPE_SLICED_VBI_CAPTURE;
        fname = (mid < ARRAY_SIZE(module_names)) ? module_names[mid] : NULL;
        if (!fname) {
                pvr2_trace(PVR2_TRACE_ERROR_LEGS,
-                          "Module ID %u for device %s has no name?"
-                          "  The driver might have a configuration problem.",
+                          "Module ID %u for device %s has no name?  The driver might have a configuration problem.",
                           mid,
                           hdw->hdw_desc->description);
                return -EINVAL;
                                                 ARRAY_SIZE(i2caddr));
                if (i2ccnt) {
                        pvr2_trace(PVR2_TRACE_INIT,
-                                  "Module ID %u:"
-                                  " Using default i2c address list",
+                                  "Module ID %u: Using default i2c address list",
                                   mid);
                }
        }
 
        if (!i2ccnt) {
                pvr2_trace(PVR2_TRACE_ERROR_LEGS,
-                          "Module ID %u (%s) for device %s:"
-                          " No i2c addresses."
-                          "  The driver might have a configuration problem.",
+                          "Module ID %u (%s) for device %s: No i2c addresses.  The driver might have a configuration problem.",
                           mid, fname, hdw->hdw_desc->description);
                return -EINVAL;
        }
 
        if (i2ccnt == 1) {
                pvr2_trace(PVR2_TRACE_INIT,
-                          "Module ID %u:"
-                          " Setting up with specified i2c address 0x%x",
+                          "Module ID %u: Setting up with specified i2c address 0x%x",
                           mid, i2caddr[0]);
                sd = v4l2_i2c_new_subdev(&hdw->v4l2_dev, &hdw->i2c_adap,
                                         fname, i2caddr[0], NULL);
        } else {
                pvr2_trace(PVR2_TRACE_INIT,
-                          "Module ID %u:"
-                          " Setting up with address probe list",
+                          "Module ID %u: Setting up with address probe list",
                           mid);
                sd = v4l2_i2c_new_subdev(&hdw->v4l2_dev, &hdw->i2c_adap,
                                         fname, 0, i2caddr);
 
        if (!sd) {
                pvr2_trace(PVR2_TRACE_ERROR_LEGS,
-                          "Module ID %u (%s) for device %s failed to load."
-                          "  Possible missing sub-device kernel module or"
-                          " initialization failure within module.",
+                          "Module ID %u (%s) for device %s failed to load.  Possible missing sub-device kernel module or initialization failure within module.",
                           mid, fname, hdw->hdw_desc->description);
                return -EIO;
        }
                                 == 0);
                        if (reloadFl) {
                                pvr2_trace(PVR2_TRACE_INIT,
-                                          "USB endpoint config looks strange"
-                                          "; possibly firmware needs to be"
-                                          " loaded");
+                                          "USB endpoint config looks strange; possibly firmware needs to be loaded");
                        }
                }
                if (!reloadFl) {
                        reloadFl = !pvr2_hdw_check_firmware(hdw);
                        if (reloadFl) {
                                pvr2_trace(PVR2_TRACE_INIT,
-                                          "Check for FX2 firmware failed"
-                                          "; possibly firmware needs to be"
-                                          " loaded");
+                                          "Check for FX2 firmware failed; possibly firmware needs to be loaded");
                        }
                }
                if (reloadFl) {
                if (!pvr2_hdw_dev_ok(hdw)) return;
                if (ret < 0) {
                        pvr2_trace(PVR2_TRACE_ERROR_LEGS,
-                                  "Unable to determine location of eeprom,"
-                                  " skipping");
+                                  "Unable to determine location of eeprom, skipping");
                } else {
                        hdw->eeprom_addr = ret;
                        pvr2_eeprom_analyze(hdw);
                idx = get_default_error_tolerance(hdw);
                if (idx) {
                        pvr2_trace(PVR2_TRACE_INIT,
-                                  "pvr2_hdw_setup: video stream %p"
-                                  " setting tolerance %u",
+                                  "pvr2_hdw_setup: video stream %p setting tolerance %u",
                                   hdw->vid_stream,idx);
                }
                pvr2_stream_setup(hdw->vid_stream,hdw->usb_dev,
                        if (hdw->flag_init_ok) {
                                pvr2_trace(
                                        PVR2_TRACE_INFO,
-                                       "Device initialization"
-                                       " completed successfully.");
+                                       "Device initialization completed successfully.");
                                break;
                        }
                        if (hdw->fw1_state == FW1_STATE_RELOAD) {
                                pvr2_trace(
                                        PVR2_TRACE_INFO,
-                                       "Device microcontroller firmware"
-                                       " (re)loaded; it should now reset"
-                                       " and reconnect.");
+                                       "Device microcontroller firmware (re)loaded; it should now reset and reconnect.");
                                break;
                        }
                        pvr2_trace(
                        if (hdw->fw1_state == FW1_STATE_MISSING) {
                                pvr2_trace(
                                        PVR2_TRACE_ERROR_LEGS,
-                                       "Giving up since device"
-                                       " microcontroller firmware"
-                                       " appears to be missing.");
+                                       "Giving up since device microcontroller firmware appears to be missing.");
                                break;
                        }
                }
                if (hdw->flag_modulefail) {
                        pvr2_trace(
                                PVR2_TRACE_ERROR_LEGS,
-                               "***WARNING*** pvrusb2 driver initialization"
-                               " failed due to the failure of one or more"
-                               " sub-device kernel modules.");
+                               "***WARNING*** pvrusb2 driver initialization failed due to the failure of one or more sub-device kernel modules.");
                        pvr2_trace(
                                PVR2_TRACE_ERROR_LEGS,
-                               "You need to resolve the failing condition"
-                               " before this driver can function.  There"
-                               " should be some earlier messages giving more"
-                               " information about the problem.");
+                               "You need to resolve the failing condition before this driver can function.  There should be some earlier messages giving more information about the problem.");
                        break;
                }
                if (procreload) {
                        pvr2_trace(
                                PVR2_TRACE_ERROR_LEGS,
-                               "Attempting pvrusb2 recovery by reloading"
-                               " primary firmware.");
+                               "Attempting pvrusb2 recovery by reloading primary firmware.");
                        pvr2_trace(
                                PVR2_TRACE_ERROR_LEGS,
-                               "If this works, device should disconnect"
-                               " and reconnect in a sane state.");
+                               "If this works, device should disconnect and reconnect in a sane state.");
                        hdw->fw1_state = FW1_STATE_UNKNOWN;
                        pvr2_upload_firmware1(hdw);
                } else {
                        pvr2_trace(
                                PVR2_TRACE_ERROR_LEGS,
-                               "***WARNING*** pvrusb2 device hardware"
-                               " appears to be jammed"
-                               " and I can't clear it.");
+                               "***WARNING*** pvrusb2 device hardware appears to be jammed and I can't clear it.");
                        pvr2_trace(
                                PVR2_TRACE_ERROR_LEGS,
-                               "You might need to power cycle"
-                               " the pvrusb2 device"
-                               " in order to recover.");
+                               "You might need to power cycle the pvrusb2 device in order to recover.");
                }
        } while (0);
        pvr2_trace(PVR2_TRACE_INIT,"pvr2_hdw_setup(hdw=%p) end",hdw);
        hdw_desc = (const struct pvr2_device_desc *)(devid->driver_info);
 
        if (hdw_desc == NULL) {
-               pvr2_trace(PVR2_TRACE_INIT, "pvr2_hdw_create:"
-                          " No device description pointer,"
-                          " unable to continue.");
-               pvr2_trace(PVR2_TRACE_INIT, "If you have a new device type,"
-                          " please contact Mike Isely <isely@pobox.com>"
-                          " to get it included in the driver\n");
+               pvr2_trace(PVR2_TRACE_INIT, "pvr2_hdw_create: No device description pointer, unable to continue.");
+               pvr2_trace(PVR2_TRACE_INIT, "If you have a new device type, please contact Mike Isely <isely@pobox.com> to get it included in the driver\n");
                goto fail;
        }
 
        if (hdw_desc->flag_is_experimental) {
                pvr2_trace(PVR2_TRACE_INFO, "**********");
                pvr2_trace(PVR2_TRACE_INFO,
-                          "WARNING: Support for this device (%s) is"
-                          " experimental.", hdw_desc->description);
+                          "WARNING: Support for this device (%s) is experimental.",
+                                                             hdw_desc->description);
                pvr2_trace(PVR2_TRACE_INFO,
-                          "Important functionality might not be"
-                          " entirely working.");
+                          "Important functionality might not be entirely working.");
                pvr2_trace(PVR2_TRACE_INFO,
-                          "Please consider contacting the driver author to"
-                          " help with further stabilization of the driver.");
+                          "Please consider contacting the driver author to help with further stabilization of the driver.");
                pvr2_trace(PVR2_TRACE_INFO, "**********");
        }
        if (!hdw) goto fail;
        eeprom = kmalloc(EEPROM_SIZE,GFP_KERNEL);
        if (!eeprom) {
                pvr2_trace(PVR2_TRACE_ERROR_LEGS,
-                          "Failed to allocate memory"
-                          " required to read eeprom");
+                          "Failed to allocate memory required to read eeprom");
                return NULL;
        }
 
           strange but it's what they do) */
        mode16 = (addr & 1);
        eepromSize = (mode16 ? EEPROM_SIZE : 256);
-       trace_eeprom("Examining %d byte eeprom at location 0x%x"
-                    " using %d bit addressing",eepromSize,addr,
+       trace_eeprom("Examining %d byte eeprom at location 0x%x using %d bit addressing",
+                    eepromSize, addr,
                     mode16 ? 16 : 8);
 
        msg[0].addr = addr;
                if (hdw->fw_cpu_flag) {
                        hdw->fw_size = (mode == 1) ? 0x4000 : 0x2000;
                        pvr2_trace(PVR2_TRACE_FIRMWARE,
-                                  "Preparing to suck out CPU firmware"
-                                  " (size=%u)", hdw->fw_size);
+                                  "Preparing to suck out CPU firmware (size=%u)",
+                                  hdw->fw_size);
                        hdw->fw_buffer = kzalloc(hdw->fw_size,GFP_KERNEL);
                        if (!hdw->fw_buffer) {
                                hdw->fw_size = 0;
        struct timer_list timer;
        if (!hdw->ctl_lock_held) {
                pvr2_trace(PVR2_TRACE_ERROR_LEGS,
-                          "Attempted to execute control transfer"
-                          " without lock!!");
+                          "Attempted to execute control transfer without lock!!");
                return -EDEADLK;
        }
        if (!hdw->flag_ok && !probe_fl) {
                pvr2_trace(PVR2_TRACE_ERROR_LEGS,
-                          "Attempted to execute control transfer"
-                          " when device not ok");
+                          "Attempted to execute control transfer when device not ok");
                return -EIO;
        }
        if (!(hdw->ctl_read_urb && hdw->ctl_write_urb)) {
                if (!probe_fl) {
                        pvr2_trace(PVR2_TRACE_ERROR_LEGS,
-                                  "Attempted to execute control transfer"
-                                  " when USB is disconnected");
+                                  "Attempted to execute control transfer when USB is disconnected");
                }
                return -ENOTTY;
        }
        if (write_len > PVR2_CTL_BUFFSIZE) {
                pvr2_trace(
                        PVR2_TRACE_ERROR_LEGS,
-                       "Attempted to execute %d byte"
-                       " control-write transfer (limit=%d)",
+                       "Attempted to execute %d byte control-write transfer (limit=%d)",
                        write_len,PVR2_CTL_BUFFSIZE);
                return -EINVAL;
        }
        if (read_len > PVR2_CTL_BUFFSIZE) {
                pvr2_trace(
                        PVR2_TRACE_ERROR_LEGS,
-                       "Attempted to execute %d byte"
-                       " control-read transfer (limit=%d)",
+                       "Attempted to execute %d byte control-read transfer (limit=%d)",
                        write_len,PVR2_CTL_BUFFSIZE);
                return -EINVAL;
        }
                status = usb_submit_urb(hdw->ctl_write_urb,GFP_KERNEL);
                if (status < 0) {
                        pvr2_trace(PVR2_TRACE_ERROR_LEGS,
-                                  "Failed to submit write-control"
-                                  " URB status=%d",status);
+                                  "Failed to submit write-control URB status=%d",
+status);
                        hdw->ctl_write_pend_flag = 0;
                        goto done;
                }
                status = usb_submit_urb(hdw->ctl_read_urb,GFP_KERNEL);
                if (status < 0) {
                        pvr2_trace(PVR2_TRACE_ERROR_LEGS,
-                                  "Failed to submit read-control"
-                                  " URB status=%d",status);
+                                  "Failed to submit read-control URB status=%d",
+status);
                        hdw->ctl_read_pend_flag = 0;
                        goto done;
                }
                        status = hdw->ctl_write_urb->status;
                        if (!probe_fl) {
                                pvr2_trace(PVR2_TRACE_ERROR_LEGS,
-                                          "control-write URB failure,"
-                                          " status=%d",
+                                          "control-write URB failure, status=%d",
                                           status);
                        }
                        goto done;
                        status = -EIO;
                        if (!probe_fl) {
                                pvr2_trace(PVR2_TRACE_ERROR_LEGS,
-                                          "control-write URB short,"
-                                          " expected=%d got=%d",
+                                          "control-write URB short, expected=%d got=%d",
                                           write_len,
                                           hdw->ctl_write_urb->actual_length);
                        }
                        status = hdw->ctl_read_urb->status;
                        if (!probe_fl) {
                                pvr2_trace(PVR2_TRACE_ERROR_LEGS,
-                                          "control-read URB failure,"
-                                          " status=%d",
+                                          "control-read URB failure, status=%d",
                                           status);
                        }
                        goto done;
                        status = -EIO;
                        if (!probe_fl) {
                                pvr2_trace(PVR2_TRACE_ERROR_LEGS,
-                                          "control-read URB short,"
-                                          " expected=%d got=%d",
+                                          "control-read URB short, expected=%d got=%d",
                                           read_len,
                                           hdw->ctl_read_urb->actual_length);
                        }
                                      0);
                return scnprintf(
                        buf,acnt,
-                       "Bytes streamed=%u"
-                       " URBs: queued=%u idle=%u ready=%u"
-                       " processed=%u failed=%u",
+                       "Bytes streamed=%u URBs: queued=%u idle=%u ready=%u processed=%u failed=%u",
                        stats.bytes_processed,
                        stats.buffers_in_queue,
                        stats.buffers_in_idle,
                if (ret) return ret;
                nval = (cval & ~msk) | (val & msk);
                pvr2_trace(PVR2_TRACE_GPIO,
-                          "GPIO direction changing 0x%x:0x%x"
-                          " from 0x%x to 0x%x",
+                          "GPIO direction changing 0x%x:0x%x from 0x%x to 0x%x",
                           msk,val,cval,nval);
        } else {
                nval = val;
           now.  (Of course, no sub-drivers seem to implement it either.
           But now it's a a chicken and egg problem...) */
        v4l2_device_call_all(&hdw->v4l2_dev, 0, tuner, g_tuner, vtp);
-       pvr2_trace(PVR2_TRACE_CHIPS, "subdev status poll"
-                  " type=%u strength=%u audio=0x%x cap=0x%x"
-                  " low=%u hi=%u",
+       pvr2_trace(PVR2_TRACE_CHIPS, "subdev status poll type=%u strength=%u audio=0x%x cap=0x%x low=%u hi=%u",
                   vtp->type,
                   vtp->signal, vtp->rxsubchans, vtp->capability,
                   vtp->rangelow, vtp->rangehigh);
 
        if (!data) length = 0;
        if (length > (sizeof(hdw->cmd_buffer) - 3)) {
                pvr2_trace(PVR2_TRACE_ERROR_LEGS,
-                          "Killing an I2C write to %u that is too large"
-                          " (desired=%u limit=%u)",
+                          "Killing an I2C write to %u that is too large (desired=%u limit=%u)",
                           i2c_addr,
                           length,(unsigned int)(sizeof(hdw->cmd_buffer) - 3));
                return -ENOTSUPP;
                if (hdw->cmd_buffer[0] != 8) {
                        ret = -EIO;
                        if (hdw->cmd_buffer[0] != 7) {
-                               trace_i2c("unexpected status"
-                                         " from i2_write[%d]: %d",
+                               trace_i2c("unexpected status from i2_write[%d]: %d",
                                          i2c_addr,hdw->cmd_buffer[0]);
                        }
                }
        if (!data) dlen = 0;
        if (dlen > (sizeof(hdw->cmd_buffer) - 4)) {
                pvr2_trace(PVR2_TRACE_ERROR_LEGS,
-                          "Killing an I2C read to %u that has wlen too large"
-                          " (desired=%u limit=%u)",
+                          "Killing an I2C read to %u that has wlen too large (desired=%u limit=%u)",
                           i2c_addr,
                           dlen,(unsigned int)(sizeof(hdw->cmd_buffer) - 4));
                return -ENOTSUPP;
        }
        if (res && (rlen > (sizeof(hdw->cmd_buffer) - 1))) {
                pvr2_trace(PVR2_TRACE_ERROR_LEGS,
-                          "Killing an I2C read to %u that has rlen too large"
-                          " (desired=%u limit=%u)",
+                          "Killing an I2C read to %u that has rlen too large (desired=%u limit=%u)",
                           i2c_addr,
                           rlen,(unsigned int)(sizeof(hdw->cmd_buffer) - 1));
                return -ENOTSUPP;
                if (hdw->cmd_buffer[0] != 8) {
                        ret = -EIO;
                        if (hdw->cmd_buffer[0] != 7) {
-                               trace_i2c("unexpected status"
-                                         " from i2_read[%d]: %d",
+                               trace_i2c("unexpected status from i2_read[%d]: %d",
                                          i2c_addr,hdw->cmd_buffer[0]);
                        }
                }
 
        if ((ret != 0) || (*rdata == 0x04) || (*rdata == 0x0a)) {
                pvr2_trace(PVR2_TRACE_ERROR_LEGS,
-                          "WARNING: Detected a wedged cx25840 chip;"
-                          " the device will not work.");
+                          "WARNING: Detected a wedged cx25840 chip; the device will not work.");
                pvr2_trace(PVR2_TRACE_ERROR_LEGS,
                           "WARNING: Try power cycling the pvrusb2 device.");
                pvr2_trace(PVR2_TRACE_ERROR_LEGS,
-                          "WARNING: Disabling further access to the device"
-                          " to prevent other foul-ups.");
+                          "WARNING: Disabling further access to the device to prevent other foul-ups.");
                // This blocks all further communication with the part.
                hdw->i2c_func[0x44] = NULL;
                pvr2_hdw_render_useless(hdw);
                }
        } else if (num == 2) {
                if (msgs[0].addr != msgs[1].addr) {
-                       trace_i2c("i2c refusing 2 phase transfer with"
-                                 " conflicting target addresses");
+                       trace_i2c("i2c refusing 2 phase transfer with conflicting target addresses");
                        ret = -ENOTSUPP;
                        goto done;
                }
                        ret = 2;
                        goto done;
                } else {
-                       trace_i2c("i2c refusing complex transfer"
-                                 " read0=%d read1=%d",
+                       trace_i2c("i2c refusing complex transfer read0=%d read1=%d",
                                  (msgs[0].flags & I2C_M_RD),
                                  (msgs[1].flags & I2C_M_RD));
                }
                for (idx = 0; idx < num; idx++) {
                        cnt = msgs[idx].len;
                        printk(KERN_INFO
-                              "pvrusb2 i2c xfer %u/%u:"
-                              " addr=0x%x len=%d %s",
+                              "pvrusb2 i2c xfer %u/%u: addr=0x%x len=%d %s",
                               idx+1,num,
                               msgs[idx].addr,
                               cnt,
                   the emulated IR receiver. */
                if (do_i2c_probe(hdw, 0x71)) {
                        pvr2_trace(PVR2_TRACE_INFO,
-                                  "Device has newer IR hardware;"
-                                  " disabling unneeded virtual IR device");
+                                  "Device has newer IR hardware; disabling unneeded virtual IR device");
                        hdw->i2c_func[0x18] = NULL;
                        /* Remember that this is a different device... */
                        hdw->ir_scheme_active = PVR2_IR_SCHEME_24XXX_MCE;
 
 static void pvr2_buffer_describe(struct pvr2_buffer *bp,const char *msg)
 {
        pvr2_trace(PVR2_TRACE_INFO,
-                  "buffer%s%s %p state=%s id=%d status=%d"
-                  " stream=%p purb=%p sig=0x%x",
+                  "buffer%s%s %p state=%s id=%d status=%d stream=%p purb=%p sig=0x%x",
                   (msg ? " " : ""),
                   (msg ? msg : ""),
                   bp,
        (*cnt)--;
        (*bcnt) -= ccnt;
        pvr2_trace(PVR2_TRACE_BUF_FLOW,
-                  "/*---TRACE_FLOW---*/"
-                  " bufferPool     %8s dec cap=%07d cnt=%02d",
+                  "/*---TRACE_FLOW---*/ bufferPool     %8s dec cap=%07d cnt=%02d",
                   pvr2_buffer_state_decode(bp->state),*bcnt,*cnt);
        bp->state = pvr2_buffer_state_none;
 }
        (sp->r_count)++;
        sp->r_bcount += bp->used_count;
        pvr2_trace(PVR2_TRACE_BUF_FLOW,
-                  "/*---TRACE_FLOW---*/"
-                  " bufferPool     %8s inc cap=%07d cnt=%02d",
+                  "/*---TRACE_FLOW---*/ bufferPool     %8s inc cap=%07d cnt=%02d",
                   pvr2_buffer_state_decode(bp->state),
                   sp->r_bcount,sp->r_count);
        spin_unlock_irqrestore(&sp->list_lock,irq_flags);
        (sp->i_count)++;
        sp->i_bcount += bp->max_count;
        pvr2_trace(PVR2_TRACE_BUF_FLOW,
-                  "/*---TRACE_FLOW---*/"
-                  " bufferPool     %8s inc cap=%07d cnt=%02d",
+                  "/*---TRACE_FLOW---*/ bufferPool     %8s inc cap=%07d cnt=%02d",
                   pvr2_buffer_state_decode(bp->state),
                   sp->i_bcount,sp->i_count);
        spin_unlock_irqrestore(&sp->list_lock,irq_flags);
        (sp->q_count)++;
        sp->q_bcount += bp->max_count;
        pvr2_trace(PVR2_TRACE_BUF_FLOW,
-                  "/*---TRACE_FLOW---*/"
-                  " bufferPool     %8s inc cap=%07d cnt=%02d",
+                  "/*---TRACE_FLOW---*/ bufferPool     %8s inc cap=%07d cnt=%02d",
                   pvr2_buffer_state_decode(bp->state),
                   sp->q_bcount,sp->q_count);
        spin_unlock_irqrestore(&sp->list_lock,irq_flags);
        bp->signature = 0;
        bp->stream = NULL;
        usb_free_urb(bp->purb);
-       pvr2_trace(PVR2_TRACE_BUF_POOL,"/*---TRACE_FLOW---*/"
-                  " bufferDone     %p",bp);
+       pvr2_trace(PVR2_TRACE_BUF_POOL, "/*---TRACE_FLOW---*/ bufferDone     %p",
+                  bp);
 }
 
 static int pvr2_stream_buffer_count(struct pvr2_stream *sp,unsigned int cnt)
        if (cnt == sp->buffer_total_count) return 0;
 
        pvr2_trace(PVR2_TRACE_BUF_POOL,
-                  "/*---TRACE_FLOW---*/ poolResize    "
-                  " stream=%p cur=%d adj=%+d",
+                  "/*---TRACE_FLOW---*/ poolResize     stream=%p cur=%d adj=%+d",
                   sp,
                   sp->buffer_total_count,
                   cnt-sp->buffer_total_count);
        if (sp->buffer_total_count == sp->buffer_target_count) return 0;
 
        pvr2_trace(PVR2_TRACE_BUF_POOL,
-                  "/*---TRACE_FLOW---*/"
-                  " poolCheck      stream=%p cur=%d tgt=%d",
+                  "/*---TRACE_FLOW---*/ poolCheck      stream=%p cur=%d tgt=%d",
                   sp,sp->buffer_total_count,sp->buffer_target_count);
 
        if (sp->buffer_total_count < sp->buffer_target_count) {
                bp->used_count = urb->actual_length;
                if (sp->fail_count) {
                        pvr2_trace(PVR2_TRACE_TOLERANCE,
-                                  "stream %p transfer ok"
-                                  " - fail count reset",sp);
+                                  "stream %p transfer ok - fail count reset",
+                                  sp);
                        sp->fail_count = 0;
                }
        } else if (sp->fail_count < sp->fail_tolerance) {
                (sp->fail_count)++;
                (sp->buffers_failed)++;
                pvr2_trace(PVR2_TRACE_TOLERANCE,
-                          "stream %p ignoring error %d"
-                          " - fail count increased to %u",
+                          "stream %p ignoring error %d - fail count increased to %u",
                           sp,urb->status,sp->fail_count);
        } else {
                (sp->buffers_failed)++;
                        bp->max_count = cnt;
                        bp->stream->i_bcount += bp->max_count;
                        pvr2_trace(PVR2_TRACE_BUF_FLOW,
-                                  "/*---TRACE_FLOW---*/ bufferPool    "
-                                  " %8s cap cap=%07d cnt=%02d",
+                                  "/*---TRACE_FLOW---*/ bufferPool     %8s cap cap=%07d cnt=%02d",
                                   pvr2_buffer_state_decode(
                                           pvr2_buffer_state_idle),
                                   bp->stream->i_bcount,bp->stream->i_count);