summaryrefslogtreecommitdiffstats
path: root/drivers/pcmcia
diff options
context:
space:
mode:
authorDominik Brodowski <linux@dominikbrodowski.net>2010-06-19 14:33:56 +0200
committerDominik Brodowski <linux@dominikbrodowski.net>2010-07-05 12:37:48 +0200
commite4f1ac2122413736bf2791d3af6533f36b46fc61 (patch)
tree5e082638cb15e8fa4631ccd05ffdd89fec097cbd /drivers/pcmcia
parent815c4163b6c8ebf8152f42b0a5fd015cfdcedc78 (diff)
downloadop-kernel-dev-e4f1ac2122413736bf2791d3af6533f36b46fc61.zip
op-kernel-dev-e4f1ac2122413736bf2791d3af6533f36b46fc61.tar.gz
pcmcia: do not initialize the present flag too late.
The "present" flag was initialized too late -- possibly, a card was already registered at this time, so re-setting the flag to 0 caused pcmcia_dev_present() to fail. Reported-by: Mikulas Patocka <mpatocka@redhat.com> Signed-off-by: Dominik Brodowski <linux@dominikbrodowski.net>
Diffstat (limited to 'drivers/pcmcia')
-rw-r--r--drivers/pcmcia/ds.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/pcmcia/ds.c b/drivers/pcmcia/ds.c
index 9fc3398..eac9614 100644
--- a/drivers/pcmcia/ds.c
+++ b/drivers/pcmcia/ds.c
@@ -1356,6 +1356,7 @@ static int __devinit pcmcia_bus_add_socket(struct device *dev,
INIT_LIST_HEAD(&socket->devices_list);
memset(&socket->pcmcia_state, 0, sizeof(u8));
socket->device_count = 0;
+ atomic_set(&socket->present, 0);
ret = pccard_register_pcmcia(socket, &pcmcia_bus_callback);
if (ret) {
@@ -1364,8 +1365,6 @@ static int __devinit pcmcia_bus_add_socket(struct device *dev,
return ret;
}
- atomic_set(&socket->present, 0);
-
return 0;
}
OpenPOWER on IntegriCloud