Commit 32363bfb authored by Dominik Brodowski's avatar Dominik Brodowski Committed by Linus Torvalds

[PATCH] pcmcia: unify bind_mtd and pcmcia_bind_mtd

Unify the pcmcia_bind_mtd and bind_mtd functions.
Signed-off-by: default avatarDominik Brodowski <linux@brodo.de>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent e05b1590
...@@ -171,46 +171,6 @@ static int pcmcia_bind_device(bind_req_t *req) ...@@ -171,46 +171,6 @@ static int pcmcia_bind_device(bind_req_t *req)
return CS_SUCCESS; return CS_SUCCESS;
} /* bind_device */ } /* bind_device */
/*======================================================================
Bind_mtd() associates a device driver with a particular memory
region. It is normally called by Driver Services after it has
identified a memory device type. An instance of the corresponding
driver will then be able to register to control this region.
======================================================================*/
static int pcmcia_bind_mtd(mtd_bind_t *req)
{
struct pcmcia_socket *s;
memory_handle_t region;
s = req->Socket;
if (!s)
return CS_BAD_SOCKET;
if (req->Attributes & REGION_TYPE_AM)
region = s->a_region;
else
region = s->c_region;
while (region) {
if (region->info.CardOffset == req->CardOffset)
break;
region = region->info.next;
}
if (!region || (region->mtd != NULL))
return CS_BAD_OFFSET;
strlcpy(region->dev_info, (char *)req->dev_info, DEV_NAME_LEN);
ds_dbg(1, "%s: bind_mtd: attr 0x%x, offset 0x%x, dev %s\n",
cs_socket_name(s), req->Attributes, req->CardOffset,
(char *)req->dev_info);
return CS_SUCCESS;
} /* bind_mtd */
/* String tables for error messages */ /* String tables for error messages */
typedef struct lookup_t { typedef struct lookup_t {
...@@ -545,22 +505,25 @@ static int ds_event(struct pcmcia_socket *skt, event_t event, int priority) ...@@ -545,22 +505,25 @@ static int ds_event(struct pcmcia_socket *skt, event_t event, int priority)
static int bind_mtd(struct pcmcia_bus_socket *bus_sock, mtd_info_t *mtd_info) static int bind_mtd(struct pcmcia_bus_socket *bus_sock, mtd_info_t *mtd_info)
{ {
mtd_bind_t bind_req; struct pcmcia_socket *s = bus_sock->parent;
int ret; memory_handle_t region;
bind_req.dev_info = &mtd_info->dev_info; if (mtd_info->Attributes & REGION_TYPE_AM)
bind_req.Attributes = mtd_info->Attributes; region = s->a_region;
bind_req.Socket = bus_sock->parent; else
bind_req.CardOffset = mtd_info->CardOffset; region = s->c_region;
ret = pcmcia_bind_mtd(&bind_req);
if (ret != CS_SUCCESS) { while (region) {
cs_error(NULL, BindMTD, ret); if (region->info.CardOffset == mtd_info->CardOffset)
printk(KERN_NOTICE "ds: unable to bind MTD '%s' to socket %d" break;
" offset 0x%x\n", region = region->info.next;
(char *)bind_req.dev_info, bus_sock->parent->sock, bind_req.CardOffset); }
return -ENODEV; if (!region || (region->mtd != NULL))
} return -ENODEV;
return 0;
strlcpy(region->dev_info, mtd_info->dev_info, DEV_NAME_LEN);
return 0;
} /* bind_mtd */ } /* bind_mtd */
/*====================================================================== /*======================================================================
......
...@@ -325,13 +325,6 @@ typedef struct bind_req_t { ...@@ -325,13 +325,6 @@ typedef struct bind_req_t {
/* Flag to bind to all functions */ /* Flag to bind to all functions */
#define BIND_FN_ALL 0xff #define BIND_FN_ALL 0xff
typedef struct mtd_bind_t {
struct pcmcia_socket *Socket;
u_int Attributes;
u_int CardOffset;
dev_info_t *dev_info;
} mtd_bind_t;
/* Events */ /* Events */
#define CS_EVENT_PRI_LOW 0 #define CS_EVENT_PRI_LOW 0
#define CS_EVENT_PRI_HIGH 1 #define CS_EVENT_PRI_HIGH 1
......
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