public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: "Philipp E, Imhof" <fippu@fippu.ch>
To: linux-kernel@vger.kernel.org
Subject: [PATCH] it8212.c, kernel 2.6.9-ac16
Date: Fri, 17 Dec 2004 15:01:26 +0100	[thread overview]
Message-ID: <41C2E6B6.20701@fippu.ch> (raw)

Hello!

I am using an ITE 8211 controller that did not work with
2.6.9-ac16, because of the PCI device ID.

I created a trivial (but useful) patch that adds the device
ID to the driver and a small comment for people grepping the
sources for the controller type.

I do not know whether the 8212 and the 8211 are fully
compatible, but in my computer, 8211 works fine with 8212's
driver.


diff -ur linux/drivers/ide/pci/it8212.c
linux-2.6.9-ite/drivers/ide/pci/it8212.c
--- linux/drivers/ide/pci/it8212.c	2004-12-17 14:43:39.000000000
+0100
+++ linux-2.6.9-ite/drivers/ide/pci/it8212.c	2004-12-17
14:48:43.000000000 +0100
@@ -16,6 +16,8 @@
   *  as an IDE controller. Smart mode only understands DMA
read/write and
   *  identify, none of the fancier commands apply.
   *
+ *  This driver also supports ITE8211.
+ *
   *  Errata:
   *  o	Rev 0x10 also requires master/slave hold the same DMA
timings and
   *	cannot do ATAPI MWDMA.
@@ -831,6 +833,7 @@
  }

  static struct pci_device_id it8212_pci_tbl[] = {
+	{ PCI_VENDOR_ID_ITE, PCI_DEVICE_ID_ITE_8211,  PCI_ANY_ID,
PCI_ANY_ID, 0, 0, 0},
  	{ PCI_VENDOR_ID_ITE, PCI_DEVICE_ID_ITE_8212,  PCI_ANY_ID,
PCI_ANY_ID, 0, 0, 0},
  	{ 0, },
  };
diff -ur linux/include/linux/pci_ids.h
linux-2.6.9-ite/include/linux/pci_ids.h
--- linux/include/linux/pci_ids.h	2004-12-17 14:43:39.000000000 +0100
+++ linux-2.6.9-ite/include/linux/pci_ids.h	2004-12-17
14:30:01.000000000 +0100
@@ -1659,6 +1659,7 @@
  #define PCI_VENDOR_ID_ITE		0x1283
  #define PCI_DEVICE_ID_ITE_IT8172G	0x8172
  #define PCI_DEVICE_ID_ITE_IT8172G_AUDIO 0x0801
+#define PCI_DEVICE_ID_ITE_8211		0x8211
  #define PCI_DEVICE_ID_ITE_8212		0x8212
  #define PCI_DEVICE_ID_ITE_8872		0x8872
  #define PCI_DEVICE_ID_ITE_IT8330G_0	0xe886


--- END ---

Thank you very much for all your work.

Kind regards

Philipp Imhof, fippu@fippu.ch

PS: Please cc: me for comments, feedback or flames, as I
am not subscribed to the list (shame!) due to my lack
of knowledge...



             reply	other threads:[~2004-12-17 13:02 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-12-17 14:01 Philipp E, Imhof [this message]
2004-12-20 14:56 ` [PATCH] it8212.c, kernel 2.6.9-ac16 Alan Cox
  -- strict thread matches above, loose matches on Subject: below --
2004-12-27  3:16 Donald.Huang

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=41C2E6B6.20701@fippu.ch \
    --to=fippu@fippu.ch \
    --cc=linux-kernel@vger.kernel.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