Commit a42bcfad authored by Adrian Bunk's avatar Adrian Bunk Committed by Linus Torvalds

[PATCH] small MCA cleanups

The patch below does the following cleanups in the MCA code:
- make some needlessly global code static
- remove three unused global functions from mca-legacy.c (two of them
  were EXPORT_SYMBOL'ed); this should IMHO be safe since mca-legacy
  is not an API drivers should move to
Signed-off-by: default avatarAdrian Bunk <bunk@stusta.de>
Acked-by: default avatarJames Bottomley <James.Bottomley@SteelEye.com>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 3404c3c1
...@@ -34,7 +34,7 @@ ...@@ -34,7 +34,7 @@
/* Very few machines have more than one MCA bus. However, there are /* Very few machines have more than one MCA bus. However, there are
* those that do (Voyager 35xx/5xxx), so we do it this way for future * those that do (Voyager 35xx/5xxx), so we do it this way for future
* expansion. None that I know have more than 2 */ * expansion. None that I know have more than 2 */
struct mca_bus *mca_root_busses[MAX_MCA_BUSSES]; static struct mca_bus *mca_root_busses[MAX_MCA_BUSSES];
#define MCA_DEVINFO(i,s) { .pos = i, .name = s } #define MCA_DEVINFO(i,s) { .pos = i, .name = s }
......
...@@ -282,25 +282,6 @@ void mca_set_adapter_name(int slot, char* name) ...@@ -282,25 +282,6 @@ void mca_set_adapter_name(int slot, char* name)
} }
EXPORT_SYMBOL(mca_set_adapter_name); EXPORT_SYMBOL(mca_set_adapter_name);
/**
* mca_get_adapter_name - get the adapter description
* @slot: slot to query
*
* Return the adapter description if set. If it has not been
* set or the slot is out range then return NULL.
*/
char *mca_get_adapter_name(int slot)
{
struct mca_device *mca_dev = mca_find_device_by_slot(slot);
if(!mca_dev)
return NULL;
return mca_device_get_name(mca_dev);
}
EXPORT_SYMBOL(mca_get_adapter_name);
/** /**
* mca_is_adapter_used - check if claimed by driver * mca_is_adapter_used - check if claimed by driver
* @slot: slot to check * @slot: slot to check
...@@ -365,43 +346,3 @@ void mca_mark_as_unused(int slot) ...@@ -365,43 +346,3 @@ void mca_mark_as_unused(int slot)
} }
EXPORT_SYMBOL(mca_mark_as_unused); EXPORT_SYMBOL(mca_mark_as_unused);
/**
* mca_isadapter - check if the slot holds an adapter
* @slot: slot to query
*
* Returns zero if the slot does not hold an adapter, non zero if
* it does.
*/
int mca_isadapter(int slot)
{
struct mca_device *mca_dev = mca_find_device_by_slot(slot);
enum MCA_AdapterStatus status;
if(!mca_dev)
return 0;
status = mca_device_status(mca_dev);
return status == MCA_ADAPTER_NORMAL
|| status == MCA_ADAPTER_DISABLED;
}
EXPORT_SYMBOL(mca_isadapter);
/**
* mca_isenabled - check if the slot holds an enabled adapter
* @slot: slot to query
*
* Returns a non zero value if the slot holds an enabled adapter
* and zero for any other case.
*/
int mca_isenabled(int slot)
{
struct mca_device *mca_dev = mca_find_device_by_slot(slot);
if(!mca_dev)
return 0;
return mca_device_status(mca_dev) == MCA_ADAPTER_NORMAL;
}
...@@ -43,8 +43,8 @@ static int get_mca_info_helper(struct mca_device *mca_dev, char *page, int len) ...@@ -43,8 +43,8 @@ static int get_mca_info_helper(struct mca_device *mca_dev, char *page, int len)
return len; return len;
} }
int get_mca_info(char *page, char **start, off_t off, static int get_mca_info(char *page, char **start, off_t off,
int count, int *eof, void *data) int count, int *eof, void *data)
{ {
int i, len = 0; int i, len = 0;
......
...@@ -34,10 +34,6 @@ ...@@ -34,10 +34,6 @@
extern int mca_find_adapter(int id, int start); extern int mca_find_adapter(int id, int start);
extern int mca_find_unused_adapter(int id, int start); extern int mca_find_unused_adapter(int id, int start);
/* adapter state info - returns 0 if no */
extern int mca_isadapter(int slot);
extern int mca_isenabled(int slot);
extern int mca_is_adapter_used(int slot); extern int mca_is_adapter_used(int slot);
extern int mca_mark_as_used(int slot); extern int mca_mark_as_used(int slot);
extern void mca_mark_as_unused(int slot); extern void mca_mark_as_unused(int slot);
...@@ -50,7 +46,6 @@ extern unsigned char mca_read_stored_pos(int slot, int reg); ...@@ -50,7 +46,6 @@ extern unsigned char mca_read_stored_pos(int slot, int reg);
* so we can have a more interesting /proc/mca. * so we can have a more interesting /proc/mca.
*/ */
extern void mca_set_adapter_name(int slot, char* name); extern void mca_set_adapter_name(int slot, char* name);
extern char* mca_get_adapter_name(int slot);
/* These routines actually mess with the hardware POS registers. They /* These routines actually mess with the hardware POS registers. They
* temporarily disable the device (and interrupts), so make sure you know * temporarily disable the device (and interrupts), so make sure you know
......
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