<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Sep 8, 2020 at 12:07 AM Gerd Hoffmann <<a href="mailto:kraxel@redhat.com">kraxel@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
<br>
Gerd Hoffmann (3):<br>
  drm/virtio: use drmm_mode_config_init<br>
  drm/virtio: return virtio_gpu_queue errors<br>
  drm/virtio: add virtio_gpu_cmd_unref_resource error handling<br></blockquote><div><br></div><div>lgtm +/- nits:</div><div>- add a simple "why" in the first commit message</div><div>- checkpatch --strict reports:</div><div><br></div><div>CHECK: Alignment should match open parenthesis<br>#101: FILE: drivers/gpu/drm/virtio/virtgpu_vq.c:450:<br>+static int virtio_gpu_queue_ctrl_buffer(struct virtio_gpu_device *vgdev,<br>                                         struct virtio_gpu_vbuffer *vbuf)</div><div><br></div><div>Either way, series is:</div><div><br></div><div>Reviewed-by: Gurchetan Singh <<a href="mailto:gurchetansingh@chromium.org">gurchetansingh@chromium.org</a>><br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
 drivers/gpu/drm/virtio/virtgpu_drv.h     |  2 +-<br>
 drivers/gpu/drm/virtio/virtgpu_display.c | 11 ++++---<br>
 drivers/gpu/drm/virtio/virtgpu_kms.c     |  6 +++-<br>
 drivers/gpu/drm/virtio/virtgpu_vq.c      | 41 +++++++++++++-----------<br>
 4 files changed, 36 insertions(+), 24 deletions(-)<br>
<br>
-- <br>
2.27.0<br>
<br>
<br>
_______________________________________________<br>
dri-devel mailing list<br>
<a href="mailto:dri-devel@lists.freedesktop.org" target="_blank">dri-devel@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/dri-devel" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/dri-devel</a><br>
</blockquote></div></div>