Reviewed-by: David Stevens <stevensd@xxxxxxxxxxxx>
On Tue, Oct 13, 2020 at 2:18 AM Andy Shevchenko
<andriy.shevchenko@xxxxxxxxxxxxxxx> 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@xxxxxxxxxxxxxxx>
> ---
> 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
>
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel
