RE: [PATCH] fix idiocy in asd_init_lseq_mdp()

From: Hammer, Jack
Date: Mon Sep 25 2006 - 15:57:24 EST


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-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/