* [PATCH] drm/virtio: clean up minor codestyle issues
@ 2025-08-13 6:21 Athul Raj Kollareth
2025-08-20 10:41 ` Dmitry Osipenko
0 siblings, 1 reply; 3+ messages in thread
From: Athul Raj Kollareth @ 2025-08-13 6:21 UTC (permalink / raw)
To: airlied, kraxel, dmitry.osipenko, gurchetansingh
Cc: Athul Raj Kollareth, dri-devel, virtualization, skhan,
linux-kernel-mentees, linux-kernel
Fix codestyle warnings and errors generated by CHECKPATCH in virtio
source files.
Signed-off-by: Athul Raj Kollareth <krathul3152@gmail.com>
---
drivers/gpu/drm/virtio/virtgpu_display.c | 3 +--
drivers/gpu/drm/virtio/virtgpu_kms.c | 20 ++++++++++----------
drivers/gpu/drm/virtio/virtgpu_object.c | 4 ++--
drivers/gpu/drm/virtio/virtgpu_plane.c | 2 +-
drivers/gpu/drm/virtio/virtgpu_vq.c | 2 ++
5 files changed, 16 insertions(+), 15 deletions(-)
diff --git a/drivers/gpu/drm/virtio/virtgpu_display.c b/drivers/gpu/drm/virtio/virtgpu_display.c
index e5805ca646c7..c3315935d8bc 100644
--- a/drivers/gpu/drm/virtio/virtgpu_display.c
+++ b/drivers/gpu/drm/virtio/virtgpu_display.c
@@ -131,9 +131,8 @@ static void virtio_gpu_crtc_atomic_flush(struct drm_crtc *crtc,
* in the plane update callback, and here we just check
* whenever we must force the modeset.
*/
- if (drm_atomic_crtc_needs_modeset(crtc_state)) {
+ if (drm_atomic_crtc_needs_modeset(crtc_state))
output->needs_modeset = true;
- }
}
static const struct drm_crtc_helper_funcs virtio_gpu_crtc_helper_funcs = {
diff --git a/drivers/gpu/drm/virtio/virtgpu_kms.c b/drivers/gpu/drm/virtio/virtgpu_kms.c
index 7dfb2006c561..1c15cbf326b7 100644
--- a/drivers/gpu/drm/virtio/virtgpu_kms.c
+++ b/drivers/gpu/drm/virtio/virtgpu_kms.c
@@ -162,18 +162,18 @@ int virtio_gpu_init(struct virtio_device *vdev, struct drm_device *dev)
if (virtio_has_feature(vgdev->vdev, VIRTIO_GPU_F_VIRGL))
vgdev->has_virgl_3d = true;
#endif
- if (virtio_has_feature(vgdev->vdev, VIRTIO_GPU_F_EDID)) {
+ if (virtio_has_feature(vgdev->vdev, VIRTIO_GPU_F_EDID))
vgdev->has_edid = true;
- }
- if (virtio_has_feature(vgdev->vdev, VIRTIO_RING_F_INDIRECT_DESC)) {
+
+ if (virtio_has_feature(vgdev->vdev, VIRTIO_RING_F_INDIRECT_DESC))
vgdev->has_indirect = true;
- }
- if (virtio_has_feature(vgdev->vdev, VIRTIO_GPU_F_RESOURCE_UUID)) {
+
+ if (virtio_has_feature(vgdev->vdev, VIRTIO_GPU_F_RESOURCE_UUID))
vgdev->has_resource_assign_uuid = true;
- }
- if (virtio_has_feature(vgdev->vdev, VIRTIO_GPU_F_RESOURCE_BLOB)) {
+
+ if (virtio_has_feature(vgdev->vdev, VIRTIO_GPU_F_RESOURCE_BLOB))
vgdev->has_resource_blob = true;
- }
+
if (virtio_get_shm_region(vgdev->vdev, &vgdev->host_visible_region,
VIRTIO_GPU_SHM_ID_HOST_VISIBLE)) {
if (!devm_request_mem_region(&vgdev->vdev->dev,
@@ -193,9 +193,9 @@ int virtio_gpu_init(struct virtio_device *vdev, struct drm_device *dev)
(unsigned long)vgdev->host_visible_region.addr,
(unsigned long)vgdev->host_visible_region.len);
}
- if (virtio_has_feature(vgdev->vdev, VIRTIO_GPU_F_CONTEXT_INIT)) {
+
+ if (virtio_has_feature(vgdev->vdev, VIRTIO_GPU_F_CONTEXT_INIT))
vgdev->has_context_init = true;
- }
DRM_INFO("features: %cvirgl %cedid %cresource_blob %chost_visible",
vgdev->has_virgl_3d ? '+' : '-',
diff --git a/drivers/gpu/drm/virtio/virtgpu_object.c b/drivers/gpu/drm/virtio/virtgpu_object.c
index 5517cff8715c..e6363c887500 100644
--- a/drivers/gpu/drm/virtio/virtgpu_object.c
+++ b/drivers/gpu/drm/virtio/virtgpu_object.c
@@ -47,6 +47,7 @@ int virtio_gpu_resource_id_get(struct virtio_gpu_device *vgdev, uint32_t *resid)
*resid = handle + 1;
} else {
int handle = ida_alloc(&vgdev->resource_ida, GFP_KERNEL);
+
if (handle < 0)
return handle;
*resid = handle + 1;
@@ -56,9 +57,8 @@ int virtio_gpu_resource_id_get(struct virtio_gpu_device *vgdev, uint32_t *resid)
static void virtio_gpu_resource_id_put(struct virtio_gpu_device *vgdev, uint32_t id)
{
- if (!virtio_gpu_virglrenderer_workaround) {
+ if (!virtio_gpu_virglrenderer_workaround)
ida_free(&vgdev->resource_ida, id - 1);
- }
}
void virtio_gpu_cleanup_object(struct virtio_gpu_object *bo)
diff --git a/drivers/gpu/drm/virtio/virtgpu_plane.c b/drivers/gpu/drm/virtio/virtgpu_plane.c
index 698ea7adb951..29e4b458ae57 100644
--- a/drivers/gpu/drm/virtio/virtgpu_plane.c
+++ b/drivers/gpu/drm/virtio/virtgpu_plane.c
@@ -120,7 +120,7 @@ static int virtio_gpu_plane_atomic_check(struct drm_plane *plane,
crtc_state = drm_atomic_get_crtc_state(state,
new_plane_state->crtc);
if (IS_ERR(crtc_state))
- return PTR_ERR(crtc_state);
+ return PTR_ERR(crtc_state);
ret = drm_atomic_helper_check_plane_state(new_plane_state, crtc_state,
DRM_PLANE_NO_SCALING,
diff --git a/drivers/gpu/drm/virtio/virtgpu_vq.c b/drivers/gpu/drm/virtio/virtgpu_vq.c
index 55a15e247dd1..8181b22b9b46 100644
--- a/drivers/gpu/drm/virtio/virtgpu_vq.c
+++ b/drivers/gpu/drm/virtio/virtgpu_vq.c
@@ -248,6 +248,7 @@ void virtio_gpu_dequeue_ctrl_func(struct work_struct *work)
if (resp->type != cpu_to_le32(VIRTIO_GPU_RESP_OK_NODATA)) {
if (le32_to_cpu(resp->type) >= VIRTIO_GPU_RESP_ERR_UNSPEC) {
struct virtio_gpu_ctrl_hdr *cmd;
+
cmd = virtio_gpu_vbuf_ctrl_hdr(entry);
DRM_ERROR_RATELIMITED("response 0x%x (command 0x%x)\n",
le32_to_cpu(resp->type),
@@ -468,6 +469,7 @@ static int virtio_gpu_queue_fenced_ctrl_buffer(struct virtio_gpu_device *vgdev,
if (vbuf->data_size) {
if (is_vmalloc_addr(vbuf->data_buf)) {
int sg_ents;
+
sgt = vmalloc_to_sgt(vbuf->data_buf, vbuf->data_size,
&sg_ents);
if (!sgt) {
--
2.50.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] drm/virtio: clean up minor codestyle issues
2025-08-13 6:21 [PATCH] drm/virtio: clean up minor codestyle issues Athul Raj Kollareth
@ 2025-08-20 10:41 ` Dmitry Osipenko
2025-08-20 16:50 ` Athul Raj Kollareth
0 siblings, 1 reply; 3+ messages in thread
From: Dmitry Osipenko @ 2025-08-20 10:41 UTC (permalink / raw)
To: Athul Raj Kollareth, airlied, kraxel, gurchetansingh
Cc: dri-devel, virtualization, skhan, linux-kernel-mentees,
linux-kernel
On 8/13/25 09:21, Athul Raj Kollareth wrote:
> Fix codestyle warnings and errors generated by CHECKPATCH in virtio
> source files.
>
> Signed-off-by: Athul Raj Kollareth <krathul3152@gmail.com>
> ---
> drivers/gpu/drm/virtio/virtgpu_display.c | 3 +--
> drivers/gpu/drm/virtio/virtgpu_kms.c | 20 ++++++++++----------
> drivers/gpu/drm/virtio/virtgpu_object.c | 4 ++--
> drivers/gpu/drm/virtio/virtgpu_plane.c | 2 +-
> drivers/gpu/drm/virtio/virtgpu_vq.c | 2 ++
> 5 files changed, 16 insertions(+), 15 deletions(-)
Applied to misc-next, thanks.
--
Best regards,
Dmitry
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] drm/virtio: clean up minor codestyle issues
2025-08-20 10:41 ` Dmitry Osipenko
@ 2025-08-20 16:50 ` Athul Raj Kollareth
0 siblings, 0 replies; 3+ messages in thread
From: Athul Raj Kollareth @ 2025-08-20 16:50 UTC (permalink / raw)
To: Dmitry Osipenko
Cc: airlied, kraxel, gurchetansingh, dri-devel, virtualization, skhan,
linux-kernel-mentees, linux-kernel
Thank you for applying the changes.
Best regards,
Athul
On Wed, Aug 20, 2025 at 01:41:01PM +0300, Dmitry Osipenko wrote:
> On 8/13/25 09:21, Athul Raj Kollareth wrote:
> > Fix codestyle warnings and errors generated by CHECKPATCH in virtio
> > source files.
> >
> > Signed-off-by: Athul Raj Kollareth <krathul3152@gmail.com>
> > ---
> > drivers/gpu/drm/virtio/virtgpu_display.c | 3 +--
> > drivers/gpu/drm/virtio/virtgpu_kms.c | 20 ++++++++++----------
> > drivers/gpu/drm/virtio/virtgpu_object.c | 4 ++--
> > drivers/gpu/drm/virtio/virtgpu_plane.c | 2 +-
> > drivers/gpu/drm/virtio/virtgpu_vq.c | 2 ++
> > 5 files changed, 16 insertions(+), 15 deletions(-)
>
> Applied to misc-next, thanks.
>
> --
> Best regards,
> Dmitry
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2025-08-20 16:51 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-08-13 6:21 [PATCH] drm/virtio: clean up minor codestyle issues Athul Raj Kollareth
2025-08-20 10:41 ` Dmitry Osipenko
2025-08-20 16:50 ` Athul Raj Kollareth
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).