James, asd_write_reg_dword() is the correct implementation for writing the LmM0INTEN_MASK to the LmSEQ_INTEN_SAVE register. Jack -----Original Message----- From: James Bottomley [mailto:James.Bottomley@xxxxxxxxxxxx] Sent: Monday, September 25, 2006 1:43 PM To: Hammer, Jack; Al Viro Cc: Luben Tuikov; dougg@xxxxxxxxxx; linux-scsi@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx Subject: Re: [PATCH] fix idiocy in asd_init_lseq_mdp() On Mon, 2006-09-25 at 18:39 +0100, Al Viro wrote: > Far more interesting question: where does the hardware expect to see > the upper 16 bits of that 32bit value? Which one it is - > LmSEQ_INTEN_SAVE(lseq) > ori LmSEQ_INTEN_SAVE(lseq) + 2? I don't honestly know. The change was made as part of a slew of changes by Robert Tarte at Adaptec to make the driver run on Big Endian platforms. I've copied Jack Hammer who's now looking after it in the hope that he can enlighten us. 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