aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorStefan Hajnoczi <stefanha@redhat.com>2015-12-17 03:53:43 -0500
committerMichael S. Tsirkin <mst@redhat.com>2016-01-12 13:47:06 -0500
commitf7ad26ff952b3ca2702d7da03aad0ab1f6c01d7c (patch)
tree37022c77c1ac3d3bcc0e6f74f9ea2588ac30701b
parent21ea9fb69e7c4b1b1559c3e410943d3ff248ffcb (diff)
virtio: make find_vqs() checkpatch.pl-friendly
checkpatch.pl wants arrays of strings declared as follows: static const char * const names[] = { "vq-1", "vq-2", "vq-3" }; Currently the find_vqs() function takes a const char *names[] argument so passing checkpatch.pl's const char * const names[] results in a compiler error due to losing the second const. This patch adjusts the find_vqs() prototype and updates all virtio transports. This makes it possible for virtio_balloon.c, virtio_input.c, virtgpu_kms.c, and virtio_rpmsg_bus.c to use the checkpatch.pl-friendly type. Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com> Acked-by: Bjorn Andersson <bjorn.andersson@sonymobile.com>
-rw-r--r--drivers/gpu/drm/virtio/virtgpu_kms.c2
-rw-r--r--drivers/misc/mic/card/mic_virtio.c2
-rw-r--r--drivers/remoteproc/remoteproc_virtio.c2
-rw-r--r--drivers/rpmsg/virtio_rpmsg_bus.c2
-rw-r--r--drivers/s390/virtio/kvm_virtio.c2
-rw-r--r--drivers/s390/virtio/virtio_ccw.c2
-rw-r--r--drivers/virtio/virtio_balloon.c2
-rw-r--r--drivers/virtio/virtio_input.c2
-rw-r--r--drivers/virtio/virtio_mmio.c2
-rw-r--r--drivers/virtio/virtio_pci_common.c4
-rw-r--r--drivers/virtio/virtio_pci_common.h2
-rw-r--r--drivers/virtio/virtio_pci_modern.c2
-rw-r--r--include/linux/virtio_config.h2
13 files changed, 14 insertions, 14 deletions
diff --git a/drivers/gpu/drm/virtio/virtgpu_kms.c b/drivers/gpu/drm/virtio/virtgpu_kms.c
index 06496a128162..4150873d432e 100644
--- a/drivers/gpu/drm/virtio/virtgpu_kms.c
+++ b/drivers/gpu/drm/virtio/virtgpu_kms.c
@@ -130,7 +130,7 @@ int virtio_gpu_driver_load(struct drm_device *dev, unsigned long flags)
130 static vq_callback_t *callbacks[] = { 130 static vq_callback_t *callbacks[] = {
131 virtio_gpu_ctrl_ack, virtio_gpu_cursor_ack 131 virtio_gpu_ctrl_ack, virtio_gpu_cursor_ack
132 }; 132 };
133 static const char *names[] = { "control", "cursor" }; 133 static const char * const names[] = { "control", "cursor" };
134 134
135 struct virtio_gpu_device *vgdev; 135 struct virtio_gpu_device *vgdev;
136 /* this will expand later */ 136 /* this will expand later */
diff --git a/drivers/misc/mic/card/mic_virtio.c b/drivers/misc/mic/card/mic_virtio.c
index e486a0c26267..f6ed57d3125c 100644
--- a/drivers/misc/mic/card/mic_virtio.c
+++ b/drivers/misc/mic/card/mic_virtio.c
@@ -311,7 +311,7 @@ unmap:
311static int mic_find_vqs(struct virtio_device *vdev, unsigned nvqs, 311static int mic_find_vqs(struct virtio_device *vdev, unsigned nvqs,
312 struct virtqueue *vqs[], 312 struct virtqueue *vqs[],
313 vq_callback_t *callbacks[], 313 vq_callback_t *callbacks[],
314 const char *names[]) 314 const char * const names[])
315{ 315{
316 struct mic_vdev *mvdev = to_micvdev(vdev); 316 struct mic_vdev *mvdev = to_micvdev(vdev);
317 struct mic_device_ctrl __iomem *dc = mvdev->dc; 317 struct mic_device_ctrl __iomem *dc = mvdev->dc;
diff --git a/drivers/remoteproc/remoteproc_virtio.c b/drivers/remoteproc/remoteproc_virtio.c
index e1a10232a943..e44872fb9e5e 100644
--- a/drivers/remoteproc/remoteproc_virtio.c
+++ b/drivers/remoteproc/remoteproc_virtio.c
@@ -147,7 +147,7 @@ static void rproc_virtio_del_vqs(struct virtio_device *vdev)
147static int rproc_virtio_find_vqs(struct virtio_device *vdev, unsigned nvqs, 147static int rproc_virtio_find_vqs(struct virtio_device *vdev, unsigned nvqs,
148 struct virtqueue *vqs[], 148 struct virtqueue *vqs[],
149 vq_callback_t *callbacks[], 149 vq_callback_t *callbacks[],
150 const char *names[]) 150 const char * const names[])
151{ 151{
152 struct rproc *rproc = vdev_to_rproc(vdev); 152 struct rproc *rproc = vdev_to_rproc(vdev);
153 int i, ret; 153 int i, ret;
diff --git a/drivers/rpmsg/virtio_rpmsg_bus.c b/drivers/rpmsg/virtio_rpmsg_bus.c
index 73354ee27877..1fcd27c1f183 100644
--- a/drivers/rpmsg/virtio_rpmsg_bus.c
+++ b/drivers/rpmsg/virtio_rpmsg_bus.c
@@ -945,7 +945,7 @@ static void rpmsg_ns_cb(struct rpmsg_channel *rpdev, void *data, int len,
945static int rpmsg_probe(struct virtio_device *vdev) 945static int rpmsg_probe(struct virtio_device *vdev)
946{ 946{
947 vq_callback_t *vq_cbs[] = { rpmsg_recv_done, rpmsg_xmit_done }; 947 vq_callback_t *vq_cbs[] = { rpmsg_recv_done, rpmsg_xmit_done };
948 const char *names[] = { "input", "output" }; 948 static const char * const names[] = { "input", "output" };
949 struct virtqueue *vqs[2]; 949 struct virtqueue *vqs[2];
950 struct virtproc_info *vrp; 950 struct virtproc_info *vrp;
951 void *bufs_va; 951 void *bufs_va;
diff --git a/drivers/s390/virtio/kvm_virtio.c b/drivers/s390/virtio/kvm_virtio.c
index 53fb975c404b..1d060fd293a3 100644
--- a/drivers/s390/virtio/kvm_virtio.c
+++ b/drivers/s390/virtio/kvm_virtio.c
@@ -255,7 +255,7 @@ static void kvm_del_vqs(struct virtio_device *vdev)
255static int kvm_find_vqs(struct virtio_device *vdev, unsigned nvqs, 255static int kvm_find_vqs(struct virtio_device *vdev, unsigned nvqs,
256 struct virtqueue *vqs[], 256 struct virtqueue *vqs[],
257 vq_callback_t *callbacks[], 257 vq_callback_t *callbacks[],
258 const char *names[]) 258 const char * const names[])
259{ 259{
260 struct kvm_device *kdev = to_kvmdev(vdev); 260 struct kvm_device *kdev = to_kvmdev(vdev);
261 int i; 261 int i;
diff --git a/drivers/s390/virtio/virtio_ccw.c b/drivers/s390/virtio/virtio_ccw.c
index 1b831598df7c..bf2d1300a957 100644
--- a/drivers/s390/virtio/virtio_ccw.c
+++ b/drivers/s390/virtio/virtio_ccw.c
@@ -635,7 +635,7 @@ out:
635static int virtio_ccw_find_vqs(struct virtio_device *vdev, unsigned nvqs, 635static int virtio_ccw_find_vqs(struct virtio_device *vdev, unsigned nvqs,
636 struct virtqueue *vqs[], 636 struct virtqueue *vqs[],
637 vq_callback_t *callbacks[], 637 vq_callback_t *callbacks[],
638 const char *names[]) 638 const char * const names[])
639{ 639{
640 struct virtio_ccw_device *vcdev = to_vc_device(vdev); 640 struct virtio_ccw_device *vcdev = to_vc_device(vdev);
641 unsigned long *indicatorp = NULL; 641 unsigned long *indicatorp = NULL;
diff --git a/drivers/virtio/virtio_balloon.c b/drivers/virtio/virtio_balloon.c
index 7d3e5d0e9aa4..0c3691f46575 100644
--- a/drivers/virtio/virtio_balloon.c
+++ b/drivers/virtio/virtio_balloon.c
@@ -388,7 +388,7 @@ static int init_vqs(struct virtio_balloon *vb)
388{ 388{
389 struct virtqueue *vqs[3]; 389 struct virtqueue *vqs[3];
390 vq_callback_t *callbacks[] = { balloon_ack, balloon_ack, stats_request }; 390 vq_callback_t *callbacks[] = { balloon_ack, balloon_ack, stats_request };
391 const char *names[] = { "inflate", "deflate", "stats" }; 391 static const char * const names[] = { "inflate", "deflate", "stats" };
392 int err, nvqs; 392 int err, nvqs;
393 393
394 /* 394 /*
diff --git a/drivers/virtio/virtio_input.c b/drivers/virtio/virtio_input.c
index c96944b59856..350a2a5a49db 100644
--- a/drivers/virtio/virtio_input.c
+++ b/drivers/virtio/virtio_input.c
@@ -170,7 +170,7 @@ static int virtinput_init_vqs(struct virtio_input *vi)
170 struct virtqueue *vqs[2]; 170 struct virtqueue *vqs[2];
171 vq_callback_t *cbs[] = { virtinput_recv_events, 171 vq_callback_t *cbs[] = { virtinput_recv_events,
172 virtinput_recv_status }; 172 virtinput_recv_status };
173 static const char *names[] = { "events", "status" }; 173 static const char * const names[] = { "events", "status" };
174 int err; 174 int err;
175 175
176 err = vi->vdev->config->find_vqs(vi->vdev, 2, vqs, cbs, names); 176 err = vi->vdev->config->find_vqs(vi->vdev, 2, vqs, cbs, names);
diff --git a/drivers/virtio/virtio_mmio.c b/drivers/virtio/virtio_mmio.c
index f499d9da7237..745c6ee1bb3e 100644
--- a/drivers/virtio/virtio_mmio.c
+++ b/drivers/virtio/virtio_mmio.c
@@ -482,7 +482,7 @@ error_available:
482static int vm_find_vqs(struct virtio_device *vdev, unsigned nvqs, 482static int vm_find_vqs(struct virtio_device *vdev, unsigned nvqs,
483 struct virtqueue *vqs[], 483 struct virtqueue *vqs[],
484 vq_callback_t *callbacks[], 484 vq_callback_t *callbacks[],
485 const char *names[]) 485 const char * const names[])
486{ 486{
487 struct virtio_mmio_device *vm_dev = to_virtio_mmio_device(vdev); 487 struct virtio_mmio_device *vm_dev = to_virtio_mmio_device(vdev);
488 unsigned int irq = platform_get_irq(vm_dev->pdev, 0); 488 unsigned int irq = platform_get_irq(vm_dev->pdev, 0);
diff --git a/drivers/virtio/virtio_pci_common.c b/drivers/virtio/virtio_pci_common.c
index 78f804af6c20..36205c27c4d0 100644
--- a/drivers/virtio/virtio_pci_common.c
+++ b/drivers/virtio/virtio_pci_common.c
@@ -296,7 +296,7 @@ void vp_del_vqs(struct virtio_device *vdev)
296static int vp_try_to_find_vqs(struct virtio_device *vdev, unsigned nvqs, 296static int vp_try_to_find_vqs(struct virtio_device *vdev, unsigned nvqs,
297 struct virtqueue *vqs[], 297 struct virtqueue *vqs[],
298 vq_callback_t *callbacks[], 298 vq_callback_t *callbacks[],
299 const char *names[], 299 const char * const names[],
300 bool use_msix, 300 bool use_msix,
301 bool per_vq_vectors) 301 bool per_vq_vectors)
302{ 302{
@@ -376,7 +376,7 @@ error_find:
376int vp_find_vqs(struct virtio_device *vdev, unsigned nvqs, 376int vp_find_vqs(struct virtio_device *vdev, unsigned nvqs,
377 struct virtqueue *vqs[], 377 struct virtqueue *vqs[],
378 vq_callback_t *callbacks[], 378 vq_callback_t *callbacks[],
379 const char *names[]) 379 const char * const names[])
380{ 380{
381 int err; 381 int err;
382 382
diff --git a/drivers/virtio/virtio_pci_common.h b/drivers/virtio/virtio_pci_common.h
index b976d968e793..2cc252270b2d 100644
--- a/drivers/virtio/virtio_pci_common.h
+++ b/drivers/virtio/virtio_pci_common.h
@@ -139,7 +139,7 @@ void vp_del_vqs(struct virtio_device *vdev);
139int vp_find_vqs(struct virtio_device *vdev, unsigned nvqs, 139int vp_find_vqs(struct virtio_device *vdev, unsigned nvqs,
140 struct virtqueue *vqs[], 140 struct virtqueue *vqs[],
141 vq_callback_t *callbacks[], 141 vq_callback_t *callbacks[],
142 const char *names[]); 142 const char * const names[]);
143const char *vp_bus_name(struct virtio_device *vdev); 143const char *vp_bus_name(struct virtio_device *vdev);
144 144
145/* Setup the affinity for a virtqueue: 145/* Setup the affinity for a virtqueue:
diff --git a/drivers/virtio/virtio_pci_modern.c b/drivers/virtio/virtio_pci_modern.c
index 8e5cf194cc0b..c0c11fad4611 100644
--- a/drivers/virtio/virtio_pci_modern.c
+++ b/drivers/virtio/virtio_pci_modern.c
@@ -418,7 +418,7 @@ err_new_queue:
418static int vp_modern_find_vqs(struct virtio_device *vdev, unsigned nvqs, 418static int vp_modern_find_vqs(struct virtio_device *vdev, unsigned nvqs,
419 struct virtqueue *vqs[], 419 struct virtqueue *vqs[],
420 vq_callback_t *callbacks[], 420 vq_callback_t *callbacks[],
421 const char *names[]) 421 const char * const names[])
422{ 422{
423 struct virtio_pci_device *vp_dev = to_vp_device(vdev); 423 struct virtio_pci_device *vp_dev = to_vp_device(vdev);
424 struct virtqueue *vq; 424 struct virtqueue *vq;
diff --git a/include/linux/virtio_config.h b/include/linux/virtio_config.h
index e5ce8ab0b8b0..6e6cb0c9d7cb 100644
--- a/include/linux/virtio_config.h
+++ b/include/linux/virtio_config.h
@@ -70,7 +70,7 @@ struct virtio_config_ops {
70 int (*find_vqs)(struct virtio_device *, unsigned nvqs, 70 int (*find_vqs)(struct virtio_device *, unsigned nvqs,
71 struct virtqueue *vqs[], 71 struct virtqueue *vqs[],
72 vq_callback_t *callbacks[], 72 vq_callback_t *callbacks[],
73 const char *names[]); 73 const char * const names[]);
74 void (*del_vqs)(struct virtio_device *); 74 void (*del_vqs)(struct virtio_device *);
75 u64 (*get_features)(struct virtio_device *vdev); 75 u64 (*get_features)(struct virtio_device *vdev);
76 int (*finalize_features)(struct virtio_device *vdev); 76 int (*finalize_features)(struct virtio_device *vdev);