diff options
author | Stefan Hajnoczi <stefanha@redhat.com> | 2015-12-17 03:53:43 -0500 |
---|---|---|
committer | Michael S. Tsirkin <mst@redhat.com> | 2016-01-12 13:47:06 -0500 |
commit | f7ad26ff952b3ca2702d7da03aad0ab1f6c01d7c (patch) | |
tree | 37022c77c1ac3d3bcc0e6f74f9ea2588ac30701b | |
parent | 21ea9fb69e7c4b1b1559c3e410943d3ff248ffcb (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.c | 2 | ||||
-rw-r--r-- | drivers/misc/mic/card/mic_virtio.c | 2 | ||||
-rw-r--r-- | drivers/remoteproc/remoteproc_virtio.c | 2 | ||||
-rw-r--r-- | drivers/rpmsg/virtio_rpmsg_bus.c | 2 | ||||
-rw-r--r-- | drivers/s390/virtio/kvm_virtio.c | 2 | ||||
-rw-r--r-- | drivers/s390/virtio/virtio_ccw.c | 2 | ||||
-rw-r--r-- | drivers/virtio/virtio_balloon.c | 2 | ||||
-rw-r--r-- | drivers/virtio/virtio_input.c | 2 | ||||
-rw-r--r-- | drivers/virtio/virtio_mmio.c | 2 | ||||
-rw-r--r-- | drivers/virtio/virtio_pci_common.c | 4 | ||||
-rw-r--r-- | drivers/virtio/virtio_pci_common.h | 2 | ||||
-rw-r--r-- | drivers/virtio/virtio_pci_modern.c | 2 | ||||
-rw-r--r-- | include/linux/virtio_config.h | 2 |
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: | |||
311 | static int mic_find_vqs(struct virtio_device *vdev, unsigned nvqs, | 311 | static 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) | |||
147 | static int rproc_virtio_find_vqs(struct virtio_device *vdev, unsigned nvqs, | 147 | static 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, | |||
945 | static int rpmsg_probe(struct virtio_device *vdev) | 945 | static 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) | |||
255 | static int kvm_find_vqs(struct virtio_device *vdev, unsigned nvqs, | 255 | static 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: | |||
635 | static int virtio_ccw_find_vqs(struct virtio_device *vdev, unsigned nvqs, | 635 | static 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: | |||
482 | static int vm_find_vqs(struct virtio_device *vdev, unsigned nvqs, | 482 | static 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) | |||
296 | static int vp_try_to_find_vqs(struct virtio_device *vdev, unsigned nvqs, | 296 | static 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: | |||
376 | int vp_find_vqs(struct virtio_device *vdev, unsigned nvqs, | 376 | int 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); | |||
139 | int vp_find_vqs(struct virtio_device *vdev, unsigned nvqs, | 139 | int 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[]); |
143 | const char *vp_bus_name(struct virtio_device *vdev); | 143 | const 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: | |||
418 | static int vp_modern_find_vqs(struct virtio_device *vdev, unsigned nvqs, | 418 | static 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); |