Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 08 Nov 2007 13:41:06 +0300
From:      Alexander Sabourenkov <screwdriver@lxnt.info>
To:        =?UTF-8?B?U8O4cmVuIFNjaG1pZHQ=?= <sos@deepcore.dk>
Cc:        Nathan Butcher <n-butcher@fusiongol.com>, freebsd-current@freebsd.org
Subject:   Re: Remaining SATA (and other) issues
Message-ID:  <4732E7C2.3030403@lxnt.info>
In-Reply-To: <4732E37F.9020707@deepcore.dk>
References:  <47326FB8.50602@fusiongol.com> <4732CEE3.3070003@lxnt.info> <4732DA32.3090601@deepcore.dk> <4732E18A.6040802@lxnt.info> <4732E37F.9020707@deepcore.dk>

next in thread | previous in thread | raw e-mail | index | archive | help
Søren Schmidt wrote:
> As I said ATA (as in my driver) uses this promise general purpose 
> register to store interrupts into, it is very much needed.
> The reason is that on some promise chips the interrupts are reset on 
> read, so I can only read the status *once* but I need it several times.

If I understand correctly you use it as a controller-local variable.

> Oh yes it does look for the "stat_reg" in ata_promise_mio_status() which 
> does the interrupt status getting etc...

It seems like I now need to understand how does ata_promise_mio_status() 
gets called on each interrupt.
I'll leave this until I have time to read the code and test your patch too.

> Good question, their docs says nothing about it actually, I suppose this 
> is more a programming style question than anything else, anyhow it will 
> only change evt behavior until the first interrupt, then I'll write the 
> entire reg anyways :)

Well, so be it.

-- 

./lxnt



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4732E7C2.3030403>