Commit 6412d927 authored by Alex Dubov's avatar Alex Dubov Committed by Pierre Ossman

tifm_7xx1: Merge media insert and media remove functions

Hardware does not say whether card was inserted or removed when reporting
socket events. Moreover, during suspend, media can be removed or switched
to some other card type without notification. Therefore, for each socket
in the change set the following is performed:
1. If there's active device in the socket it's unregistered
2. Media detection is performed
3. If detection recognizes supportable media, new device is registered

This patch also alters some macros and variable names to enhance clarity.
Signed-off-by: default avatarAlex Dubov <oakad@yahoo.com>
Signed-off-by: default avatarPierre Ossman <drzeus@drzeus.cx>
parent 1499ead3
This diff is collapsed.
...@@ -62,11 +62,10 @@ enum { ...@@ -62,11 +62,10 @@ enum {
#define TIFM_IRQ_ENABLE 0x80000000 #define TIFM_IRQ_ENABLE 0x80000000
#define TIFM_IRQ_SOCKMASK 0x00000001 #define TIFM_IRQ_SOCKMASK(x) (x)
#define TIFM_IRQ_CARDMASK 0x00000100 #define TIFM_IRQ_CARDMASK(x) ((x) << 8)
#define TIFM_IRQ_FIFOMASK 0x00010000 #define TIFM_IRQ_FIFOMASK(x) ((x) << 16)
#define TIFM_IRQ_SETALL 0xffffffff #define TIFM_IRQ_SETALL 0xffffffff
#define TIFM_IRQ_SETALLSOCK 0x0000000f
#define TIFM_CTRL_LED 0x00000040 #define TIFM_CTRL_LED 0x00000040
#define TIFM_CTRL_FAST_CLK 0x00000100 #define TIFM_CTRL_FAST_CLK 0x00000100
...@@ -108,18 +107,16 @@ struct tifm_driver { ...@@ -108,18 +107,16 @@ struct tifm_driver {
struct tifm_adapter { struct tifm_adapter {
char __iomem *addr; char __iomem *addr;
unsigned int irq_status;
unsigned int insert_mask;
unsigned int remove_mask;
spinlock_t lock; spinlock_t lock;
unsigned int irq_status;
unsigned int socket_change_set;
unsigned int id; unsigned int id;
unsigned int max_sockets; unsigned int num_sockets;
struct tifm_dev **sockets;
char wq_name[KOBJ_NAME_LEN]; char wq_name[KOBJ_NAME_LEN];
unsigned int inhibit_new_cards; unsigned int inhibit_new_cards;
struct workqueue_struct *wq; struct workqueue_struct *wq;
struct work_struct media_inserter; struct work_struct media_switcher;
struct work_struct media_remover;
struct tifm_dev **sockets;
struct class_device cdev; struct class_device cdev;
struct device *dev; struct device *dev;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment