Just for future reference, git doesn't like patches in this format. What it's expecting is the change log in the actual email and the patch as an attachment. On Fri, 2009-10-30 at 21:53 +0800, jack_wang wrote: > @@ -1342,34 +1360,43 @@ static u32 mpi_msg_consume(struct pm8001_hba_info *pm8001_ha, > *pBC = (u8)((msgHeader_tmp >> 24) & > 0x1f); > PM8001_IO_DBG(pm8001_ha, > - pm8001_printk("mpi_msg_consume" > - ": CI=%d PI=%d msgHeader=%x\n", > + pm8001_printk(": CI=%d PI=%d " > + "msgHeader=%x\n", > circularQ->consumer_idx, > circularQ->producer_index, > msgHeader_tmp)); > return MPI_IO_STATUS_SUCCESS; > } else { > - u32 producer_index; > - void *pi_virt = circularQ->pi_virt; > - /* free the circular queue buffer > - elements associated with the message*/ > circularQ->consumer_idx = > (circularQ->consumer_idx + > - ((msgHeader_tmp >> 24) & 0x1f)) > - % 256; > + ((msgHeader_tmp >> 24) & 0x1f)) > + % 256; > + msgHeader_tmp = 0; > + pm8001_write_32(msgHeader, 0, 0); > /* update the CI of outbound queue */ > pm8001_cw32(pm8001_ha, > circularQ->ci_pci_bar, > circularQ->ci_offset, > circularQ->consumer_idx); > - /* Update the producer index from SPC */ > - producer_index = > - pm8001_read_32(pi_virt); > - circularQ->producer_index = > - cpu_to_le32(producer_index); > + > } > - } else > + } else { > + circularQ->consumer_idx = > + (circularQ->consumer_idx + > + ((msgHeader_tmp >> 24) & 0x1f)) % 256; > + msgHeader_tmp = 0; > + pm8001_write_32(msgHeader, 0, 0); > + /* update the CI of outbound queue */ > + pm8001_cw32(pm8001_ha, circularQ->ci_pci_bar, > + circularQ->ci_offset, > + circularQ->consumer_idx); > return MPI_IO_STATUS_FAIL; > + } > + } else { > + u32 producer_index; > + void *pi_virt = circularQ->pi_virt; > + /* Update the producer index from SPC */ > + producer_index = pm8001_read_32(pi_virt); > + circularQ->producer_index = cpu_to_le32(producer_index); > } > } while (circularQ->producer_index != circularQ->consumer_idx); > /* while we don't have any more not-yet-delivered message */ This hunk is refusing to apply: patching file drivers/scsi/pm8001/pm8001_hwi.c patch: **** malformed patch at line 164: /* while we don't have any more not-yet-delivered message */ The problem seems to be that the header promises there will be 43 lines after patching and there are actually 44 ... did you edit this patch after generating the diff? James -- 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