From: Randy Dunlap <randy.dunlap@oracle.com>
To: Sergei Shtylyov <sshtylyov@ru.mvista.com>
Cc: akpm@osdl.org, bzolnier@gmail.com, linux-ide@vger.kernel.org,
linux-kernel@vger.kernel.org, alan@lxorguk.ukuu.org.uk
Subject: Re: [PATCH 2.6.19-rc1] Toshiba TC86C001 IDE driver
Date: Tue, 12 Dec 2006 15:00:52 -0800 [thread overview]
Message-ID: <20061212150052.b05b05db.randy.dunlap@oracle.com> (raw)
In-Reply-To: <200612130148.34539.sshtylyov@ru.mvista.com>
On Wed, 13 Dec 2006 01:48:34 +0300 Sergei Shtylyov wrote:
> Behold! This is the driver for the Toshiba TC86C001 GOKU-S IDE controller,
> completely reworked from the original brain-damaged Toshiba's 2.4 version.
>
> This single channel UltraDMA/66 controller is very simple in programming, yet
> Toshiba managed to plant many interesting bugs in it. The particularly nasty
> "limitation 5" (as they call the errata) caused me to abuse the IDE core in a
> possibly most interesting way so far. However, this is still better than the
> #ifdef mess in drivers/ide/ide-io.c that the original version included (well,
> it had much more mess)...
>
> Signed-off-by: Sergei Shtylyov <sshtylyov@ru.mvista.com>
>
> drivers/ide/Kconfig | 5
> drivers/ide/pci/Makefile | 1
> drivers/ide/pci/tc86c001.c | 304 +++++++++++++++++++++++++++++++++++++++++++++
> drivers/pci/quirks.c | 18 ++
> include/linux/pci_ids.h | 1
> 5 files changed, 329 insertions(+)
>
> Index: linux-2.6/drivers/ide/Kconfig
> ===================================================================
> --- linux-2.6.orig/drivers/ide/Kconfig
> +++ linux-2.6/drivers/ide/Kconfig
> @@ -742,6 +742,11 @@ config BLK_DEV_VIA82CXXX
> This allows the kernel to change PIO, DMA and UDMA speeds and to
> configure the chip to optimum performance.
>
> +config BLK_DEV_TC86C001
> + tristate "Toshiba TC86C001 support"
Needs something here like lots of other IDE PCI drivers have:
depends on PCI && BLK_DEV_IDEPCI
or at least: depends on PCI
> + help
> + This driver adds support for Toshiba TC86C001 GOKU-S chip.
> +
> endif
---
~Randy
next prev parent reply other threads:[~2006-12-12 23:00 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-12-12 22:48 [PATCH 2.6.19-rc1] Toshiba TC86C001 IDE driver Sergei Shtylyov
2006-12-12 22:57 ` Sergei Shtylyov
2006-12-12 23:00 ` Randy Dunlap [this message]
2006-12-12 23:07 ` Sergei Shtylyov
2006-12-12 23:12 ` Randy Dunlap
2006-12-12 23:41 ` Alan
2006-12-13 1:09 ` Bartlomiej Zolnierkiewicz
2006-12-13 11:32 ` Alan
2006-12-13 14:01 ` Bartlomiej Zolnierkiewicz
2006-12-13 14:16 ` Alan
2006-12-13 14:19 ` Bartlomiej Zolnierkiewicz
2006-12-13 14:16 ` Sergei Shtylyov
2006-12-13 14:53 ` Alan
2006-12-13 0:40 ` Alan
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=20061212150052.b05b05db.randy.dunlap@oracle.com \
--to=randy.dunlap@oracle.com \
--cc=akpm@osdl.org \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=bzolnier@gmail.com \
--cc=linux-ide@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=sshtylyov@ru.mvista.com \
/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.