All of lore.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 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.