Skip to content

Commit

Permalink
Staging: comedi: drivers.c sparse cleanup
Browse files Browse the repository at this point in the history
Fix up some sparse issues in drivers.c

Cc: Ian Abbott <abbotti@mev.co.uk>
Cc: Frank Mori Hess <fmhess@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
  • Loading branch information
Greg Kroah-Hartman committed May 11, 2010
1 parent 3b6b25b commit 7029a87
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 18 deletions.
3 changes: 0 additions & 3 deletions drivers/staging/comedi/comedidev.h
Original file line number Diff line number Diff line change
Expand Up @@ -351,9 +351,6 @@ void cleanup_polling(void);
void start_polling(struct comedi_device *);
void stop_polling(struct comedi_device *);

int comedi_buf_alloc(struct comedi_device *dev, struct comedi_subdevice *s,
unsigned long new_size);

#ifdef CONFIG_PROC_FS
void comedi_proc_init(void);
void comedi_proc_cleanup(void);
Expand Down
24 changes: 9 additions & 15 deletions drivers/staging/comedi/drivers.c
Original file line number Diff line number Diff line change
Expand Up @@ -54,16 +54,9 @@ static int insn_rw_emulate_bits(struct comedi_device *dev,
static void *comedi_recognize(struct comedi_driver *driv, const char *name);
static void comedi_report_boards(struct comedi_driver *driv);
static int poll_invalid(struct comedi_device *dev, struct comedi_subdevice *s);
int comedi_buf_alloc(struct comedi_device *dev, struct comedi_subdevice *s,
unsigned long new_size);

struct comedi_driver *comedi_drivers;

int comedi_modprobe(int minor)
{
return -EINVAL;
}

static void cleanup_device(struct comedi_device *dev)
{
int i;
Expand All @@ -84,7 +77,7 @@ static void cleanup_device(struct comedi_device *dev)
}
kfree(dev->private);
dev->private = NULL;
dev->driver = 0;
dev->driver = NULL;
dev->board_name = NULL;
dev->board_ptr = NULL;
dev->iobase = 0;
Expand Down Expand Up @@ -309,7 +302,7 @@ static int postconfig(struct comedi_device *dev)

/* generic recognize function for drivers
* that register their supported board names */
void *comedi_recognize(struct comedi_driver *driv, const char *name)
static void *comedi_recognize(struct comedi_driver *driv, const char *name)
{
unsigned i;
const char *const *name_ptr = driv->board_name;
Expand All @@ -324,7 +317,7 @@ void *comedi_recognize(struct comedi_driver *driv, const char *name)
return NULL;
}

void comedi_report_boards(struct comedi_driver *driv)
static void comedi_report_boards(struct comedi_driver *driv)
{
unsigned int i;
const char *const *name_ptr;
Expand Down Expand Up @@ -548,8 +541,8 @@ int comedi_buf_alloc(struct comedi_device *dev, struct comedi_subdevice *s,

/* munging is applied to data by core as it passes between user
* and kernel space */
unsigned int comedi_buf_munge(struct comedi_async *async,
unsigned int num_bytes)
static unsigned int comedi_buf_munge(struct comedi_async *async,
unsigned int num_bytes)
{
struct comedi_subdevice *s = async->subdevice;
unsigned int count = 0;
Expand Down Expand Up @@ -812,8 +805,9 @@ void comedi_reset_async_buf(struct comedi_async *async)
async->events = 0;
}

int comedi_auto_config(struct device *hardware_device, const char *board_name,
const int *options, unsigned num_options)
static int comedi_auto_config(struct device *hardware_device,
const char *board_name, const int *options,
unsigned num_options)
{
struct comedi_devconfig it;
int minor;
Expand Down Expand Up @@ -858,7 +852,7 @@ int comedi_auto_config(struct device *hardware_device, const char *board_name,
return retval;
}

void comedi_auto_unconfig(struct device *hardware_device)
static void comedi_auto_unconfig(struct device *hardware_device)
{
unsigned *minor = (unsigned *)dev_get_drvdata(hardware_device);
if (minor == NULL)
Expand Down
2 changes: 2 additions & 0 deletions drivers/staging/comedi/internal.h
Original file line number Diff line number Diff line change
Expand Up @@ -8,3 +8,5 @@ int insn_inval(struct comedi_device *dev, struct comedi_subdevice *s,
int comedi_alloc_board_minor(struct device *hardware_device);
void comedi_free_board_minor(unsigned minor);
void comedi_reset_async_buf(struct comedi_async *async);
int comedi_buf_alloc(struct comedi_device *dev, struct comedi_subdevice *s,
unsigned long new_size);

0 comments on commit 7029a87

Please sign in to comment.