* [PATCH 0/7] drm/virtio: Checkpatch cleanup for virtio
@ 2018-02-22 23:59 Rodrigo Siqueira
2018-02-22 23:59 ` [PATCH 1/7] drm/virtio: Add tabs at the start of a line Rodrigo Siqueira
` (8 more replies)
0 siblings, 9 replies; 10+ messages in thread
From: Rodrigo Siqueira @ 2018-02-22 23:59 UTC (permalink / raw)
To: David Airlie, Gerd Hoffmann
Cc: dri-devel, virtualization, linux-kernel, kernel-janitors
This patchset fixes warnings and errors found by checkpatch.pl in the
drm/virtio:
* Removes return from void function;
* Adds */ in block comments to separate line;
* Adds blank line after variable declarations;
* Adds tabs at the start of a line;
* Replaces 'unsigned' for 'unsigned int';
* Removes multiple blank lines;
* Adds spaces around operators.
Rodrigo Siqueira (7):
drm/virtio: Add tabs at the start of a line
drm/virtio: Add blank line after variable declarations
drm/virtio: Add */ in block comments to separate line
drm/virtio: Remove return from void function
drm/virtio: Replace 'unsigned' for 'unsigned int'
drm/virtio: Remove multiple blank lines
drm/virtio: Add spaces around operators
drivers/gpu/drm/virtio/virtgpu_display.c | 7 ++++---
drivers/gpu/drm/virtio/virtgpu_drv.c | 2 +-
drivers/gpu/drm/virtio/virtgpu_drv.h | 8 ++++----
drivers/gpu/drm/virtio/virtgpu_fb.c | 6 +++++-
drivers/gpu/drm/virtio/virtgpu_gem.c | 1 +
drivers/gpu/drm/virtio/virtgpu_ioctl.c | 23 +++++++++++++----------
drivers/gpu/drm/virtio/virtgpu_prime.c | 4 ++--
drivers/gpu/drm/virtio/virtgpu_ttm.c | 4 +---
drivers/gpu/drm/virtio/virtgpu_vq.c | 13 ++++++++-----
9 files changed, 39 insertions(+), 29 deletions(-)
--
2.16.2
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 1/7] drm/virtio: Add tabs at the start of a line
2018-02-22 23:59 [PATCH 0/7] drm/virtio: Checkpatch cleanup for virtio Rodrigo Siqueira
@ 2018-02-22 23:59 ` Rodrigo Siqueira
2018-02-23 0:00 ` [PATCH 2/7] drm/virtio: Add blank line after variable declarations Rodrigo Siqueira
` (7 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Rodrigo Siqueira @ 2018-02-22 23:59 UTC (permalink / raw)
To: David Airlie, Gerd Hoffmann
Cc: dri-devel, virtualization, linux-kernel, kernel-janitors
This patch fixes the checkpatch.pl errors:
drivers/gpu/drm/virtio/virtgpu_drv.h:371: ERROR: code indent should use
tabs where possible
...
Signed-off-by: Rodrigo Siqueira <rodrigosiqueiramelo@gmail.com>
---
drivers/gpu/drm/virtio/virtgpu_display.c | 2 +-
drivers/gpu/drm/virtio/virtgpu_drv.h | 6 +++---
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/virtio/virtgpu_display.c b/drivers/gpu/drm/virtio/virtgpu_display.c
index 19114a3c5ee4..22513d7f895c 100644
--- a/drivers/gpu/drm/virtio/virtgpu_display.c
+++ b/drivers/gpu/drm/virtio/virtgpu_display.c
@@ -387,7 +387,7 @@ int virtio_gpu_modeset_init(struct virtio_gpu_device *vgdev)
for (i = 0 ; i < vgdev->num_scanouts; ++i)
vgdev_output_init(vgdev, i);
- drm_mode_config_reset(vgdev->ddev);
+ drm_mode_config_reset(vgdev->ddev);
return 0;
}
diff --git a/drivers/gpu/drm/virtio/virtgpu_drv.h b/drivers/gpu/drm/virtio/virtgpu_drv.h
index da2fb585fea4..59801728cfeb 100644
--- a/drivers/gpu/drm/virtio/virtgpu_drv.h
+++ b/drivers/gpu/drm/virtio/virtgpu_drv.h
@@ -363,12 +363,12 @@ int virtgpu_gem_prime_pin(struct drm_gem_object *obj);
void virtgpu_gem_prime_unpin(struct drm_gem_object *obj);
struct sg_table *virtgpu_gem_prime_get_sg_table(struct drm_gem_object *obj);
struct drm_gem_object *virtgpu_gem_prime_import_sg_table(
- struct drm_device *dev, struct dma_buf_attachment *attach,
- struct sg_table *sgt);
+ struct drm_device *dev, struct dma_buf_attachment *attach,
+ struct sg_table *sgt);
void *virtgpu_gem_prime_vmap(struct drm_gem_object *obj);
void virtgpu_gem_prime_vunmap(struct drm_gem_object *obj, void *vaddr);
int virtgpu_gem_prime_mmap(struct drm_gem_object *obj,
- struct vm_area_struct *vma);
+ struct vm_area_struct *vma);
static inline struct virtio_gpu_object*
virtio_gpu_object_ref(struct virtio_gpu_object *bo)
--
2.16.2
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 2/7] drm/virtio: Add blank line after variable declarations
2018-02-22 23:59 [PATCH 0/7] drm/virtio: Checkpatch cleanup for virtio Rodrigo Siqueira
2018-02-22 23:59 ` [PATCH 1/7] drm/virtio: Add tabs at the start of a line Rodrigo Siqueira
@ 2018-02-23 0:00 ` Rodrigo Siqueira
2018-02-23 0:00 ` [PATCH 3/7] drm/virtio: Add */ in block comments to separate line Rodrigo Siqueira
` (6 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Rodrigo Siqueira @ 2018-02-23 0:00 UTC (permalink / raw)
To: David Airlie, Gerd Hoffmann
Cc: dri-devel, virtualization, linux-kernel, kernel-janitors
This patch fixes the checkpatch.pl warnings:
virtgpu_drv.c:57: WARNING: Missing a blank line after declarations
virtgpu_display.c:99: WARNING: Missing a blank line after declarations
...
Signed-off-by: Rodrigo Siqueira <rodrigosiqueiramelo@gmail.com>
---
drivers/gpu/drm/virtio/virtgpu_display.c | 1 +
drivers/gpu/drm/virtio/virtgpu_drv.c | 1 +
drivers/gpu/drm/virtio/virtgpu_fb.c | 4 ++++
drivers/gpu/drm/virtio/virtgpu_gem.c | 1 +
drivers/gpu/drm/virtio/virtgpu_ioctl.c | 2 ++
drivers/gpu/drm/virtio/virtgpu_vq.c | 3 +++
6 files changed, 12 insertions(+)
diff --git a/drivers/gpu/drm/virtio/virtgpu_display.c b/drivers/gpu/drm/virtio/virtgpu_display.c
index 22513d7f895c..2d1b25630c0e 100644
--- a/drivers/gpu/drm/virtio/virtgpu_display.c
+++ b/drivers/gpu/drm/virtio/virtgpu_display.c
@@ -96,6 +96,7 @@ virtio_gpu_framebuffer_init(struct drm_device *dev,
{
int ret;
struct virtio_gpu_object *bo;
+
vgfb->obj = obj;
bo = gem_to_virtio_gpu_obj(obj);
diff --git a/drivers/gpu/drm/virtio/virtgpu_drv.c b/drivers/gpu/drm/virtio/virtgpu_drv.c
index 49a3d8d5a249..5d21433b1cdf 100644
--- a/drivers/gpu/drm/virtio/virtgpu_drv.c
+++ b/drivers/gpu/drm/virtio/virtgpu_drv.c
@@ -54,6 +54,7 @@ static int virtio_gpu_probe(struct virtio_device *vdev)
static void virtio_gpu_remove(struct virtio_device *vdev)
{
struct drm_device *dev = vdev->priv;
+
drm_put_dev(dev);
}
diff --git a/drivers/gpu/drm/virtio/virtgpu_fb.c b/drivers/gpu/drm/virtio/virtgpu_fb.c
index 15d18fd0c64b..b933cbba0685 100644
--- a/drivers/gpu/drm/virtio/virtgpu_fb.c
+++ b/drivers/gpu/drm/virtio/virtgpu_fb.c
@@ -127,6 +127,7 @@ int virtio_gpu_surface_dirty(struct virtio_gpu_framebuffer *vgfb,
int left, right, top, bottom;
int i;
int inc = 1;
+
if (!num_clips) {
num_clips = 1;
clips = &norect;
@@ -172,6 +173,7 @@ static void virtio_gpu_3d_fillrect(struct fb_info *info,
const struct fb_fillrect *rect)
{
struct virtio_gpu_fbdev *vfbdev = info->par;
+
drm_fb_helper_sys_fillrect(info, rect);
virtio_gpu_dirty_update(&vfbdev->vgfb, true, rect->dx, rect->dy,
rect->width, rect->height);
@@ -182,6 +184,7 @@ static void virtio_gpu_3d_copyarea(struct fb_info *info,
const struct fb_copyarea *area)
{
struct virtio_gpu_fbdev *vfbdev = info->par;
+
drm_fb_helper_sys_copyarea(info, area);
virtio_gpu_dirty_update(&vfbdev->vgfb, true, area->dx, area->dy,
area->width, area->height);
@@ -192,6 +195,7 @@ static void virtio_gpu_3d_imageblit(struct fb_info *info,
const struct fb_image *image)
{
struct virtio_gpu_fbdev *vfbdev = info->par;
+
drm_fb_helper_sys_imageblit(info, image);
virtio_gpu_dirty_update(&vfbdev->vgfb, true, image->dx, image->dy,
image->width, image->height);
diff --git a/drivers/gpu/drm/virtio/virtgpu_gem.c b/drivers/gpu/drm/virtio/virtgpu_gem.c
index 92fb27753b9e..0f2768eacaee 100644
--- a/drivers/gpu/drm/virtio/virtgpu_gem.c
+++ b/drivers/gpu/drm/virtio/virtgpu_gem.c
@@ -124,6 +124,7 @@ int virtio_gpu_mode_dumb_mmap(struct drm_file *file_priv,
{
struct drm_gem_object *gobj;
struct virtio_gpu_object *obj;
+
BUG_ON(!offset_p);
gobj = drm_gem_object_lookup(file_priv, handle);
if (gobj == NULL)
diff --git a/drivers/gpu/drm/virtio/virtgpu_ioctl.c b/drivers/gpu/drm/virtio/virtgpu_ioctl.c
index 5720a0d4ac0a..86243f131fc1 100644
--- a/drivers/gpu/drm/virtio/virtgpu_ioctl.c
+++ b/drivers/gpu/drm/virtio/virtgpu_ioctl.c
@@ -83,6 +83,7 @@ static void virtio_gpu_unref_list(struct list_head *head)
struct ttm_validate_buffer *buf;
struct ttm_buffer_object *bo;
struct virtio_gpu_object *qobj;
+
list_for_each_entry(buf, head, head) {
bo = buf->bo;
qobj = container_of(bo, struct virtio_gpu_object, tbo);
@@ -478,6 +479,7 @@ static int virtio_gpu_get_caps_ioctl(struct drm_device *dev,
int ret;
struct virtio_gpu_drv_cap_cache *cache_ent;
void *ptr;
+
if (vgdev->num_capsets == 0)
return -ENOSYS;
diff --git a/drivers/gpu/drm/virtio/virtgpu_vq.c b/drivers/gpu/drm/virtio/virtgpu_vq.c
index 9eb96fb2c147..16c05bc34bef 100644
--- a/drivers/gpu/drm/virtio/virtgpu_vq.c
+++ b/drivers/gpu/drm/virtio/virtgpu_vq.c
@@ -62,6 +62,7 @@ void virtio_gpu_ctrl_ack(struct virtqueue *vq)
{
struct drm_device *dev = vq->vdev->priv;
struct virtio_gpu_device *vgdev = dev->dev_private;
+
schedule_work(&vgdev->ctrlq.dequeue_work);
}
@@ -69,6 +70,7 @@ void virtio_gpu_cursor_ack(struct virtqueue *vq)
{
struct drm_device *dev = vq->vdev->priv;
struct virtio_gpu_device *vgdev = dev->dev_private;
+
schedule_work(&vgdev->cursorq.dequeue_work);
}
@@ -852,6 +854,7 @@ int virtio_gpu_object_attach(struct virtio_gpu_device *vgdev,
if (!obj->pages) {
int ret;
+
ret = virtio_gpu_object_get_sg_table(vgdev, obj);
if (ret)
return ret;
--
2.16.2
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 3/7] drm/virtio: Add */ in block comments to separate line
2018-02-22 23:59 [PATCH 0/7] drm/virtio: Checkpatch cleanup for virtio Rodrigo Siqueira
2018-02-22 23:59 ` [PATCH 1/7] drm/virtio: Add tabs at the start of a line Rodrigo Siqueira
2018-02-23 0:00 ` [PATCH 2/7] drm/virtio: Add blank line after variable declarations Rodrigo Siqueira
@ 2018-02-23 0:00 ` Rodrigo Siqueira
2018-02-23 0:00 ` [PATCH 4/7] drm/virtio: Remove return from void function Rodrigo Siqueira
` (5 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Rodrigo Siqueira @ 2018-02-23 0:00 UTC (permalink / raw)
To: David Airlie, Gerd Hoffmann
Cc: dri-devel, virtualization, linux-kernel, kernel-janitors
This patch fixes the checkpatch.pl warning:
virtgpu_ioctl.c:551: WARNING: Block comments use a trailing */ on a
separate line
...
Signed-off-by: Rodrigo Siqueira <rodrigosiqueiramelo@gmail.com>
---
drivers/gpu/drm/virtio/virtgpu_ioctl.c | 3 ++-
drivers/gpu/drm/virtio/virtgpu_prime.c | 3 ++-
drivers/gpu/drm/virtio/virtgpu_vq.c | 3 ++-
3 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/virtio/virtgpu_ioctl.c b/drivers/gpu/drm/virtio/virtgpu_ioctl.c
index 86243f131fc1..d52a86eaa884 100644
--- a/drivers/gpu/drm/virtio/virtgpu_ioctl.c
+++ b/drivers/gpu/drm/virtio/virtgpu_ioctl.c
@@ -548,7 +548,8 @@ struct drm_ioctl_desc virtio_gpu_ioctls[DRM_VIRTIO_NUM_IOCTLS] = {
DRM_AUTH|DRM_UNLOCKED|DRM_RENDER_ALLOW),
/* make transfer async to the main ring? - no sure, can we
- thread these in the underlying GL */
+ * thread these in the underlying GL
+ */
DRM_IOCTL_DEF_DRV(VIRTGPU_TRANSFER_FROM_HOST,
virtio_gpu_transfer_from_host_ioctl,
DRM_AUTH|DRM_UNLOCKED|DRM_RENDER_ALLOW),
diff --git a/drivers/gpu/drm/virtio/virtgpu_prime.c b/drivers/gpu/drm/virtio/virtgpu_prime.c
index 385e0eb9826a..6c6e9dbab096 100644
--- a/drivers/gpu/drm/virtio/virtgpu_prime.c
+++ b/drivers/gpu/drm/virtio/virtgpu_prime.c
@@ -25,7 +25,8 @@
#include "virtgpu_drv.h"
/* Empty Implementations as there should not be any other driver for a virtual
- * device that might share buffers with virtgpu */
+ * device that might share buffers with virtgpu
+ */
int virtgpu_gem_prime_pin(struct drm_gem_object *obj)
{
diff --git a/drivers/gpu/drm/virtio/virtgpu_vq.c b/drivers/gpu/drm/virtio/virtgpu_vq.c
index 16c05bc34bef..809d20eb6571 100644
--- a/drivers/gpu/drm/virtio/virtgpu_vq.c
+++ b/drivers/gpu/drm/virtio/virtgpu_vq.c
@@ -383,7 +383,8 @@ static int virtio_gpu_queue_cursor(struct virtio_gpu_device *vgdev,
}
/* just create gem objects for userspace and long lived objects,
- just use dma_alloced pages for the queue objects? */
+ * just use dma_alloced pages for the queue objects?
+ */
/* create a basic resource */
void virtio_gpu_cmd_create_resource(struct virtio_gpu_device *vgdev,
--
2.16.2
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 4/7] drm/virtio: Remove return from void function
2018-02-22 23:59 [PATCH 0/7] drm/virtio: Checkpatch cleanup for virtio Rodrigo Siqueira
` (2 preceding siblings ...)
2018-02-23 0:00 ` [PATCH 3/7] drm/virtio: Add */ in block comments to separate line Rodrigo Siqueira
@ 2018-02-23 0:00 ` Rodrigo Siqueira
2018-02-23 0:00 ` [PATCH 5/7] drm/virtio: Replace 'unsigned' for 'unsigned int' Rodrigo Siqueira
` (4 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Rodrigo Siqueira @ 2018-02-23 0:00 UTC (permalink / raw)
To: David Airlie, Gerd Hoffmann
Cc: dri-devel, virtualization, linux-kernel, kernel-janitors
This patch fixes the checkpatch.pl warning:
virtgpu_ttm.c:181: WARNING: void function return statements are not
generally useful
...
Signed-off-by: Rodrigo Siqueira <rodrigosiqueiramelo@gmail.com>
---
drivers/gpu/drm/virtio/virtgpu_ttm.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/gpu/drm/virtio/virtgpu_ttm.c b/drivers/gpu/drm/virtio/virtgpu_ttm.c
index 36655b709eb2..cd4a17a1409a 100644
--- a/drivers/gpu/drm/virtio/virtgpu_ttm.c
+++ b/drivers/gpu/drm/virtio/virtgpu_ttm.c
@@ -177,7 +177,6 @@ static void ttm_bo_man_put_node(struct ttm_mem_type_manager *man,
struct ttm_mem_reg *mem)
{
mem->mm_node = (void *)NULL;
- return;
}
static int ttm_bo_man_init(struct ttm_mem_type_manager *man,
@@ -244,7 +243,6 @@ static void virtio_gpu_evict_flags(struct ttm_buffer_object *bo,
placement->busy_placement = &placements;
placement->num_placement = 1;
placement->num_busy_placement = 1;
- return;
}
static int virtio_gpu_verify_access(struct ttm_buffer_object *bo,
--
2.16.2
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 5/7] drm/virtio: Replace 'unsigned' for 'unsigned int'
2018-02-22 23:59 [PATCH 0/7] drm/virtio: Checkpatch cleanup for virtio Rodrigo Siqueira
` (3 preceding siblings ...)
2018-02-23 0:00 ` [PATCH 4/7] drm/virtio: Remove return from void function Rodrigo Siqueira
@ 2018-02-23 0:00 ` Rodrigo Siqueira
2018-02-23 0:00 ` [PATCH 6/7] drm/virtio: Remove multiple blank lines Rodrigo Siqueira
` (3 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Rodrigo Siqueira @ 2018-02-23 0:00 UTC (permalink / raw)
To: David Airlie, Gerd Hoffmann
Cc: dri-devel, virtualization, linux-kernel, kernel-janitors
This patch fixes the checkpatch.pl warning:
drivers/gpu/drm/virtio/virtgpu_display.c:64: WARNING: Prefer 'unsigned
int' to bare use of 'unsigned'
...
Signed-off-by: Rodrigo Siqueira <rodrigosiqueiramelo@gmail.com>
---
drivers/gpu/drm/virtio/virtgpu_display.c | 4 ++--
drivers/gpu/drm/virtio/virtgpu_drv.h | 2 +-
drivers/gpu/drm/virtio/virtgpu_fb.c | 2 +-
drivers/gpu/drm/virtio/virtgpu_ttm.c | 2 +-
4 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/virtio/virtgpu_display.c b/drivers/gpu/drm/virtio/virtgpu_display.c
index 2d1b25630c0e..8cc8c34d67f5 100644
--- a/drivers/gpu/drm/virtio/virtgpu_display.c
+++ b/drivers/gpu/drm/virtio/virtgpu_display.c
@@ -61,9 +61,9 @@ static void virtio_gpu_user_framebuffer_destroy(struct drm_framebuffer *fb)
static int
virtio_gpu_framebuffer_surface_dirty(struct drm_framebuffer *fb,
struct drm_file *file_priv,
- unsigned flags, unsigned color,
+ unsigned int flags, unsigned int color,
struct drm_clip_rect *clips,
- unsigned num_clips)
+ unsigned int num_clips)
{
struct virtio_gpu_framebuffer *virtio_gpu_fb
= to_virtio_gpu_framebuffer(fb);
diff --git a/drivers/gpu/drm/virtio/virtgpu_drv.h b/drivers/gpu/drm/virtio/virtgpu_drv.h
index 59801728cfeb..d25c8ca224aa 100644
--- a/drivers/gpu/drm/virtio/virtgpu_drv.h
+++ b/drivers/gpu/drm/virtio/virtgpu_drv.h
@@ -246,7 +246,7 @@ int virtio_gpu_fbdev_init(struct virtio_gpu_device *vgdev);
void virtio_gpu_fbdev_fini(struct virtio_gpu_device *vgdev);
int virtio_gpu_surface_dirty(struct virtio_gpu_framebuffer *qfb,
struct drm_clip_rect *clips,
- unsigned num_clips);
+ unsigned int num_clips);
/* virtio vg */
int virtio_gpu_alloc_vbufs(struct virtio_gpu_device *vgdev);
void virtio_gpu_free_vbufs(struct virtio_gpu_device *vgdev);
diff --git a/drivers/gpu/drm/virtio/virtgpu_fb.c b/drivers/gpu/drm/virtio/virtgpu_fb.c
index b933cbba0685..8af69ab58b89 100644
--- a/drivers/gpu/drm/virtio/virtgpu_fb.c
+++ b/drivers/gpu/drm/virtio/virtgpu_fb.c
@@ -118,7 +118,7 @@ static int virtio_gpu_dirty_update(struct virtio_gpu_framebuffer *fb,
int virtio_gpu_surface_dirty(struct virtio_gpu_framebuffer *vgfb,
struct drm_clip_rect *clips,
- unsigned num_clips)
+ unsigned int num_clips)
{
struct virtio_gpu_device *vgdev = vgfb->base.dev->dev_private;
struct virtio_gpu_object *obj = gem_to_virtio_gpu_obj(vgfb->obj);
diff --git a/drivers/gpu/drm/virtio/virtgpu_ttm.c b/drivers/gpu/drm/virtio/virtgpu_ttm.c
index cd4a17a1409a..580323ceeb8a 100644
--- a/drivers/gpu/drm/virtio/virtgpu_ttm.c
+++ b/drivers/gpu/drm/virtio/virtgpu_ttm.c
@@ -224,7 +224,7 @@ static int virtio_gpu_init_mem_type(struct ttm_bo_device *bdev, uint32_t type,
man->default_caching = TTM_PL_FLAG_CACHED;
break;
default:
- DRM_ERROR("Unsupported memory type %u\n", (unsigned)type);
+ DRM_ERROR("Unsupported memory type %u\n", (unsigned int)type);
return -EINVAL;
}
return 0;
--
2.16.2
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 6/7] drm/virtio: Remove multiple blank lines
2018-02-22 23:59 [PATCH 0/7] drm/virtio: Checkpatch cleanup for virtio Rodrigo Siqueira
` (4 preceding siblings ...)
2018-02-23 0:00 ` [PATCH 5/7] drm/virtio: Replace 'unsigned' for 'unsigned int' Rodrigo Siqueira
@ 2018-02-23 0:00 ` Rodrigo Siqueira
2018-02-23 0:01 ` [PATCH 7/7] drm/virtio: Add spaces around operators Rodrigo Siqueira
` (2 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Rodrigo Siqueira @ 2018-02-23 0:00 UTC (permalink / raw)
To: David Airlie, Gerd Hoffmann
Cc: dri-devel, virtualization, linux-kernel, kernel-janitors
This patch fixes the checkpatch.pl check:
virtgpu_drv.c:116: CHECK: Please don't use multiple blank lines
virtgpu_vq.c:599: CHECK: Please don't use multiple blank lines
virtgpu_prime.c:42: CHECK: Please don't use multiple blank lines
Signed-off-by: Rodrigo Siqueira <rodrigosiqueiramelo@gmail.com>
---
drivers/gpu/drm/virtio/virtgpu_drv.c | 1 -
drivers/gpu/drm/virtio/virtgpu_prime.c | 1 -
drivers/gpu/drm/virtio/virtgpu_vq.c | 1 -
3 files changed, 3 deletions(-)
diff --git a/drivers/gpu/drm/virtio/virtgpu_drv.c b/drivers/gpu/drm/virtio/virtgpu_drv.c
index 5d21433b1cdf..d9287c144fe5 100644
--- a/drivers/gpu/drm/virtio/virtgpu_drv.c
+++ b/drivers/gpu/drm/virtio/virtgpu_drv.c
@@ -113,7 +113,6 @@ static const struct file_operations virtio_gpu_driver_fops = {
.llseek = noop_llseek,
};
-
static struct drm_driver driver = {
.driver_features = DRIVER_MODESET | DRIVER_GEM | DRIVER_PRIME | DRIVER_RENDER | DRIVER_ATOMIC,
.load = virtio_gpu_driver_load,
diff --git a/drivers/gpu/drm/virtio/virtgpu_prime.c b/drivers/gpu/drm/virtio/virtgpu_prime.c
index 6c6e9dbab096..d27a1688714f 100644
--- a/drivers/gpu/drm/virtio/virtgpu_prime.c
+++ b/drivers/gpu/drm/virtio/virtgpu_prime.c
@@ -39,7 +39,6 @@ void virtgpu_gem_prime_unpin(struct drm_gem_object *obj)
WARN_ONCE(1, "not implemented");
}
-
struct sg_table *virtgpu_gem_prime_get_sg_table(struct drm_gem_object *obj)
{
WARN_ONCE(1, "not implemented");
diff --git a/drivers/gpu/drm/virtio/virtgpu_vq.c b/drivers/gpu/drm/virtio/virtgpu_vq.c
index 809d20eb6571..18c78a7fa044 100644
--- a/drivers/gpu/drm/virtio/virtgpu_vq.c
+++ b/drivers/gpu/drm/virtio/virtgpu_vq.c
@@ -596,7 +596,6 @@ static void virtio_gpu_cmd_capset_cb(struct virtio_gpu_device *vgdev,
wake_up(&vgdev->resp_wq);
}
-
int virtio_gpu_cmd_get_display_info(struct virtio_gpu_device *vgdev)
{
struct virtio_gpu_ctrl_hdr *cmd_p;
--
2.16.2
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 7/7] drm/virtio: Add spaces around operators
2018-02-22 23:59 [PATCH 0/7] drm/virtio: Checkpatch cleanup for virtio Rodrigo Siqueira
` (5 preceding siblings ...)
2018-02-23 0:00 ` [PATCH 6/7] drm/virtio: Remove multiple blank lines Rodrigo Siqueira
@ 2018-02-23 0:01 ` Rodrigo Siqueira
2018-02-23 2:27 ` [PATCH 0/7] drm/virtio: Checkpatch cleanup for virtio Gurchetan Singh
2018-02-27 6:47 ` Gerd Hoffmann
8 siblings, 0 replies; 10+ messages in thread
From: Rodrigo Siqueira @ 2018-02-23 0:01 UTC (permalink / raw)
To: David Airlie, Gerd Hoffmann
Cc: dri-devel, virtualization, linux-kernel, kernel-janitors
This patch fixes the checkpatch.pl check:
virtgpu_ioctl.c:535: CHECK: spaces preferred around that '|' (ctx:VxV)
virtgpu_vq.c:277: CHECK: spaces preferred around that '+' (ctx:VxV)
...
Signed-off-by: Rodrigo Siqueira <rodrigosiqueiramelo@gmail.com>
---
drivers/gpu/drm/virtio/virtgpu_ioctl.c | 18 +++++++++---------
drivers/gpu/drm/virtio/virtgpu_vq.c | 6 +++---
2 files changed, 12 insertions(+), 12 deletions(-)
diff --git a/drivers/gpu/drm/virtio/virtgpu_ioctl.c b/drivers/gpu/drm/virtio/virtgpu_ioctl.c
index d52a86eaa884..04e09fc56f66 100644
--- a/drivers/gpu/drm/virtio/virtgpu_ioctl.c
+++ b/drivers/gpu/drm/virtio/virtgpu_ioctl.c
@@ -532,34 +532,34 @@ static int virtio_gpu_get_caps_ioctl(struct drm_device *dev,
struct drm_ioctl_desc virtio_gpu_ioctls[DRM_VIRTIO_NUM_IOCTLS] = {
DRM_IOCTL_DEF_DRV(VIRTGPU_MAP, virtio_gpu_map_ioctl,
- DRM_AUTH|DRM_UNLOCKED|DRM_RENDER_ALLOW),
+ DRM_AUTH | DRM_UNLOCKED | DRM_RENDER_ALLOW),
DRM_IOCTL_DEF_DRV(VIRTGPU_EXECBUFFER, virtio_gpu_execbuffer_ioctl,
- DRM_AUTH|DRM_UNLOCKED|DRM_RENDER_ALLOW),
+ DRM_AUTH | DRM_UNLOCKED | DRM_RENDER_ALLOW),
DRM_IOCTL_DEF_DRV(VIRTGPU_GETPARAM, virtio_gpu_getparam_ioctl,
- DRM_AUTH|DRM_UNLOCKED|DRM_RENDER_ALLOW),
+ DRM_AUTH | DRM_UNLOCKED | DRM_RENDER_ALLOW),
DRM_IOCTL_DEF_DRV(VIRTGPU_RESOURCE_CREATE,
virtio_gpu_resource_create_ioctl,
- DRM_AUTH|DRM_UNLOCKED|DRM_RENDER_ALLOW),
+ DRM_AUTH | DRM_UNLOCKED | DRM_RENDER_ALLOW),
DRM_IOCTL_DEF_DRV(VIRTGPU_RESOURCE_INFO, virtio_gpu_resource_info_ioctl,
- DRM_AUTH|DRM_UNLOCKED|DRM_RENDER_ALLOW),
+ DRM_AUTH | DRM_UNLOCKED | DRM_RENDER_ALLOW),
/* make transfer async to the main ring? - no sure, can we
* thread these in the underlying GL
*/
DRM_IOCTL_DEF_DRV(VIRTGPU_TRANSFER_FROM_HOST,
virtio_gpu_transfer_from_host_ioctl,
- DRM_AUTH|DRM_UNLOCKED|DRM_RENDER_ALLOW),
+ DRM_AUTH | DRM_UNLOCKED | DRM_RENDER_ALLOW),
DRM_IOCTL_DEF_DRV(VIRTGPU_TRANSFER_TO_HOST,
virtio_gpu_transfer_to_host_ioctl,
- DRM_AUTH|DRM_UNLOCKED|DRM_RENDER_ALLOW),
+ DRM_AUTH | DRM_UNLOCKED | DRM_RENDER_ALLOW),
DRM_IOCTL_DEF_DRV(VIRTGPU_WAIT, virtio_gpu_wait_ioctl,
- DRM_AUTH|DRM_UNLOCKED|DRM_RENDER_ALLOW),
+ DRM_AUTH | DRM_UNLOCKED | DRM_RENDER_ALLOW),
DRM_IOCTL_DEF_DRV(VIRTGPU_GET_CAPS, virtio_gpu_get_caps_ioctl,
- DRM_AUTH|DRM_UNLOCKED|DRM_RENDER_ALLOW),
+ DRM_AUTH | DRM_UNLOCKED | DRM_RENDER_ALLOW),
};
diff --git a/drivers/gpu/drm/virtio/virtgpu_vq.c b/drivers/gpu/drm/virtio/virtgpu_vq.c
index 18c78a7fa044..48e4f1df6e5d 100644
--- a/drivers/gpu/drm/virtio/virtgpu_vq.c
+++ b/drivers/gpu/drm/virtio/virtgpu_vq.c
@@ -274,7 +274,7 @@ static int virtio_gpu_queue_ctrl_buffer_locked(struct virtio_gpu_device *vgdev,
return -ENODEV;
sg_init_one(&vcmd, vbuf->buf, vbuf->size);
- sgs[outcnt+incnt] = &vcmd;
+ sgs[outcnt + incnt] = &vcmd;
outcnt++;
if (vbuf->data_size) {
@@ -709,8 +709,8 @@ void virtio_gpu_cmd_context_create(struct virtio_gpu_device *vgdev, uint32_t id,
cmd_p->hdr.type = cpu_to_le32(VIRTIO_GPU_CMD_CTX_CREATE);
cmd_p->hdr.ctx_id = cpu_to_le32(id);
cmd_p->nlen = cpu_to_le32(nlen);
- strncpy(cmd_p->debug_name, name, sizeof(cmd_p->debug_name)-1);
- cmd_p->debug_name[sizeof(cmd_p->debug_name)-1] = 0;
+ strncpy(cmd_p->debug_name, name, sizeof(cmd_p->debug_name) - 1);
+ cmd_p->debug_name[sizeof(cmd_p->debug_name) - 1] = 0;
virtio_gpu_queue_ctrl_buffer(vgdev, vbuf);
}
--
2.16.2
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 0/7] drm/virtio: Checkpatch cleanup for virtio
2018-02-22 23:59 [PATCH 0/7] drm/virtio: Checkpatch cleanup for virtio Rodrigo Siqueira
` (6 preceding siblings ...)
2018-02-23 0:01 ` [PATCH 7/7] drm/virtio: Add spaces around operators Rodrigo Siqueira
@ 2018-02-23 2:27 ` Gurchetan Singh
2018-02-27 6:47 ` Gerd Hoffmann
8 siblings, 0 replies; 10+ messages in thread
From: Gurchetan Singh @ 2018-02-23 2:27 UTC (permalink / raw)
To: Rodrigo Siqueira
Cc: David Airlie, Gerd Hoffmann, kernel-janitors, linux-kernel,
ML dri-devel, virtualization
Patches 1 to 7 are:
Reviewed-by: Gurchetan Singh <gurchetansingh@chromium.org>
On Thu, Feb 22, 2018 at 3:59 PM, Rodrigo Siqueira
<rodrigosiqueiramelo@gmail.com> wrote:
> This patchset fixes warnings and errors found by checkpatch.pl in the
> drm/virtio:
>
> * Removes return from void function;
> * Adds */ in block comments to separate line;
> * Adds blank line after variable declarations;
> * Adds tabs at the start of a line;
> * Replaces 'unsigned' for 'unsigned int';
> * Removes multiple blank lines;
> * Adds spaces around operators.
>
> Rodrigo Siqueira (7):
> drm/virtio: Add tabs at the start of a line
> drm/virtio: Add blank line after variable declarations
> drm/virtio: Add */ in block comments to separate line
> drm/virtio: Remove return from void function
> drm/virtio: Replace 'unsigned' for 'unsigned int'
> drm/virtio: Remove multiple blank lines
> drm/virtio: Add spaces around operators
>
> drivers/gpu/drm/virtio/virtgpu_display.c | 7 ++++---
> drivers/gpu/drm/virtio/virtgpu_drv.c | 2 +-
> drivers/gpu/drm/virtio/virtgpu_drv.h | 8 ++++----
> drivers/gpu/drm/virtio/virtgpu_fb.c | 6 +++++-
> drivers/gpu/drm/virtio/virtgpu_gem.c | 1 +
> drivers/gpu/drm/virtio/virtgpu_ioctl.c | 23 +++++++++++++----------
> drivers/gpu/drm/virtio/virtgpu_prime.c | 4 ++--
> drivers/gpu/drm/virtio/virtgpu_ttm.c | 4 +---
> drivers/gpu/drm/virtio/virtgpu_vq.c | 13 ++++++++-----
> 9 files changed, 39 insertions(+), 29 deletions(-)
>
> --
> 2.16.2
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 0/7] drm/virtio: Checkpatch cleanup for virtio
2018-02-22 23:59 [PATCH 0/7] drm/virtio: Checkpatch cleanup for virtio Rodrigo Siqueira
` (7 preceding siblings ...)
2018-02-23 2:27 ` [PATCH 0/7] drm/virtio: Checkpatch cleanup for virtio Gurchetan Singh
@ 2018-02-27 6:47 ` Gerd Hoffmann
8 siblings, 0 replies; 10+ messages in thread
From: Gerd Hoffmann @ 2018-02-27 6:47 UTC (permalink / raw)
To: Rodrigo Siqueira
Cc: David Airlie, dri-devel, virtualization, linux-kernel, kernel-janitors
On Thu, Feb 22, 2018 at 08:59:33PM -0300, Rodrigo Siqueira wrote:
> This patchset fixes warnings and errors found by checkpatch.pl in the
> drm/virtio:
Added to drm-qemu queue, will land in drm-misc soon.
thanks,
Gerd
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2018-02-27 6:47 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-02-22 23:59 [PATCH 0/7] drm/virtio: Checkpatch cleanup for virtio Rodrigo Siqueira
2018-02-22 23:59 ` [PATCH 1/7] drm/virtio: Add tabs at the start of a line Rodrigo Siqueira
2018-02-23 0:00 ` [PATCH 2/7] drm/virtio: Add blank line after variable declarations Rodrigo Siqueira
2018-02-23 0:00 ` [PATCH 3/7] drm/virtio: Add */ in block comments to separate line Rodrigo Siqueira
2018-02-23 0:00 ` [PATCH 4/7] drm/virtio: Remove return from void function Rodrigo Siqueira
2018-02-23 0:00 ` [PATCH 5/7] drm/virtio: Replace 'unsigned' for 'unsigned int' Rodrigo Siqueira
2018-02-23 0:00 ` [PATCH 6/7] drm/virtio: Remove multiple blank lines Rodrigo Siqueira
2018-02-23 0:01 ` [PATCH 7/7] drm/virtio: Add spaces around operators Rodrigo Siqueira
2018-02-23 2:27 ` [PATCH 0/7] drm/virtio: Checkpatch cleanup for virtio Gurchetan Singh
2018-02-27 6:47 ` Gerd Hoffmann
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).