Reviewed-by: David Stevens stevensd@chromium.org

On Tue, Oct 13, 2020 at 2:18 AM Andy Shevchenko andriy.shevchenko@linux.intel.com wrote:

There is import_uuid() function which imports u8 array to the uuid_t. Use it instead of open coding variant.

This allows to hide the uuid_t internals.

Signed-off-by: Andy Shevchenko andriy.shevchenko@linux.intel.com

drivers/gpu/drm/virtio/virtgpu_vq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/virtio/virtgpu_vq.c b/drivers/gpu/drm/virtio/virtgpu_vq.c index 07945ca238e2..8944cc0bf8eb 100644 --- a/drivers/gpu/drm/virtio/virtgpu_vq.c +++ b/drivers/gpu/drm/virtio/virtgpu_vq.c @@ -1129,7 +1129,7 @@ static void virtio_gpu_cmd_resource_uuid_cb(struct virtio_gpu_device *vgdev,

    if (resp_type == VIRTIO_GPU_RESP_OK_RESOURCE_UUID &&
        obj->uuid_state == UUID_INITIALIZING) {

•           memcpy(&obj->uuid.b, resp->uuid, sizeof(obj->uuid.b));
  

•           import_uuid(&obj->uuid, resp->uuid);
            obj->uuid_state = UUID_INITIALIZED;
    } else {
            obj->uuid_state = UUID_INITIALIZATION_FAILED;
  

-- 2.28.0