]> pilppa.org Git - linux-2.6-omap-h63xx.git/commitdiff
Merge master.kernel.org:/pub/scm/linux/kernel/git/brodo/pcmcia-2.6
authorLinus Torvalds <torvalds@g5.osdl.org>
Sun, 2 Apr 2006 19:49:59 +0000 (12:49 -0700)
committerLinus Torvalds <torvalds@g5.osdl.org>
Sun, 2 Apr 2006 19:49:59 +0000 (12:49 -0700)
* master.kernel.org:/pub/scm/linux/kernel/git/brodo/pcmcia-2.6: (33 commits)
  [PATCH] pcmcia: declare pccard_iodyn_ops (fix m8xx_pcmcia.c compilation error)
  [PATCH] pcmcia: fix pcmcia_device_remove oops
  [PATCH] pcmcia: Add support for Possio GCC AKA PCMCIA Siemens MC45
  [PATCH] pcmcia: pseudo device handling update
  [PATCH] pcmcia: convert DEV_OK to pcmcia_dev_present
  [PATCH] pcmcia: use bitfield instead of p_state and state
  [PATCH] pcmcia: remove unused p_dev->state flags
  [PATCH] pcmcia: make pcmcia_release_{io,irq} static
  [PATCH] pcmcia: add return value to _config() functions
  [PATCH] pcmcia: remove dev_link_t and client_handle_t indirection
  [PATCH] pcmcia: embed dev_link_t into struct pcmcia_device
  [PATCH] pcmcia: rename pcmcia_device.state
  [PATCH] pcmcia: remove unneeded Vcc pseudo setting
  [PATCH] pcmcia: remove export of pcmcia_release_configuration
  [PATCH] pcmcia: default suspend and resume handling
  [PATCH] pcmcia: convert remaining users of pcmcia_release_io and _irq
  [PATCH] pcmcia: add pcmcia_disable_device
  [PATCH] serial_cs: add Merlin U630 IDs
  [PATCH] pcmcia: AT91RM9200 Compact Flash driver
  [PATCH] pcmcia: socket.functions starts with 1
  ...

1  2 
drivers/mtd/maps/pcmciamtd.c
drivers/net/pcmcia/xirc2ps_cs.c

index 8bbc751a60218eed3c017c87001631a62b084df2,a01e04bf484a3d38ca4748a5fa8339eabc0fa2f1..d27f4129afd3c952b5e5f33d85cec2e73d41584a
@@@ -54,7 -54,7 +54,7 @@@ static const int debug = 0
  #define MAX_PCMCIA_ADDR       0x4000000
  
  struct pcmciamtd_dev {
-       dev_link_t      link;           /* PCMCIA link */
+       struct pcmcia_device    *p_dev;
        dev_node_t      node;           /* device node */
        caddr_t         win_base;       /* ioremapped address of PCMCIA window */
        unsigned int    win_size;       /* size of window */
@@@ -111,8 -111,8 +111,8 @@@ static caddr_t remap_window(struct map_
        memreq_t mrq;
        int ret;
  
-       if(!(dev->link.state & DEV_PRESENT)) {
-               DEBUG(1, "device removed state = 0x%4.4X", dev->link.state);
+       if (!pcmcia_dev_present(dev->p_dev)) {
+               DEBUG(1, "device removed");
                return 0;
        }
  
                      dev->offset, mrq.CardOffset);
                mrq.Page = 0;
                if( (ret = pcmcia_map_mem_page(win, &mrq)) != CS_SUCCESS) {
-                       cs_error(dev->link.handle, MapMemPage, ret);
+                       cs_error(dev->p_dev, MapMemPage, ret);
                        return NULL;
                }
                dev->offset = mrq.CardOffset;
@@@ -238,7 -238,7 +238,7 @@@ static void pcmcia_copy_to_remap(struc
  
  /* read/write{8,16} copy_{from,to} routines with direct access */
  
- #define DEV_REMOVED(x)  (!(*(u_int *)x->map_priv_1 & DEV_PRESENT))
+ #define DEV_REMOVED(x)  (!(pcmcia_dev_present(((struct pcmciamtd_dev *)map->map_priv_1)->p_dev)))
  
  static map_word pcmcia_read8(struct map_info *map, unsigned long ofs)
  {
@@@ -319,7 -319,7 +319,7 @@@ static void pcmcia_copy_to(struct map_i
  static void pcmciamtd_set_vpp(struct map_info *map, int on)
  {
        struct pcmciamtd_dev *dev = (struct pcmciamtd_dev *)map->map_priv_1;
-       dev_link_t *link = &dev->link;
+       struct pcmcia_device *link = dev->p_dev;
        modconf_t mod;
        int ret;
  
        mod.Vpp1 = mod.Vpp2 = on ? dev->vpp : 0;
  
        DEBUG(2, "dev = %p on = %d vpp = %d\n", dev, on, dev->vpp);
-       ret = pcmcia_modify_configuration(link->handle, &mod);
+       ret = pcmcia_modify_configuration(link, &mod);
        if(ret != CS_SUCCESS) {
-               cs_error(link->handle, ModifyConfiguration, ret);
+               cs_error(link, ModifyConfiguration, ret);
        }
  }
  
   * still open, this will be postponed until it is closed.
   */
  
- static void pcmciamtd_release(dev_link_t *link)
+ static void pcmciamtd_release(struct pcmcia_device *link)
  {
        struct pcmciamtd_dev *dev = link->priv;
  
                }
                pcmcia_release_window(link->win);
        }
-       pcmcia_release_configuration(link->handle);
-       link->state &= ~DEV_CONFIG;
+       pcmcia_disable_device(link);
  }
  
  
- static void card_settings(struct pcmciamtd_dev *dev, dev_link_t *link, int *new_name)
+ static void card_settings(struct pcmciamtd_dev *dev, struct pcmcia_device *link, int *new_name)
  {
        int rc;
        tuple_t tuple;
        tuple.TupleOffset = 0;
        tuple.DesiredTuple = RETURN_FIRST_TUPLE;
  
-       rc = pcmcia_get_first_tuple(link->handle, &tuple);
+       rc = pcmcia_get_first_tuple(link, &tuple);
        while(rc == CS_SUCCESS) {
-               rc = pcmcia_get_tuple_data(link->handle, &tuple);
+               rc = pcmcia_get_tuple_data(link, &tuple);
                if(rc != CS_SUCCESS) {
-                       cs_error(link->handle, GetTupleData, rc);
+                       cs_error(link, GetTupleData, rc);
                        break;
                }
-               rc = pcmcia_parse_tuple(link->handle, &tuple, &parse);
+               rc = pcmcia_parse_tuple(link, &tuple, &parse);
                if(rc != CS_SUCCESS) {
-                       cs_error(link->handle, ParseTuple, rc);
+                       cs_error(link, ParseTuple, rc);
                        break;
                }
  
                        DEBUG(2, "Unknown tuple code %d", tuple.TupleCode);
                }
  
-               rc = pcmcia_get_next_tuple(link->handle, &tuple);
+               rc = pcmcia_get_next_tuple(link, &tuple);
        }
        if(!dev->pcmcia_map.size)
                dev->pcmcia_map.size = MAX_PCMCIA_ADDR;
  #define CS_CHECK(fn, ret) \
  do { last_fn = (fn); if ((last_ret = (ret)) != 0) goto cs_failed; } while (0)
  
- static void pcmciamtd_config(dev_link_t *link)
+ static int pcmciamtd_config(struct pcmcia_device *link)
  {
        struct pcmciamtd_dev *dev = link->priv;
        struct mtd_info *mtd = NULL;
  
        DEBUG(3, "link=0x%p", link);
  
-       /* Configure card */
-       link->state |= DEV_CONFIG;
        DEBUG(2, "Validating CIS");
-       ret = pcmcia_validate_cis(link->handle, &cisinfo);
+       ret = pcmcia_validate_cis(link, &cisinfo);
        if(ret != CS_SUCCESS) {
-               cs_error(link->handle, GetTupleData, ret);
+               cs_error(link, GetTupleData, ret);
        } else {
                DEBUG(2, "ValidateCIS found %d chains", cisinfo.Chains);
        }
        req.Attributes |= (dev->pcmcia_map.bankwidth == 1) ? WIN_DATA_WIDTH_8 : WIN_DATA_WIDTH_16;
        req.Base = 0;
        req.AccessSpeed = mem_speed;
-       link->win = (window_handle_t)link->handle;
+       link->win = (window_handle_t)link;
        req.Size = (force_size) ? force_size << 20 : MAX_PCMCIA_ADDR;
        dev->win_size = 0;
  
                int ret;
                DEBUG(2, "requesting window with size = %dKiB memspeed = %d",
                      req.Size >> 10, req.AccessSpeed);
-               ret = pcmcia_request_window(&link->handle, &req, &link->win);
+               ret = pcmcia_request_window(&link, &req, &link->win);
                DEBUG(2, "ret = %d dev->win_size = %d", ret, dev->win_size);
                if(ret) {
                        req.Size >>= 1;
        if(!dev->win_size) {
                err("Cant allocate memory window");
                pcmciamtd_release(link);
-               return;
+               return -ENODEV;
        }
        DEBUG(1, "Allocated a window of %dKiB", dev->win_size >> 10);
  
        /* Get write protect status */
-       CS_CHECK(GetStatus, pcmcia_get_status(link->handle, &status));
+       CS_CHECK(GetStatus, pcmcia_get_status(link, &status));
        DEBUG(2, "status value: 0x%x window handle = 0x%8.8lx",
              status.CardState, (unsigned long)link->win);
        dev->win_base = ioremap(req.Base, req.Size);
        if(!dev->win_base) {
                err("ioremap(%lu, %u) failed", req.Base, req.Size);
                pcmciamtd_release(link);
-               return;
+               return -ENODEV;
        }
        DEBUG(1, "mapped window dev = %p req.base = 0x%lx base = %p size = 0x%x",
              dev, req.Base, dev->win_base, req.Size);
        dev->pcmcia_map.map_priv_2 = (unsigned long)link->win;
  
        DEBUG(2, "Getting configuration");
-       CS_CHECK(GetConfigurationInfo, pcmcia_get_configuration_info(link->handle, &t));
+       CS_CHECK(GetConfigurationInfo, pcmcia_get_configuration_info(link, &t));
        DEBUG(2, "Vcc = %d Vpp1 = %d Vpp2 = %d", t.Vcc, t.Vpp1, t.Vpp2);
        dev->vpp = (vpp) ? vpp : t.Vpp1;
        link->conf.Attributes = 0;
-       link->conf.Vcc = t.Vcc;
        if(setvpp == 2) {
-               link->conf.Vpp1 = dev->vpp;
-               link->conf.Vpp2 = dev->vpp;
+               link->conf.Vpp = dev->vpp;
        } else {
-               link->conf.Vpp1 = 0;
-               link->conf.Vpp2 = 0;
+               link->conf.Vpp = 0;
        }
  
        link->conf.IntType = INT_MEMORY;
        link->conf.ConfigIndex = 0;
        link->conf.Present = t.Present;
        DEBUG(2, "Setting Configuration");
-       ret = pcmcia_request_configuration(link->handle, &link->conf);
+       ret = pcmcia_request_configuration(link, &link->conf);
        if(ret != CS_SUCCESS) {
-               cs_error(link->handle, RequestConfiguration, ret);
+               cs_error(link, RequestConfiguration, ret);
+               return -ENODEV;
        }
  
        if(mem_type == 1) {
        } else if(mem_type == 2) {
                mtd = do_map_probe("map_rom", &dev->pcmcia_map);
        } else {
 -              for(i = 0; i < sizeof(probes) / sizeof(char *); i++) {
 +              for(i = 0; i < ARRAY_SIZE(probes); i++) {
                        DEBUG(1, "Trying %s", probes[i]);
                        mtd = do_map_probe(probes[i], &dev->pcmcia_map);
                        if(mtd)
        if(!mtd) {
                DEBUG(1, "Cant find an MTD");
                pcmciamtd_release(link);
-               return;
+               return -ENODEV;
        }
  
        dev->mtd_info = mtd;
           use the faster non-remapping read/write functions */
        if(mtd->size <= dev->win_size) {
                DEBUG(1, "Using non remapping memory functions");
-               dev->pcmcia_map.map_priv_1 = (unsigned long)&(dev->link.state);
                dev->pcmcia_map.map_priv_2 = (unsigned long)dev->win_base;
                if (dev->pcmcia_map.bankwidth == 1) {
                        dev->pcmcia_map.read = pcmcia_read8;
                dev->mtd_info = NULL;
                err("Couldnt register MTD device");
                pcmciamtd_release(link);
-               return;
+               return -ENODEV;
        }
        snprintf(dev->node.dev_name, sizeof(dev->node.dev_name), "mtd%d", mtd->index);
        info("mtd%d: %s", mtd->index, mtd->name);
-       link->state &= ~DEV_CONFIG_PENDING;
-       link->dev = &dev->node;
-       return;
+       link->dev_node = &dev->node;
+       return 0;
  
   cs_failed:
-       cs_error(link->handle, last_fn, last_ret);
+       cs_error(link, last_fn, last_ret);
        err("CS Error, exiting");
        pcmciamtd_release(link);
-       return;
+       return -ENODEV;
  }
  
  
@@@ -713,21 -705,18 +705,18 @@@ static int pcmciamtd_resume(struct pcmc
   * when the device is released.
   */
  
- static void pcmciamtd_detach(struct pcmcia_device *p_dev)
+ static void pcmciamtd_detach(struct pcmcia_device *link)
  {
-       dev_link_t *link = dev_to_instance(p_dev);
+       struct pcmciamtd_dev *dev = link->priv;
  
        DEBUG(3, "link=0x%p", link);
  
-       if(link->state & DEV_CONFIG) {
-               struct pcmciamtd_dev *dev = link->priv;
-               if(dev->mtd_info) {
-                       del_mtd_device(dev->mtd_info);
-                       info("mtd%d: Removed", dev->mtd_info->index);
-               }
-               pcmciamtd_release(link);
+       if(dev->mtd_info) {
+               del_mtd_device(dev->mtd_info);
+               info("mtd%d: Removed", dev->mtd_info->index);
        }
+       pcmciamtd_release(link);
  }
  
  
   * with Card Services.
   */
  
- static int pcmciamtd_attach(struct pcmcia_device *p_dev)
+ static int pcmciamtd_probe(struct pcmcia_device *link)
  {
        struct pcmciamtd_dev *dev;
-       dev_link_t *link;
  
        /* Create new memory card device */
        dev = kmalloc(sizeof(*dev), GFP_KERNEL);
        DEBUG(1, "dev=0x%p", dev);
  
        memset(dev, 0, sizeof(*dev));
-       link = &dev->link;
+       dev->p_dev = link;
        link->priv = dev;
  
        link->conf.Attributes = 0;
        link->conf.IntType = INT_MEMORY;
  
-       link->next = NULL;
-       link->handle = p_dev;
-       p_dev->instance = link;
-       link->state |= DEV_PRESENT | DEV_CONFIG_PENDING;
-       pcmciamtd_config(link);
-       return 0;
+       return pcmciamtd_config(link);
  }
  
  static struct pcmcia_device_id pcmciamtd_ids[] = {
@@@ -794,7 -775,7 +775,7 @@@ static struct pcmcia_driver pcmciamtd_d
        .drv            = {
                .name   = "pcmciamtd"
        },
-       .probe          = pcmciamtd_attach,
+       .probe          = pcmciamtd_probe,
        .remove         = pcmciamtd_detach,
        .owner          = THIS_MODULE,
        .id_table       = pcmciamtd_ids,
index e8f849e1297658c7511db6d8439765f8027c8b22,a92a3134c8336944802576e0638dab76371c9485..71f45056a70cef15b33970c6f8b530a3ee74dc7b
@@@ -289,9 -289,9 +289,9 @@@ static void mii_wr(kio_addr_t ioaddr, u
   * and ejection events.  They are invoked from the event handler.
   */
  
- static int has_ce2_string(dev_link_t * link);
- static void xirc2ps_config(dev_link_t * link);
- static void xirc2ps_release(dev_link_t * link);
+ static int has_ce2_string(struct pcmcia_device * link);
+ static int xirc2ps_config(struct pcmcia_device * link);
+ static void xirc2ps_release(struct pcmcia_device * link);
  
  /****************
   * The attach() and detach() entry points are used to create and destroy
@@@ -313,10 -313,10 +313,10 @@@ static irqreturn_t xirc2ps_interrupt(in
  /****************
   * A linked list of "instances" of the device.  Each actual
   * PCMCIA card corresponds to one device instance, and is described
-  * by one dev_link_t structure (defined in ds.h).
+  * by one struct pcmcia_device structure (defined in ds.h).
   *
   * You may not want to use a linked list for this -- for example, the
-  * memory card driver uses an array of dev_link_t pointers, where minor
+  * memory card driver uses an array of struct pcmcia_device pointers, where minor
   * device numbers are used to derive the corresponding array index.
   */
  
   * example, ethernet cards, modems).  In other cases, there may be
   * many actual or logical devices (SCSI adapters, memory cards with
   * multiple partitions).  The dev_node_t structures need to be kept
-  * in a linked list starting at the 'dev' field of a dev_link_t
+  * in a linked list starting at the 'dev' field of a struct pcmcia_device
   * structure.  We allocate them in the card's private data structure,
   * because they generally can't be allocated dynamically.
   */
  
  typedef struct local_info_t {
-     dev_link_t link;
+       struct pcmcia_device    *p_dev;
      dev_node_t node;
      struct net_device_stats stats;
      int card_type;
@@@ -355,7 -355,7 +355,7 @@@ static void do_tx_timeout(struct net_de
  static struct net_device_stats *do_get_stats(struct net_device *dev);
  static void set_addresses(struct net_device *dev);
  static void set_multicast_list(struct net_device *dev);
- static int set_card_type(dev_link_t *link, const void *s);
+ static int set_card_type(struct pcmcia_device *link, const void *s);
  static int do_config(struct net_device *dev, struct ifmap *map);
  static int do_open(struct net_device *dev);
  static int do_ioctl(struct net_device *dev, struct ifreq *rq, int cmd);
@@@ -368,7 -368,7 +368,7 @@@ static int do_stop(struct net_device *d
  
  /*=============== Helper functions =========================*/
  static int
- first_tuple(client_handle_t handle, tuple_t *tuple, cisparse_t *parse)
+ first_tuple(struct pcmcia_device *handle, tuple_t *tuple, cisparse_t *parse)
  {
        int err;
  
  }
  
  static int
- next_tuple(client_handle_t handle, tuple_t *tuple, cisparse_t *parse)
+ next_tuple(struct pcmcia_device *handle, tuple_t *tuple, cisparse_t *parse)
  {
        int err;
  
@@@ -553,9 -553,8 +553,8 @@@ mii_wr(kio_addr_t ioaddr, u_char phyadd
   */
  
  static int
- xirc2ps_attach(struct pcmcia_device *p_dev)
+ xirc2ps_probe(struct pcmcia_device *link)
  {
-     dev_link_t *link;
      struct net_device *dev;
      local_info_t *local;
  
      if (!dev)
            return -ENOMEM;
      local = netdev_priv(dev);
-     link = &local->link;
+     local->p_dev = link;
      link->priv = dev;
  
      /* General socket configuration */
      link->conf.Attributes = CONF_ENABLE_IRQ;
-     link->conf.Vcc = 50;
      link->conf.IntType = INT_MEMORY_AND_IO;
      link->conf.ConfigIndex = 1;
      link->conf.Present = PRESENT_OPTION;
      dev->watchdog_timeo = TX_TIMEOUT;
  #endif
  
-     link->handle = p_dev;
-     p_dev->instance = link;
-     link->state |= DEV_PRESENT | DEV_CONFIG_PENDING;
-     xirc2ps_config(link);
-     return 0;
+     return xirc2ps_config(link);
  } /* xirc2ps_attach */
  
  /****************
   */
  
  static void
- xirc2ps_detach(struct pcmcia_device *p_dev)
+ xirc2ps_detach(struct pcmcia_device *link)
  {
-     dev_link_t *link = dev_to_instance(p_dev);
      struct net_device *dev = link->priv;
  
      DEBUG(0, "detach(0x%p)\n", link);
  
-     if (link->dev)
+     if (link->dev_node)
        unregister_netdev(dev);
  
-     if (link->state & DEV_CONFIG)
-       xirc2ps_release(link);
+     xirc2ps_release(link);
  
      free_netdev(dev);
  } /* xirc2ps_detach */
   *
   */
  static int
- set_card_type(dev_link_t *link, const void *s)
+ set_card_type(struct pcmcia_device *link, const void *s)
  {
      struct net_device *dev = link->priv;
      local_info_t *local = netdev_priv(dev);
   * Returns: true if this is a CE2
   */
  static int
- has_ce2_string(dev_link_t * link)
+ has_ce2_string(struct pcmcia_device * link)
  {
-     client_handle_t handle = link->handle;
      tuple_t tuple;
      cisparse_t parse;
      u_char buf[256];
      tuple.TupleDataMax = 254;
      tuple.TupleOffset = 0;
      tuple.DesiredTuple = CISTPL_VERS_1;
-     if (!first_tuple(handle, &tuple, &parse) && parse.version_1.ns > 2) {
+     if (!first_tuple(link, &tuple, &parse) && parse.version_1.ns > 2) {
        if (strstr(parse.version_1.str + parse.version_1.ofs[2], "CE2"))
            return 1;
      }
   * is received, to configure the PCMCIA socket, and to make the
   * ethernet device available to the system.
   */
- static void
- xirc2ps_config(dev_link_t * link)
+ static int
+ xirc2ps_config(struct pcmcia_device * link)
  {
-     client_handle_t handle = link->handle;
      struct net_device *dev = link->priv;
      local_info_t *local = netdev_priv(dev);
      tuple_t tuple;
  
      /* Is this a valid        card */
      tuple.DesiredTuple = CISTPL_MANFID;
-     if ((err=first_tuple(handle, &tuple, &parse))) {
+     if ((err=first_tuple(link, &tuple, &parse))) {
        printk(KNOT_XIRC "manfid not found in CIS\n");
        goto failure;
      }
  
      /* get configuration stuff */
      tuple.DesiredTuple = CISTPL_CONFIG;
-     if ((err=first_tuple(handle, &tuple, &parse)))
+     if ((err=first_tuple(link, &tuple, &parse)))
        goto cis_error;
      link->conf.ConfigBase = parse.config.base;
      link->conf.Present =    parse.config.rmask[0];
  
      /* get the ethernet address from the CIS */
      tuple.DesiredTuple = CISTPL_FUNCE;
-     for (err = first_tuple(handle, &tuple, &parse); !err;
-                            err = next_tuple(handle, &tuple, &parse)) {
+     for (err = first_tuple(link, &tuple, &parse); !err;
+                            err = next_tuple(link, &tuple, &parse)) {
        /* Once I saw two CISTPL_FUNCE_LAN_NODE_ID entries:
         * the first one with a length of zero the second correct -
         * so I skip all entries with length 0 */
      }
      if (err) { /* not found: try to get the node-id from tuple 0x89 */
        tuple.DesiredTuple = 0x89;  /* data layout looks like tuple 0x22 */
-       if ((err = pcmcia_get_first_tuple(handle, &tuple)) == 0 &&
-               (err = pcmcia_get_tuple_data(handle, &tuple)) == 0) {
+       if ((err = pcmcia_get_first_tuple(link, &tuple)) == 0 &&
+               (err = pcmcia_get_tuple_data(link, &tuple)) == 0) {
            if (tuple.TupleDataLen == 8 && *buf == CISTPL_FUNCE_LAN_NODE_ID)
                memcpy(&parse, buf, 8);
            else
      }
      if (err) { /* another try (James Lehmer's CE2 version 4.1)*/
        tuple.DesiredTuple = CISTPL_FUNCE;
-       for (err = first_tuple(handle, &tuple, &parse); !err;
-                                err = next_tuple(handle, &tuple, &parse)) {
+       for (err = first_tuple(link, &tuple, &parse); !err;
+                                err = next_tuple(link, &tuple, &parse)) {
            if (parse.funce.type == 0x02 && parse.funce.data[0] == 1
                && parse.funce.data[1] == 6 && tuple.TupleDataLen == 13) {
                buf[1] = 4;
      for (i=0; i < 6; i++)
        dev->dev_addr[i] = node_id->id[i];
  
-     /* Configure card */
-     link->state |= DEV_CONFIG;
      link->io.IOAddrLines =10;
      link->io.Attributes1 = IO_DATA_PATH_WIDTH_16;
      link->irq.Attributes = IRQ_HANDLE_PRESENT;
             * Ethernet port */
            link->io.NumPorts1 = 16; /* no Mako stuff anymore */
            tuple.DesiredTuple = CISTPL_CFTABLE_ENTRY;
-           for (err = first_tuple(handle, &tuple, &parse); !err;
-                                err = next_tuple(handle, &tuple, &parse)) {
+           for (err = first_tuple(link, &tuple, &parse); !err;
+                                err = next_tuple(link, &tuple, &parse)) {
                if (cf->io.nwin > 0  &&  (cf->io.win[0].base & 0xf) == 8) {
                    for (ioaddr = 0x300; ioaddr < 0x400; ioaddr += 0x10) {
                        link->conf.ConfigIndex = cf->index ;
                        link->io.BasePort2 = cf->io.win[0].base;
                        link->io.BasePort1 = ioaddr;
-                       if (!(err=pcmcia_request_io(link->handle, &link->io)))
+                       if (!(err=pcmcia_request_io(link, &link->io)))
                            goto port_found;
                    }
                }
             */
            for (pass=0; pass < 2; pass++) {
                tuple.DesiredTuple = CISTPL_CFTABLE_ENTRY;
-               for (err = first_tuple(handle, &tuple, &parse); !err;
-                                    err = next_tuple(handle, &tuple, &parse)){
+               for (err = first_tuple(link, &tuple, &parse); !err;
+                                    err = next_tuple(link, &tuple, &parse)){
                    if (cf->io.nwin > 0  &&  (cf->io.win[0].base & 0xf) == 8){
                        link->conf.ConfigIndex = cf->index ;
                        link->io.BasePort2 = cf->io.win[0].base;
                        link->io.BasePort1 = link->io.BasePort2
                                    + (pass ? (cf->index & 0x20 ? -24:8)
                                            : (cf->index & 0x20 ?   8:-24));
-                       if (!(err=pcmcia_request_io(link->handle, &link->io)))
+                       if (!(err=pcmcia_request_io(link, &link->io)))
                            goto port_found;
                    }
                }
        link->io.NumPorts1 = 16;
        for (ioaddr = 0x300; ioaddr < 0x400; ioaddr += 0x10) {
            link->io.BasePort1 = ioaddr;
-           if (!(err=pcmcia_request_io(link->handle, &link->io)))
+           if (!(err=pcmcia_request_io(link, &link->io)))
                goto port_found;
        }
        link->io.BasePort1 = 0; /* let CS decide */
-       if ((err=pcmcia_request_io(link->handle, &link->io))) {
-           cs_error(link->handle, RequestIO, err);
+       if ((err=pcmcia_request_io(link, &link->io))) {
+           cs_error(link, RequestIO, err);
            goto config_error;
        }
      }
       * Now allocate an interrupt line.        Note that this does not
       * actually assign a handler to the interrupt.
       */
-     if ((err=pcmcia_request_irq(link->handle, &link->irq))) {
-       cs_error(link->handle, RequestIRQ, err);
+     if ((err=pcmcia_request_irq(link, &link->irq))) {
+       cs_error(link, RequestIRQ, err);
        goto config_error;
      }
  
       * This actually configures the PCMCIA socket -- setting up
       * the I/O windows and the interrupt mapping.
       */
-     if ((err=pcmcia_request_configuration(link->handle, &link->conf))) {
-       cs_error(link->handle, RequestConfiguration, err);
+     if ((err=pcmcia_request_configuration(link, &link->conf))) {
+       cs_error(link, RequestConfiguration, err);
        goto config_error;
      }
  
        reg.Action = CS_WRITE;
        reg.Offset = CISREG_IOBASE_0;
        reg.Value = link->io.BasePort2 & 0xff;
-       if ((err = pcmcia_access_configuration_register(link->handle, &reg))) {
-           cs_error(link->handle, AccessConfigurationRegister, err);
+       if ((err = pcmcia_access_configuration_register(link, &reg))) {
+           cs_error(link, AccessConfigurationRegister, err);
            goto config_error;
        }
        reg.Action = CS_WRITE;
        reg.Offset = CISREG_IOBASE_1;
        reg.Value = (link->io.BasePort2 >> 8) & 0xff;
-       if ((err = pcmcia_access_configuration_register(link->handle, &reg))) {
-           cs_error(link->handle, AccessConfigurationRegister, err);
+       if ((err = pcmcia_access_configuration_register(link, &reg))) {
+           cs_error(link, AccessConfigurationRegister, err);
            goto config_error;
        }
  
        req.Attributes = WIN_DATA_WIDTH_8|WIN_MEMORY_TYPE_AM|WIN_ENABLE;
        req.Base = req.Size = 0;
        req.AccessSpeed = 0;
-       if ((err = pcmcia_request_window(&link->handle, &req, &link->win))) {
-           cs_error(link->handle, RequestWindow, err);
+       if ((err = pcmcia_request_window(&link, &req, &link->win))) {
+           cs_error(link, RequestWindow, err);
            goto config_error;
        }
        local->dingo_ccr = ioremap(req.Base,0x1000) + 0x0800;
        mem.CardOffset = 0x0;
        mem.Page = 0;
        if ((err = pcmcia_map_mem_page(link->win, &mem))) {
-           cs_error(link->handle, MapMemPage, err);
+           cs_error(link, MapMemPage, err);
            goto config_error;
        }
  
      if (local->dingo)
        do_reset(dev, 1); /* a kludge to make the cem56 work */
  
-     link->dev = &local->node;
-     link->state &= ~DEV_CONFIG_PENDING;
-     SET_NETDEV_DEV(dev, &handle_to_dev(handle));
+     link->dev_node = &local->node;
+     SET_NETDEV_DEV(dev, &handle_to_dev(link));
  
      if ((err=register_netdev(dev))) {
        printk(KNOT_XIRC "register_netdev() failed\n");
-       link->dev = NULL;
+       link->dev_node = NULL;
        goto config_error;
      }
  
        printk("%c%02X", i?':':' ', dev->dev_addr[i]);
      printk("\n");
  
-     return;
+     return 0;
  
    config_error:
-     link->state &= ~DEV_CONFIG_PENDING;
      xirc2ps_release(link);
-     return;
+     return -ENODEV;
  
    cis_error:
      printk(KNOT_XIRC "unable to parse CIS\n");
    failure:
-     link->state &= ~DEV_CONFIG_PENDING;
+     return -ENODEV;
  } /* xirc2ps_config */
  
  /****************
   * still open, this will be postponed until it is closed.
   */
  static void
- xirc2ps_release(dev_link_t *link)
+ xirc2ps_release(struct pcmcia_device *link)
  {
+       DEBUG(0, "release(0x%p)\n", link);
  
-     DEBUG(0, "release(0x%p)\n", link);
-     if (link->win) {
-       struct net_device *dev = link->priv;
-       local_info_t *local = netdev_priv(dev);
-       if (local->dingo)
-           iounmap(local->dingo_ccr - 0x0800);
-       pcmcia_release_window(link->win);
-     }
-     pcmcia_release_configuration(link->handle);
-     pcmcia_release_io(link->handle, &link->io);
-     pcmcia_release_irq(link->handle, &link->irq);
-     link->state &= ~DEV_CONFIG;
+       if (link->win) {
+               struct net_device *dev = link->priv;
+               local_info_t *local = netdev_priv(dev);
+               if (local->dingo)
+                       iounmap(local->dingo_ccr - 0x0800);
+       }
+       pcmcia_disable_device(link);
  } /* xirc2ps_release */
  
  /*====================================================================*/
  
  
- static int xirc2ps_suspend(struct pcmcia_device *p_dev)
+ static int xirc2ps_suspend(struct pcmcia_device *link)
  {
-       dev_link_t *link = dev_to_instance(p_dev);
        struct net_device *dev = link->priv;
  
-       link->state |= DEV_SUSPEND;
-       if (link->state & DEV_CONFIG) {
-               if (link->open) {
-                       netif_device_detach(dev);
-                       do_powerdown(dev);
-               }
-               pcmcia_release_configuration(link->handle);
+       if (link->open) {
+               netif_device_detach(dev);
+               do_powerdown(dev);
        }
  
        return 0;
  }
  
- static int xirc2ps_resume(struct pcmcia_device *p_dev)
+ static int xirc2ps_resume(struct pcmcia_device *link)
  {
-       dev_link_t *link = dev_to_instance(p_dev);
        struct net_device *dev = link->priv;
  
-       link->state &= ~DEV_SUSPEND;
-       if (link->state & DEV_CONFIG) {
-               pcmcia_request_configuration(link->handle, &link->conf);
-               if (link->open) {
-                       do_reset(dev,1);
-                       netif_device_attach(dev);
-               }
+       if (link->open) {
+               do_reset(dev,1);
+               netif_device_attach(dev);
        }
  
        return 0;
@@@ -1552,13 -1519,13 +1519,13 @@@ static in
  do_open(struct net_device *dev)
  {
      local_info_t *lp = netdev_priv(dev);
-     dev_link_t *link = &lp->link;
+     struct pcmcia_device *link = lp->p_dev;
  
      DEBUG(0, "do_open(%p)\n", dev);
  
      /* Check that the PCMCIA card is still here. */
      /* Physical device present signature. */
-     if (!DEV_OK(link))
+     if (!pcmcia_dev_present(link))
        return -ENODEV;
  
      /* okay */
@@@ -1882,7 -1849,7 +1849,7 @@@ do_stop(struct net_device *dev
  {
      kio_addr_t ioaddr = dev->base_addr;
      local_info_t *lp = netdev_priv(dev);
-     dev_link_t *link = &lp->link;
+     struct pcmcia_device *link = lp->p_dev;
  
      DEBUG(0, "do_stop(%p)\n", dev);
  
@@@ -1935,7 -1902,7 +1902,7 @@@ static struct pcmcia_driver xirc2ps_cs_
        .drv            = {
                .name   = "xirc2ps_cs",
        },
-       .probe          = xirc2ps_attach,
+       .probe          = xirc2ps_probe,
        .remove         = xirc2ps_detach,
        .id_table       = xirc2ps_ids,
        .suspend        = xirc2ps_suspend,
@@@ -1973,7 -1940,7 +1940,7 @@@ static int __init setup_xirc2ps_cs(cha
        MAYBE_SET(lockup_hack, 6);
  #undef  MAYBE_SET
  
 -      return 0;
 +      return 1;
  }
  
  __setup("xirc2ps_cs=", setup_xirc2ps_cs);