All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jeff Garzik <jgarzik@pobox.com>
To: Pat LaVarre <p.lavarre@ieee.org>
Cc: linux-ide@vger.kernel.org
Subject: Re: [PATCH] libata atapi work #5
Date: Fri, 14 May 2004 15:09:16 -0400	[thread overview]
Message-ID: <40A5195C.2000402@pobox.com> (raw)
In-Reply-To: <1084560167.8752.7.camel@patibmrh9>

Pat LaVarre wrote:
>>-				if (qc && ((qc->flags & ATA_QCFLAG_POLL) == 0))
>>+				if (qc && (!(qc->tf.ctl & ATA_NIEN)))
>>...
>>-	ATA_QCFLAG_POLL		= (1 << 5), /* polling, no interrupts */
> 
> 
> Back in PATA, when I set x02 ATA_NIEN, I mis/remember I saw INTRQ float,
> rather than becoming reliably deasserted.
> 
> Is life better in SATA?

On pure SATA, you just have a bit in a FIS, as I mentioned in one of the 
other emails (in this current email flurry :)).

On bridged SATA, where either the host controller or the device, or 
both, have an on-device SATA<->PATA bridge, the bridge must deal with 
INTRQ.  On the whole it is _much_ more clean.

Read up in the SATA docs on the difference between a "command FIS" and a 
"control FIS".  Here's a quick intro...

Both are Host-to-Device Register FIS's.  The former is sent to the 
device when the Command shadow register is written.  The latter is sent 
to the device when the Device Control register is written.  Both types 
of H2D Register FIS, command and control, send a complete copy of the 
taskfile shadow registers to the device at that time.

Reading your emails, you sound like a "hardware guy" to me :)  SATA is 
_very_ different from PATA at the hardware level.  What used to be chip 
selects and signals are now bits in a data structure that is sent -- 
much like an ethernet frame -- over a serial link.

	Jeff




  reply	other threads:[~2004-05-14 19:09 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-05-14 18:42 [PATCH] libata atapi work #5 Pat LaVarre
2004-05-14 19:09 ` Jeff Garzik [this message]
2004-05-14 19:27   ` Pat LaVarre
2004-05-14 19:47     ` Jeff Garzik
2004-05-14 20:00       ` Pat LaVarre
2004-05-14 20:14         ` Leon Woestenberg
2004-05-14 20:15         ` Jeff Garzik
2004-05-14 20:23           ` Leon Woestenberg
2004-05-14 20:29             ` Jeff Garzik
2004-05-14 23:09           ` Pat LaVarre
2004-05-15  0:01           ` Jeff Garzik
  -- strict thread matches above, loose matches on Subject: below --
2004-05-14 18:26 Jeff Garzik

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=40A5195C.2000402@pobox.com \
    --to=jgarzik@pobox.com \
    --cc=linux-ide@vger.kernel.org \
    --cc=p.lavarre@ieee.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.