From: Jeff Garzik <jgarzik@pobox.com>
To: Mickael Marchand <marchand@kde.org>
Cc: linux-kernel@vger.kernel.org,
Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl>,
"Justin T. Gibbs" <gibbs@scsiguy.com>,
Hugo Mills <hugo-lkml@carfax.org.uk>
Subject: Re: [PATCH] adaptec 1210sa
Date: Tue, 30 Dec 2003 19:20:24 -0500 [thread overview]
Message-ID: <3FF21648.8030604@pobox.com> (raw)
In-Reply-To: <200312220305.29955.marchand@kde.org>
[-- Attachment #1: Type: text/plain, Size: 463 bytes --]
Mickael Marchand wrote:
> reading linux-scsi I found a suggestion by Justin to make adaptec's 1210 sa
> working. I made the corresponding patch for libata, and it actually works :)
>
> it needs some redesign to only apply to aar1210 (as standard sil3112 does not
> need it) and I guess some testing before inclusion.
Here is the patch I'm applying. Please test and let me know how it goes.
Also, someone please send me a patch for the PCI ids :)
Jeff
[-- Attachment #2: patch --]
[-- Type: text/plain, Size: 1033 bytes --]
===== drivers/scsi/sata_sil.c 1.3 vs edited =====
--- 1.3/drivers/scsi/sata_sil.c Tue Dec 16 19:16:55 2003
+++ edited/drivers/scsi/sata_sil.c Tue Dec 30 19:11:02 2003
@@ -40,6 +40,10 @@
enum {
sil_3112 = 0,
+ SIL_SYSCFG = 0x48,
+ SIL_MASK_IDE0_INT = (1 << 22),
+ SIL_MASK_IDE1_INT = (1 << 23),
+
SIL_IDE0_TF = 0x80,
SIL_IDE0_CTL = 0x8A,
SIL_IDE0_BMDMA = 0x00,
@@ -236,6 +240,7 @@
unsigned long base;
void *mmio_base;
int rc;
+ u32 tmp;
if (!printed_version++)
printk(KERN_DEBUG DRV_NAME " version " DRV_VERSION "\n");
@@ -295,6 +300,14 @@
probe_ent->port[1].bmdma_addr = base + SIL_IDE1_BMDMA;
probe_ent->port[1].scr_addr = base + SIL_IDE1_SCR;
ata_std_ports(&probe_ent->port[1]);
+
+ /* make sure IDE0/1 interrupts are not masked */
+ tmp = readl(mmio_base + SIL_SYSCFG);
+ if (tmp & (SIL_MASK_IDE0_INT | SIL_MASK_IDE1_INT)) {
+ tmp &= ~(SIL_MASK_IDE0_INT | SIL_MASK_IDE1_INT);
+ writel(tmp, mmio_base + SIL_SYSCFG);
+ readl(mmio_base + SIL_SYSCFG); /* flush */
+ }
pci_set_master(pdev);
next prev parent reply other threads:[~2003-12-31 0:20 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-12-22 2:05 [PATCH] adaptec 1210sa Mickael Marchand
2003-12-30 23:34 ` Jeff Garzik
2003-12-30 23:57 ` Justin T. Gibbs
2003-12-30 23:55 ` Jeff Garzik
2003-12-31 0:10 ` Hugo Mills
2003-12-30 23:51 ` Jeff Garzik
2003-12-31 0:20 ` Jeff Garzik [this message]
2003-12-31 11:25 ` Mickael Marchand
2004-01-03 20:46 ` Mickael Marchand
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=3FF21648.8030604@pobox.com \
--to=jgarzik@pobox.com \
--cc=B.Zolnierkiewicz@elka.pw.edu.pl \
--cc=gibbs@scsiguy.com \
--cc=hugo-lkml@carfax.org.uk \
--cc=linux-kernel@vger.kernel.org \
--cc=marchand@kde.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox