I just tried applying the patch to 2.6.22.1 with your previous patch applied 
and get the following:

[EMAIL PROTECTED] linux-2.6.22.1]# patch -p1 < ../fix-sata_sil24
(Stripping trailing CRs from patch.)
patching file drivers/ata/sata_sil24.c
Hunk #1 succeeded at 1145 with fuzz 2 (offset 256 lines).
Hunk #2 FAILED at 1162.
Hunk #3 succeeded at 915 (offset -3 lines).
1 out of 3 hunks FAILED -- saving rejects to file drivers/ata/sata_sil24.c.rej
[EMAIL PROTECTED] linux-2.6.22.1]# vim ../sil24-irq-woc-fix.patch
[EMAIL PROTECTED] linux-2.6.22.1]# vim ../fix-sata_sil24
[EMAIL PROTECTED] linux-2.6.22.1]# make

[EMAIL PROTECTED] linux-2.6.22.1]# make modules
  CHK     include/linux/version.h
  CHK     include/linux/utsrelease.h
  CALL    scripts/checksyscalls.sh
  CC [M]  drivers/ata/sata_sil24.o
drivers/ata/sata_sil24.c: In function âsil24_host_intrâ:
drivers/ata/sata_sil24.c:1178: error: stray [EMAIL PROTECTED] in program
drivers/ata/sata_sil24.c:1178: error: âSILâ undeclared (first use in this 
function)
drivers/ata/sata_sil24.c:1178: error: (Each undeclared identifier is reported 
only once
drivers/ata/sata_sil24.c:1178: error: for each function it appears in.)
drivers/ata/sata_sil24.c:1178: error: expected â)â before â$_FLAG_PCIX_IRQ_WOCâ
make[2]: *** [drivers/ata/sata_sil24.o] Error 1
make[1]: *** [drivers/ata] Error 2
make: *** [drivers] Error 2



Thanks,
Jon

-----Original Message-----
From: Tejun Heo [mailto:[EMAIL PROTECTED] 
Sent: Saturday, September 22, 2007 8:25 PM
To: Jon Chelton
Cc: [email protected]
Subject: Re: sata_sil24 with port multiplier

Jon Chelton wrote:
> Thanks Tejun,
> 
> I have applied the patch and rebuilt the module.  I do not seem to be 
> getting the same error (although it only shows up during heavy disk 
> activity).  But now I am getting a lot of spurious interrupts.

Yeah, those are harmless and expected.  The attached patch should fix the 
problem.

--
tejun

-
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to