staging: xillybus: Fixed sparse errors
Changes: * xillybus.h: __iomem added to struct xilly_endpoint -> registers to suppress "different address spaces" errors. * xillybus_core.c: __user added as required for the same reason. * The two member names of struct xilly_endpoint_hardware of the form sync_single_for_{cpu,device} were changed to something that won't look like the well-known functions. * All *.c files: Variables and functions made static as required. Reported-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Signed-off-by: Eli Billauer <eli.billauer@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
853a57f2d3
commit
7ee9ded220
|
@ -122,7 +122,7 @@ struct xilly_endpoint {
|
||||||
|
|
||||||
struct list_head ep_list;
|
struct list_head ep_list;
|
||||||
int dma_using_dac; /* =1 if 64-bit DMA is used, =0 otherwise. */
|
int dma_using_dac; /* =1 if 64-bit DMA is used, =0 otherwise. */
|
||||||
u32 *registers;
|
__iomem u32 *registers;
|
||||||
int fatal_error;
|
int fatal_error;
|
||||||
|
|
||||||
struct mutex register_mutex;
|
struct mutex register_mutex;
|
||||||
|
@ -150,11 +150,11 @@ struct xilly_endpoint {
|
||||||
|
|
||||||
struct xilly_endpoint_hardware {
|
struct xilly_endpoint_hardware {
|
||||||
struct module *owner;
|
struct module *owner;
|
||||||
void (*sync_single_for_cpu)(struct xilly_endpoint *,
|
void (*hw_sync_sgl_for_cpu)(struct xilly_endpoint *,
|
||||||
dma_addr_t,
|
dma_addr_t,
|
||||||
size_t,
|
size_t,
|
||||||
int);
|
int);
|
||||||
void (*sync_single_for_device)(struct xilly_endpoint *,
|
void (*hw_sync_sgl_for_device)(struct xilly_endpoint *,
|
||||||
dma_addr_t,
|
dma_addr_t,
|
||||||
size_t,
|
size_t,
|
||||||
int);
|
int);
|
||||||
|
|
|
@ -71,7 +71,7 @@ static struct class *xillybus_class;
|
||||||
|
|
||||||
static LIST_HEAD(list_of_endpoints);
|
static LIST_HEAD(list_of_endpoints);
|
||||||
static struct mutex ep_list_lock;
|
static struct mutex ep_list_lock;
|
||||||
struct workqueue_struct *xillybus_wq;
|
static struct workqueue_struct *xillybus_wq;
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* Locking scheme: Mutexes protect invocations of character device methods.
|
* Locking scheme: Mutexes protect invocations of character device methods.
|
||||||
|
@ -145,7 +145,7 @@ irqreturn_t xillybus_isr(int irq, void *data)
|
||||||
buf_size = ep->msg_buf_size/sizeof(u32);
|
buf_size = ep->msg_buf_size/sizeof(u32);
|
||||||
|
|
||||||
|
|
||||||
ep->ephw->sync_single_for_cpu(ep,
|
ep->ephw->hw_sync_sgl_for_cpu(ep,
|
||||||
ep->msgbuf_dma_addr,
|
ep->msgbuf_dma_addr,
|
||||||
ep->msg_buf_size,
|
ep->msg_buf_size,
|
||||||
DMA_FROM_DEVICE);
|
DMA_FROM_DEVICE);
|
||||||
|
@ -163,7 +163,7 @@ irqreturn_t xillybus_isr(int irq, void *data)
|
||||||
pr_err("xillybus: Lost sync with "
|
pr_err("xillybus: Lost sync with "
|
||||||
"interrupt messages. Stopping.\n");
|
"interrupt messages. Stopping.\n");
|
||||||
else {
|
else {
|
||||||
ep->ephw->sync_single_for_device(
|
ep->ephw->hw_sync_sgl_for_device(
|
||||||
ep,
|
ep,
|
||||||
ep->msgbuf_dma_addr,
|
ep->msgbuf_dma_addr,
|
||||||
ep->msg_buf_size,
|
ep->msg_buf_size,
|
||||||
|
@ -297,7 +297,7 @@ irqreturn_t xillybus_isr(int irq, void *data)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
ep->ephw->sync_single_for_device(ep,
|
ep->ephw->hw_sync_sgl_for_device(ep,
|
||||||
ep->msgbuf_dma_addr,
|
ep->msgbuf_dma_addr,
|
||||||
ep->msg_buf_size,
|
ep->msg_buf_size,
|
||||||
DMA_FROM_DEVICE);
|
DMA_FROM_DEVICE);
|
||||||
|
@ -796,7 +796,7 @@ static int xilly_obtain_idt(struct xilly_endpoint *endpoint)
|
||||||
return rc;
|
return rc;
|
||||||
}
|
}
|
||||||
|
|
||||||
endpoint->ephw->sync_single_for_cpu(
|
endpoint->ephw->hw_sync_sgl_for_cpu(
|
||||||
channel->endpoint,
|
channel->endpoint,
|
||||||
channel->wr_buffers[0]->dma_addr,
|
channel->wr_buffers[0]->dma_addr,
|
||||||
channel->wr_buf_size,
|
channel->wr_buf_size,
|
||||||
|
@ -832,8 +832,8 @@ static int xilly_obtain_idt(struct xilly_endpoint *endpoint)
|
||||||
return 0; /* Success */
|
return 0; /* Success */
|
||||||
}
|
}
|
||||||
|
|
||||||
static ssize_t xillybus_read(struct file *filp, char *userbuf, size_t count,
|
static ssize_t xillybus_read(struct file *filp, char __user *userbuf,
|
||||||
loff_t *f_pos)
|
size_t count, loff_t *f_pos)
|
||||||
{
|
{
|
||||||
ssize_t rc;
|
ssize_t rc;
|
||||||
unsigned long flags;
|
unsigned long flags;
|
||||||
|
@ -917,7 +917,7 @@ static ssize_t xillybus_read(struct file *filp, char *userbuf, size_t count,
|
||||||
if (!empty) { /* Go on, now without the spinlock */
|
if (!empty) { /* Go on, now without the spinlock */
|
||||||
|
|
||||||
if (bufpos == 0) /* Position zero means it's virgin */
|
if (bufpos == 0) /* Position zero means it's virgin */
|
||||||
channel->endpoint->ephw->sync_single_for_cpu(
|
channel->endpoint->ephw->hw_sync_sgl_for_cpu(
|
||||||
channel->endpoint,
|
channel->endpoint,
|
||||||
channel->wr_buffers[bufidx]->dma_addr,
|
channel->wr_buffers[bufidx]->dma_addr,
|
||||||
channel->wr_buf_size,
|
channel->wr_buf_size,
|
||||||
|
@ -934,7 +934,7 @@ static ssize_t xillybus_read(struct file *filp, char *userbuf, size_t count,
|
||||||
|
|
||||||
if (bufferdone) {
|
if (bufferdone) {
|
||||||
channel->endpoint->ephw->
|
channel->endpoint->ephw->
|
||||||
sync_single_for_device
|
hw_sync_sgl_for_device
|
||||||
(
|
(
|
||||||
channel->endpoint,
|
channel->endpoint,
|
||||||
channel->wr_buffers[bufidx]->
|
channel->wr_buffers[bufidx]->
|
||||||
|
@ -1243,7 +1243,7 @@ static int xillybus_myflush(struct xilly_channel *channel, long timeout)
|
||||||
else
|
else
|
||||||
channel->rd_host_buf_idx++;
|
channel->rd_host_buf_idx++;
|
||||||
|
|
||||||
channel->endpoint->ephw->sync_single_for_device(
|
channel->endpoint->ephw->hw_sync_sgl_for_device(
|
||||||
channel->endpoint,
|
channel->endpoint,
|
||||||
channel->rd_buffers[bufidx]->dma_addr,
|
channel->rd_buffers[bufidx]->dma_addr,
|
||||||
channel->rd_buf_size,
|
channel->rd_buf_size,
|
||||||
|
@ -1362,7 +1362,7 @@ static void xillybus_autoflush(struct work_struct *work)
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static ssize_t xillybus_write(struct file *filp, const char *userbuf,
|
static ssize_t xillybus_write(struct file *filp, const char __user *userbuf,
|
||||||
size_t count, loff_t *f_pos)
|
size_t count, loff_t *f_pos)
|
||||||
{
|
{
|
||||||
ssize_t rc;
|
ssize_t rc;
|
||||||
|
@ -1471,7 +1471,7 @@ static ssize_t xillybus_write(struct file *filp, const char *userbuf,
|
||||||
|
|
||||||
if ((bufpos == 0) || /* Zero means it's virgin */
|
if ((bufpos == 0) || /* Zero means it's virgin */
|
||||||
(channel->rd_leftovers[3] != 0)) {
|
(channel->rd_leftovers[3] != 0)) {
|
||||||
channel->endpoint->ephw->sync_single_for_cpu(
|
channel->endpoint->ephw->hw_sync_sgl_for_cpu(
|
||||||
channel->endpoint,
|
channel->endpoint,
|
||||||
channel->rd_buffers[bufidx]->dma_addr,
|
channel->rd_buffers[bufidx]->dma_addr,
|
||||||
channel->rd_buf_size,
|
channel->rd_buf_size,
|
||||||
|
@ -1494,7 +1494,7 @@ static ssize_t xillybus_write(struct file *filp, const char *userbuf,
|
||||||
|
|
||||||
if (bufferdone) {
|
if (bufferdone) {
|
||||||
channel->endpoint->ephw->
|
channel->endpoint->ephw->
|
||||||
sync_single_for_device(
|
hw_sync_sgl_for_device(
|
||||||
channel->endpoint,
|
channel->endpoint,
|
||||||
channel->rd_buffers[bufidx]->
|
channel->rd_buffers[bufidx]->
|
||||||
dma_addr,
|
dma_addr,
|
||||||
|
@ -1867,7 +1867,7 @@ static int xillybus_release(struct inode *inode, struct file *filp)
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
loff_t xillybus_llseek(struct file *filp, loff_t offset, int whence)
|
static loff_t xillybus_llseek(struct file *filp, loff_t offset, int whence)
|
||||||
{
|
{
|
||||||
struct xilly_channel *channel = filp->private_data;
|
struct xilly_channel *channel = filp->private_data;
|
||||||
loff_t pos = filp->f_pos;
|
loff_t pos = filp->f_pos;
|
||||||
|
|
|
@ -85,7 +85,7 @@ static dma_addr_t xilly_map_single_of(struct xilly_cleanup *mem,
|
||||||
return addr;
|
return addr;
|
||||||
}
|
}
|
||||||
|
|
||||||
void xilly_unmap_single_of(struct xilly_dma *entry)
|
static void xilly_unmap_single_of(struct xilly_dma *entry)
|
||||||
{
|
{
|
||||||
dma_unmap_single(entry->dev,
|
dma_unmap_single(entry->dev,
|
||||||
entry->dma_addr,
|
entry->dma_addr,
|
||||||
|
@ -95,8 +95,8 @@ void xilly_unmap_single_of(struct xilly_dma *entry)
|
||||||
|
|
||||||
static struct xilly_endpoint_hardware of_hw = {
|
static struct xilly_endpoint_hardware of_hw = {
|
||||||
.owner = THIS_MODULE,
|
.owner = THIS_MODULE,
|
||||||
.sync_single_for_cpu = xilly_dma_sync_single_for_cpu_of,
|
.hw_sync_sgl_for_cpu = xilly_dma_sync_single_for_cpu_of,
|
||||||
.sync_single_for_device = xilly_dma_sync_single_for_device_of,
|
.hw_sync_sgl_for_device = xilly_dma_sync_single_for_device_of,
|
||||||
.map_single = xilly_map_single_of,
|
.map_single = xilly_map_single_of,
|
||||||
.unmap_single = xilly_unmap_single_of
|
.unmap_single = xilly_unmap_single_of
|
||||||
};
|
};
|
||||||
|
|
|
@ -112,7 +112,7 @@ static dma_addr_t xilly_map_single_pci(struct xilly_cleanup *mem,
|
||||||
return addr;
|
return addr;
|
||||||
}
|
}
|
||||||
|
|
||||||
void xilly_unmap_single_pci(struct xilly_dma *entry)
|
static void xilly_unmap_single_pci(struct xilly_dma *entry)
|
||||||
{
|
{
|
||||||
pci_unmap_single(entry->pdev,
|
pci_unmap_single(entry->pdev,
|
||||||
entry->dma_addr,
|
entry->dma_addr,
|
||||||
|
@ -122,8 +122,8 @@ void xilly_unmap_single_pci(struct xilly_dma *entry)
|
||||||
|
|
||||||
static struct xilly_endpoint_hardware pci_hw = {
|
static struct xilly_endpoint_hardware pci_hw = {
|
||||||
.owner = THIS_MODULE,
|
.owner = THIS_MODULE,
|
||||||
.sync_single_for_cpu = xilly_dma_sync_single_for_cpu_pci,
|
.hw_sync_sgl_for_cpu = xilly_dma_sync_single_for_cpu_pci,
|
||||||
.sync_single_for_device = xilly_dma_sync_single_for_device_pci,
|
.hw_sync_sgl_for_device = xilly_dma_sync_single_for_device_pci,
|
||||||
.map_single = xilly_map_single_pci,
|
.map_single = xilly_map_single_pci,
|
||||||
.unmap_single = xilly_unmap_single_pci
|
.unmap_single = xilly_unmap_single_pci
|
||||||
};
|
};
|
||||||
|
|
Loading…
Reference in New Issue