From: Sergei Shtylyov <sshtylyov@ru.mvista.com> To: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> Cc: Andrew Morton <akpm@osdl.org>, Olaf Hering <olaf@aepfle.de>, Jeff Garzik <jeff@garzik.org>, linux-ide@vger.kernel.org, linuxppc-dev@ozlabs.org, Paul Mackeras <paulus@samba.org> Subject: Re: [PATCH] use winbond libata instead of ide driver for pseries CD drives Date: Fri, 16 Feb 2007 18:11:06 +0300 [thread overview] Message-ID: <45D5C98A.70302@ru.mvista.com> (raw) In-Reply-To: <45D5C85B.7070201@ru.mvista.com> Hello, I wrote: >>>>> I'm sorry, how those 2 drivers can be equivalent?! The pata_winbond driver >>>>>is for VLB only, sl81c105 was for PCI only -- you certainly want to use >>>>>pata_sl82c105 (indeed better version of sl82c105 driver). >>>>I was under the impression that with your fixes these drivers >>>>(sl82c105 and pata_sl82c105) are functionally equivalent. >>> They're not yet equivalent. pata_sl82c105 supports all MWDMA modes, >>>sl82c105 still supports only MWDMA2. >>Indeed and it seems that IDE core needs ->dma_mode to fix this. > Not really. It just needs the proper DMA timings to be set by adding the > missing speedproc() method (and also DMA timings possibly merged with PIO like > libata does generally -- except in this driver). By "this driver" I meant pata_sl82c105 of course. :-) >>Thanks, >>Bart MBR, Sergei
WARNING: multiple messages have this Message-ID (diff)
From: Sergei Shtylyov <sshtylyov@ru.mvista.com> To: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> Cc: Andrew Morton <akpm@osdl.org>, Olaf Hering <olaf@aepfle.de>, Jeff Garzik <jeff@garzik.org>, linuxppc-dev@ozlabs.org, linux-ide@vger.kernel.org, Paul Mackeras <paulus@samba.org> Subject: Re: [PATCH] use winbond libata instead of ide driver for pseries CD drives Date: Fri, 16 Feb 2007 18:11:06 +0300 [thread overview] Message-ID: <45D5C98A.70302@ru.mvista.com> (raw) In-Reply-To: <45D5C85B.7070201@ru.mvista.com> Hello, I wrote: >>>>> I'm sorry, how those 2 drivers can be equivalent?! The pata_winbond driver >>>>>is for VLB only, sl81c105 was for PCI only -- you certainly want to use >>>>>pata_sl82c105 (indeed better version of sl82c105 driver). >>>>I was under the impression that with your fixes these drivers >>>>(sl82c105 and pata_sl82c105) are functionally equivalent. >>> They're not yet equivalent. pata_sl82c105 supports all MWDMA modes, >>>sl82c105 still supports only MWDMA2. >>Indeed and it seems that IDE core needs ->dma_mode to fix this. > Not really. It just needs the proper DMA timings to be set by adding the > missing speedproc() method (and also DMA timings possibly merged with PIO like > libata does generally -- except in this driver). By "this driver" I meant pata_sl82c105 of course. :-) >>Thanks, >>Bart MBR, Sergei
next prev parent reply other threads:[~2007-02-16 15:11 UTC|newest] Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top 2007-02-10 20:35 [PATCH] move variables in drivers/macintosh to bss Olaf Hering 2007-02-10 20:36 ` [PATCH] add delay around sl82c105_reset_engine calls Olaf Hering 2007-02-10 20:37 ` [PATCH] cleanup variable usage in mesh interrupt handler Olaf Hering 2007-02-10 20:38 ` [PATCH] mark winbond IDE PCI resources with start 0 as unassigned Olaf Hering 2007-02-10 20:40 ` [PATCH] correct AC Power: in /proc/pmu/info on ibook1 Olaf Hering 2007-02-15 23:09 ` [PATCH] add delay around sl82c105_reset_engine calls Jeff Garzik 2007-02-16 9:20 ` [PATCH] use winbond libata instead of ide driver for pseries CD drives Olaf Hering 2007-02-16 13:52 ` Sergei Shtylyov 2007-02-16 13:52 ` Sergei Shtylyov 2007-02-16 14:51 ` Sergei Shtylyov 2007-02-16 14:53 ` Bartlomiej Zolnierkiewicz 2007-02-16 14:53 ` Bartlomiej Zolnierkiewicz 2007-02-16 14:55 ` Sergei Shtylyov 2007-02-16 14:55 ` Sergei Shtylyov 2007-02-16 15:09 ` Bartlomiej Zolnierkiewicz 2007-02-16 15:09 ` Bartlomiej Zolnierkiewicz 2007-02-16 15:06 ` Sergei Shtylyov 2007-02-16 15:06 ` Sergei Shtylyov 2007-02-16 15:11 ` Sergei Shtylyov [this message] 2007-02-16 15:11 ` Sergei Shtylyov 2007-02-16 21:41 ` Benjamin Herrenschmidt 2007-02-16 14:46 ` Bartlomiej Zolnierkiewicz 2007-02-16 14:46 ` Bartlomiej Zolnierkiewicz 2007-02-16 14:51 ` Bartlomiej Zolnierkiewicz 2007-02-16 14:51 ` Bartlomiej Zolnierkiewicz 2007-02-16 17:37 ` Olaf Hering 2007-02-16 17:37 ` Olaf Hering 2007-02-17 20:36 ` Jeff Garzik 2007-02-17 20:36 ` 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=45D5C98A.70302@ru.mvista.com \ --to=sshtylyov@ru.mvista.com \ --cc=akpm@osdl.org \ --cc=bzolnier@gmail.com \ --cc=jeff@garzik.org \ --cc=linux-ide@vger.kernel.org \ --cc=linuxppc-dev@ozlabs.org \ --cc=olaf@aepfle.de \ --cc=paulus@samba.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: linkBe 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.