Subject: [PATCH 0/19] new IDE quilt tree


Hi,

My working IDE tree (against Linus' tree) now resides here:

http://kernel.org/pub/linux/kernel/people/bart/pata-2.6/patches/

and is managed by quilt.


Currently it contains the following changes:

* all IDE patches from 2.6.20-rc3-mm1
(great work done by Sergei Shtylyov <[email protected]>)

* atiixp fixes [in -mm already]
(from Conke Hu <[email protected]>)

* pci/generic update for jmicron [in -mm already]
(from Alan Cox <[email protected]>)

* new IT8213 driver
(from Jack Lee <[email protected]> and Alan Cox, some fixes by me)

* add pci_get_legacy_ide_irq() to ia64 [fixes amd74xx build] (by me)

* add missing __init tags to IDE PCI host drivers (by me)

* bunch of cleanups (by me)

diffstat:
51 files changed, 1918 insertions(+), 1498 deletions(-)


I'm sending only new patches for review/comments.

If you would like to see the full quilt series (or to get combined patch)
against 2.6.20-rc3-mm1 / 2.6.20-rc4, they are available here:

http://kernel.org/pub/linux/kernel/people/bart/pata-2.6/releases/

Thanks,
Bart


Subject: [PATCH 1/19] ide: update MAINTAINERS entry

[PATCH] ide: update MAINTAINERS entry

Signed-off-by: Bartlomiej Zolnierkiewicz <[email protected]>
---
MAINTAINERS | 7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)

Index: my-2.6/MAINTAINERS
===================================================================
--- my-2.6.orig/MAINTAINERS
+++ my-2.6/MAINTAINERS
@@ -1585,12 +1585,11 @@ M: [email protected]
W: http://www.developer.ibm.com/welcome/netfinity/serveraid.html
S: Supported

-IDE DRIVER [GENERAL]
+IDE SUBSYSTEM
P: Bartlomiej Zolnierkiewicz
-M: [email protected]
-L: [email protected]
+M: [email protected]
L: [email protected]
-T: git kernel.org:/pub/scm/linux/kernel/git/bart/ide-2.6.git
+T: quilt kernel.org/pub/linux/kernel/people/bart/pata-2.6/
S: Maintained

IDE/ATAPI CDROM DRIVER

2007-01-12 10:01:54

by Alan

[permalink] [raw]
Subject: Re: [PATCH 0/19] new IDE quilt tree

Generally looks very good - couple of odd items I noticed I've send
replies about. Good to have someone working on the old IDE code again.

Alan

2007-01-13 16:59:49

by Mark Lord

[permalink] [raw]
Subject: [PATCH] IDE Driver for Delkin/Lexar/etc.. cardbus CF adapter

On Thursday 11 January 2007 23:17, Bartlomiej Zolnierkiewicz wrote:
>
> My working IDE tree (against Linus' tree) now resides here:
>
> http://kernel.org/pub/linux/kernel/people/bart/pata-2.6/patches/

Bart, here's a driver I've been keeping out-of-tree for the past couple of years.
This is for the Delking/Lexar/ASKA/etc.. 32-bit cardbus IDE CompactFlash adapter card.

It's probably way out of sync with the latest driver model (??), but it still builds/works.
I'm not interested in doing much of a rewrite, other than for libata someday,
as I no longer use the card myself.

But lots of other people do seem to use it, so it might be nice to see it "in-tree".

Signed-off-by: Mark Lord <[email protected]>

---
--- linux-2.6.11/drivers/ide/Kconfig 2005-03-02 10:07:37.000000000 -0500
+++ linux/drivers/ide/Kconfig 2005-03-02 10:11:54.000000000 -0500
@@ -166,6 +166,13 @@
Support for outboard IDE disks, tape drives, and CD-ROM drives
connected through a CARDBUS card.

+config BLK_DEV_DELKIN
+ tristate "Cardbus IDE support (Delkin/ASKA/Workbit)"
+ depends on CARDBUS && PCI
+ help
+ Support for Delkin, ASKA, and Workbit Cardbus CompactFlash
+ Adapters. This may also work for similar SD and XD adapters.
+
config BLK_DEV_IDECD
tristate "Include IDE/ATAPI CDROM support"
---help---
--- linux-2.6.11/drivers/ide/pci/delkin_cb.c 1969-12-31 19:00:00.000000000 -0500
+++ linux/drivers/ide/pci/delkin_cb.c 2005-02-14 11:58:59.000000000 -0500
@@ -0,0 +1,140 @@
+/*
+ * linux/drivers/ide/pci/delkin_cb.c
+ *
+ * Created 20 Oct 2004 by Mark Lord
+ *
+ * Basic support for Delkin/ASKA/Workbit Cardbus CompactFlash adapter
+ *
+ * Modeled after the 16-bit PCMCIA driver: ide-cs.c
+ *
+ * This is slightly peculiar, in that it is a PCI driver,
+ * but is NOT an IDE PCI driver -- the IDE layer does not directly
+ * support hot insertion/removal of PCI interfaces, so this driver
+ * is unable to use the IDE PCI interfaces. Instead, it uses the
+ * same interfaces as the ide-cs (PCMCIA) driver uses.
+ * On the plus side, the driver is also smaller/simpler this way.
+ *
+ * This file is subject to the terms and conditions of the GNU General Public
+ * License. See the file COPYING in the main directory of this archive for
+ * more details.
+ */
+#include <linux/autoconf.h>
+#include <linux/types.h>
+#include <linux/module.h>
+#include <linux/mm.h>
+#include <linux/blkdev.h>
+#include <linux/hdreg.h>
+#include <linux/ide.h>
+#include <linux/init.h>
+#include <linux/pci.h>
+#include <asm/io.h>
+
+/*
+ * No chip documentation has yet been found,
+ * so these configuration values were pulled from
+ * a running Win98 system using "debug".
+ * This gives around 3MByte/second read performance,
+ * which is about 2/3 of what the chip is capable of.
+ *
+ * There is also a 4KByte mmio region on the card,
+ * but its purpose has yet to be reverse-engineered.
+ */
+static const u8 setup[] = {
+ 0x00, 0x05, 0xbe, 0x01, 0x20, 0x8f, 0x00, 0x00,
+ 0xa4, 0x1f, 0xb3, 0x1b, 0x00, 0x00, 0x00, 0x80,
+ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+ 0x00, 0x00, 0x00, 0x00, 0xa4, 0x83, 0x02, 0x13,
+};
+
+static int __devinit
+delkin_cb_probe (struct pci_dev *dev, const struct pci_device_id *id)
+{
+ unsigned long base;
+ hw_regs_t hw;
+ ide_hwif_t *hwif = NULL;
+ ide_drive_t *drive;
+ int i, rc;
+
+ rc = pci_enable_device(dev);
+ if (rc) {
+ printk(KERN_ERR "delkin_cb: pci_enable_device failed (%d)\n", rc);
+ return rc;
+ }
+ rc = pci_request_regions(dev, "delkin_cb");
+ if (rc) {
+ printk(KERN_ERR "delkin_cb: pci_request_regions failed (%d)\n", rc);
+ pci_disable_device(dev);
+ return rc;
+ }
+ base = pci_resource_start(dev, 0);
+ outb(0x02, base + 0x1e); /* set nIEN to block interrupts */
+ inb(base + 0x17); /* read status to clear interrupts */
+ for (i = 0; i < sizeof(setup); ++i) {
+ if (setup[i])
+ outb(setup[i], base + i);
+ }
+ pci_release_regions(dev); /* IDE layer handles regions itself */
+
+ memset(&hw, 0, sizeof(hw));
+ ide_std_init_ports(&hw, base + 0x10, base + 0x1e);
+ hw.irq = dev->irq;
+ hw.chipset = ide_pci; /* this enables IRQ sharing */
+
+ rc = ide_register_hw_with_fixup(&hw, &hwif, ide_undecoded_slave);
+ if (rc < 0) {
+ printk(KERN_ERR "delkin_cb: ide_register_hw failed (%d)\n", rc);
+ pci_disable_device(dev);
+ return -ENODEV;
+ }
+ pci_set_drvdata(dev, hwif);
+ hwif->pci_dev = dev;
+ drive = &hwif->drives[0];
+ if (drive->present) {
+ drive->io_32bit = 1;
+ drive->unmask = 1;
+ }
+ return 0;
+}
+
+static void
+delkin_cb_remove (struct pci_dev *dev)
+{
+ ide_hwif_t *hwif = pci_get_drvdata(dev);
+
+ if (hwif)
+ ide_unregister(hwif->index);
+ pci_disable_device(dev);
+}
+
+static struct pci_device_id delkin_cb_pci_tbl[] __devinitdata = {
+ { 0x1145, 0xf021, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0},
+ { 0, },
+};
+MODULE_DEVICE_TABLE(pci, delkin_cb_pci_tbl);
+
+static struct pci_driver driver = {
+ .name = "Delkin-ASKA-Workbit Cardbus IDE",
+ .id_table = delkin_cb_pci_tbl,
+ .probe = delkin_cb_probe,
+ .remove = delkin_cb_remove,
+};
+
+static int
+delkin_cb_init (void)
+{
+ return pci_module_init(&driver);
+}
+
+static void
+delkin_cb_exit (void)
+{
+ pci_unregister_driver(&driver);
+}
+
+module_init(delkin_cb_init);
+module_exit(delkin_cb_exit);
+
+MODULE_AUTHOR("Mark Lord");
+MODULE_DESCRIPTION("Basic support for Delkin/ASKA/Workbit Cardbus IDE");
+MODULE_LICENSE("GPL");
+
--- linux-2.6.11/drivers/ide/pci/Makefile 2005-03-02 10:07:37.000000000 -0500
+++ linux/drivers/ide/pci/Makefile 2005-03-02 10:11:54.000000000 -0500
@@ -8,6 +8,7 @@
obj-$(CONFIG_BLK_DEV_CS5530) += cs5530.o
obj-$(CONFIG_BLK_DEV_SC1200) += sc1200.o
obj-$(CONFIG_BLK_DEV_CY82C693) += cy82c693.o
+obj-$(CONFIG_BLK_DEV_DELKIN) += delkin_cb.o
obj-$(CONFIG_BLK_DEV_HPT34X) += hpt34x.o
obj-$(CONFIG_BLK_DEV_HPT366) += hpt366.o
#obj-$(CONFIG_BLK_DEV_HPT37X) += hpt37x.o

Subject: Re: [PATCH] IDE Driver for Delkin/Lexar/etc.. cardbus CF adapter

On 1/13/07, Mark Lord <[email protected]> wrote:
> On Thursday 11 January 2007 23:17, Bartlomiej Zolnierkiewicz wrote:
> >
> > My working IDE tree (against Linus' tree) now resides here:
> >
> > http://kernel.org/pub/linux/kernel/people/bart/pata-2.6/patches/
>
> Bart, here's a driver I've been keeping out-of-tree for the past couple of years.
> This is for the Delking/Lexar/ASKA/etc.. 32-bit cardbus IDE CompactFlash adapter card.
>
> It's probably way out of sync with the latest driver model (??), but it still builds/works.
> I'm not interested in doing much of a rewrite, other than for libata someday,
> as I no longer use the card myself.
>
> But lots of other people do seem to use it, so it might be nice to see it "in-tree".
>
> Signed-off-by: Mark Lord <[email protected]>

Thanks, I applied it to IDE tree.

[ I must have been really somehow fixated on IDE unregistering/ordering mess
not to merge it earlier. Sorry for that. ]

2007-01-19 15:46:34

by Sergei Shtylyov

[permalink] [raw]
Subject: Re: [PATCH 1/19] ide: update MAINTAINERS entry

Hello.

Bartlomiej Zolnierkiewicz wrote:
> [PATCH] ide: update MAINTAINERS entry

> Signed-off-by: Bartlomiej Zolnierkiewicz <[email protected]>
> ---
> MAINTAINERS | 7 +++----
> 1 file changed, 3 insertions(+), 4 deletions(-)

> Index: my-2.6/MAINTAINERS
> ===================================================================
> --- my-2.6.orig/MAINTAINERS
> +++ my-2.6/MAINTAINERS
> @@ -1585,12 +1585,11 @@ M: [email protected]
> W: http://www.developer.ibm.com/welcome/netfinity/serveraid.html
> S: Supported
>
> -IDE DRIVER [GENERAL]
> +IDE SUBSYSTEM
> P: Bartlomiej Zolnierkiewicz
> -M: [email protected]
> -L: [email protected]
> +M: [email protected]
> L: [email protected]
> -T: git kernel.org:/pub/scm/linux/kernel/git/bart/ide-2.6.git
> +T: quilt kernel.org/pub/linux/kernel/people/bart/pata-2.6/
> S: Maintained

BTW, if I'm (or somebody else) sending patches with IDE driver *fixes*,
should these be against that quilt tree (containing largely reworked IDE
code), against fresh Linus' tree (or some recent -rc), or against recent -mm tree?

WBR, Sergei