From: Tvrtko Ursulin <tvrtko.ursu...@igalia.com>

It makes it just a tiny bit more obvious what is going on.

Signed-off-by: Tvrtko Ursulin <tvrtko.ursu...@igalia.com>
Reviewed-by: Maíra Canal <mca...@igalia.com>
---
 drivers/gpu/drm/v3d/v3d_submit.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/v3d/v3d_submit.c b/drivers/gpu/drm/v3d/v3d_submit.c
index d1060e60aafa..b282d12571b5 100644
--- a/drivers/gpu/drm/v3d/v3d_submit.c
+++ b/drivers/gpu/drm/v3d/v3d_submit.c
@@ -485,14 +485,14 @@ v3d_get_cpu_timestamp_query_params(struct drm_file 
*file_priv,
        for (i = 0; i < timestamp.count; i++) {
                u32 offset, sync;
 
-               if (copy_from_user(&offset, offsets++, sizeof(offset))) {
+               if (get_user(offset, offsets++)) {
                        err = -EFAULT;
                        goto error;
                }
 
                job->timestamp_query.queries[i].offset = offset;
 
-               if (copy_from_user(&sync, syncs++, sizeof(sync))) {
+               if (get_user(sync, syncs++)) {
                        err = -EFAULT;
                        goto error;
                }
@@ -550,7 +550,7 @@ v3d_get_cpu_reset_timestamp_params(struct drm_file 
*file_priv,
 
                job->timestamp_query.queries[i].offset = reset.offset + 8 * i;
 
-               if (copy_from_user(&sync, syncs++, sizeof(sync))) {
+               if (get_user(sync, syncs++)) {
                        err = -EFAULT;
                        goto error;
                }
@@ -611,14 +611,14 @@ v3d_get_cpu_copy_query_results_params(struct drm_file 
*file_priv,
        for (i = 0; i < copy.count; i++) {
                u32 offset, sync;
 
-               if (copy_from_user(&offset, offsets++, sizeof(offset))) {
+               if (get_user(offset, offsets++)) {
                        err = -EFAULT;
                        goto error;
                }
 
                job->timestamp_query.queries[i].offset = offset;
 
-               if (copy_from_user(&sync, syncs++, sizeof(sync))) {
+               if (get_user(sync, syncs++)) {
                        err = -EFAULT;
                        goto error;
                }
-- 
2.44.0

Reply via email to