On Wed, 24 Oct 2007 19:48:26 -0400 (EDT) Jeff Garzik <jeff@xxxxxxxxxx> wrote: > drivers/scsi/ips.c | 178 ++++++++++++++++++++++++---------------------------- this driver seems a bit of a basket case :( What's going on here? scb->dcdb.cmd_attribute = ips_command_direction[scb->scsi_cmd->cmnd[0]]; /* Allow a WRITE BUFFER Command to Have no Data */ /* This is Used by Tape Flash Utilites */ if ((scb->scsi_cmd->cmnd[0] == WRITE_BUFFER) && (scb->data_len == 0)) scb->dcdb.cmd_attribute = 0; if (!(scb->dcdb.cmd_attribute & 0x3)) scb->dcdb.transfer_length = 0; if (scb->data_len >= IPS_MAX_XFER) { I hope that's just busted indentation and not a missing {} block. - To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html