>Umm, you sure this is right? ide_build_sglist takes an ata_channel
>argument in my 2.5.20.
Right. As the order of the file names in the diff confirms,
I accidentally submitted a diff in reverse order. You are also
correct about:
>If this is reversed, you also forgot to change where it is used in
>icside.c.
Russell: sorry for not cc'ing you in my original patch
submission to Martin. I infer that since you are adding another patch
of your own and adding Martin to the recipient list that it is OK with
you to volunteer Martin to combine your patch and mine in this case
and submit them to Linus.
Martin: unless you, Russell, or anyone else sees a problem
with this, could you please also apply the attached patch to icside.c,
which I should have included in my original submission. I missed my
error when I checked for compiler warnings, because icside is not
built on x86.
Adam J. Richter __ ______________ 575 Oroville Road
adam@yggdrasil.com \ / Milpitas, California 95035
+1 408 309-6081 | g g d r a s i l United States of America
"Free Software For The Rest Of Us."
--- before/drivers/ide/icside.c 2002-06-02 18:44:41.000000000 -0700
+++ linux/drivers/ide/icside.c 2002-06-04 15:31:21.000000000 -0700
@@ -491,7 +492,7 @@
*/
BUG_ON(dma_channel_active(ch->hw.dma));
- count = ch->sg_nents = ide_build_sglist(ch, rq);
+ count = ch->sg_nents = ide_build_sglist(drive, rq);
if (!count)
return 1;
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/