Merge git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/pcmcia-2.6
authorLinus Torvalds <torvalds@linux-foundation.org>
Wed, 16 Sep 2009 15:11:23 +0000 (08:11 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Wed, 16 Sep 2009 15:11:23 +0000 (08:11 -0700)
* git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/pcmcia-2.6:
  pcmcia: document return value of pcmcia_loop_config
  pcmcia: dtl1_cs: fix pcmcia_loop_config logic
  pcmcia: drop non-existant includes
  pcmcia: disable prefetch/burst for OZ6933
  pcmcia: fix incorrect argument order to list_add_tail()
  pcmcia: drivers/pcmcia/pcmcia_resource.c: Remove unnecessary semicolons
  pcmcia: Use phys_addr_t for physical addresses
  pcmcia: drivers/pcmcia: Make static

drivers/bluetooth/dtl1_cs.c
drivers/pcmcia/au1000_pb1x00.c
drivers/pcmcia/au1000_xxs1500.c
drivers/pcmcia/ds.c
drivers/pcmcia/o2micro.h
drivers/pcmcia/pcmcia_ioctl.c
drivers/pcmcia/pcmcia_resource.c
include/pcmcia/ss.h

index 2cc7b3266eaf6bd423d2f7af2cab55ef9b4bd31e..b881a9cd8741d78cb3c24adfb3f868cb604c294d 100644 (file)
@@ -618,7 +618,7 @@ static int dtl1_config(struct pcmcia_device *link)
 
        /* Look for a generic full-sized window */
        link->io.NumPorts1 = 8;
-       if (!pcmcia_loop_config(link, dtl1_confcheck, NULL))
+       if (pcmcia_loop_config(link, dtl1_confcheck, NULL) < 0)
                goto failed;
 
        i = pcmcia_request_irq(link, &link->irq);
index d6b4bd1db7d7e6b9edbb5a3117070cc0f718df58..b1984ed72d1d78ef6f0a673ab5dd797c5c76f6cb 100644 (file)
@@ -26,7 +26,6 @@
 #include <linux/delay.h>
 #include <linux/ioport.h>
 #include <linux/kernel.h>
-#include <linux/tqueue.h>
 #include <linux/timer.h>
 #include <linux/mm.h>
 #include <linux/proc_fs.h>
index 9627390835caa2cd28b97090ec90d76cf37bf5e2..b43d47b50819e4b8a1ab78778ec7ee472fe787dd 100644 (file)
@@ -30,7 +30,6 @@
 #include <linux/delay.h>
 #include <linux/ioport.h>
 #include <linux/kernel.h>
-#include <linux/tqueue.h>
 #include <linux/timer.h>
 #include <linux/mm.h>
 #include <linux/proc_fs.h>
index 304ff6d5cf3b2ec13850a253bc908481027cd844..9f300d3cb12539377f30105e0cdb697106b46524 100644 (file)
@@ -236,7 +236,6 @@ pcmcia_store_new_id(struct device_driver *driver, const char *buf, size_t count)
        if (!dynid)
                return -ENOMEM;
 
-       INIT_LIST_HEAD(&dynid->node);
        dynid->id.match_flags = match_flags;
        dynid->id.manf_id = manf_id;
        dynid->id.card_id = card_id;
@@ -246,7 +245,7 @@ pcmcia_store_new_id(struct device_driver *driver, const char *buf, size_t count)
        memcpy(dynid->id.prod_id_hash, prod_id_hash, sizeof(__u32) * 4);
 
        spin_lock(&pdrv->dynids.lock);
-       list_add_tail(&pdrv->dynids.list, &dynid->node);
+       list_add_tail(&dynid->node, &pdrv->dynids.list);
        spin_unlock(&pdrv->dynids.lock);
 
        if (get_driver(&pdrv->drv)) {
index 5554015a7813c74f7c84352aacc190345dde6da6..72188c462c9cc6222b82ff8e79c87e35b3731201 100644 (file)
@@ -48,6 +48,9 @@
 #ifndef PCI_DEVICE_ID_O2_6812
 #define PCI_DEVICE_ID_O2_6812          0x6872
 #endif
+#ifndef PCI_DEVICE_ID_O2_6933
+#define PCI_DEVICE_ID_O2_6933           0x6933
+#endif
 
 /* Additional PCI configuration registers */
 
@@ -154,6 +157,7 @@ static int o2micro_override(struct yenta_socket *socket)
                case PCI_DEVICE_ID_O2_6812:
                case PCI_DEVICE_ID_O2_6832:
                case PCI_DEVICE_ID_O2_6836:
+               case PCI_DEVICE_ID_O2_6933:
                        dev_printk(KERN_INFO, &socket->dev->dev,
                                   "Yenta O2: old bridge, disabling read "
                                   "prefetch/write burst\n");
index 6095f8daecd7f1eb31fc1a3bc14626ad0df33db5..7b424e0b0449f948ae632a9ef633f1cfd6c68535 100644 (file)
@@ -286,7 +286,7 @@ static int pccard_get_status(struct pcmcia_socket *s,
        return 0;
 } /* pccard_get_status */
 
-int pccard_get_configuration_info(struct pcmcia_socket *s,
+static int pccard_get_configuration_info(struct pcmcia_socket *s,
                                  struct pcmcia_device *p_dev,
                                  config_info_t *config)
 {
index f5d0ba8e22d5473fb46e00bfbe5e3bfe2e7719a9..d919e96c0afd62f5bde18da2ca0a974b63b66079 100644 (file)
@@ -489,7 +489,7 @@ int pcmcia_request_configuration(struct pcmcia_device *p_dev,
        pccard_io_map iomap;
 
        if (!(s->state & SOCKET_PRESENT))
-               return -ENODEV;;
+               return -ENODEV;
 
        if (req->IntType & INT_CARDBUS) {
                ds_dbg(p_dev->socket, 0, "IntType may not be INT_CARDBUS\n");
@@ -902,7 +902,7 @@ struct pcmcia_cfg_mem {
  *
  * pcmcia_loop_config() loops over all configuration options, and calls
  * the driver-specific conf_check() for each one, checking whether
- * it is a valid one.
+ * it is a valid one. Returns 0 on success or errorcode otherwise.
  */
 int pcmcia_loop_config(struct pcmcia_device *p_dev,
                       int      (*conf_check)   (struct pcmcia_device *p_dev,
@@ -915,7 +915,7 @@ int pcmcia_loop_config(struct pcmcia_device *p_dev,
        struct pcmcia_cfg_mem *cfg_mem;
 
        tuple_t *tuple;
-       int ret = -ENODEV;
+       int ret;
        unsigned int vcc;
 
        cfg_mem = kzalloc(sizeof(struct pcmcia_cfg_mem), GFP_KERNEL);
index 9b4ac9385f5d094a07dcaae9daad1c04b6bfb2bd..9a3b4986517324cf50d98db5cfdb169b32ff3711 100644 (file)
@@ -90,14 +90,14 @@ typedef struct pccard_io_map {
        u_char  map;
        u_char  flags;
        u_short speed;
-       u_int   start, stop;
+       phys_addr_t start, stop;
 } pccard_io_map;
 
 typedef struct pccard_mem_map {
        u_char          map;
        u_char          flags;
        u_short         speed;
-       u_long          static_start;
+       phys_addr_t     static_start;
        u_int           card_start;
        struct resource *res;
 } pccard_mem_map;