summaryrefslogtreecommitdiffstats
path: root/drivers/gpu
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2013-08-23 12:19:11 +0200
committerThierry Reding <treding@nvidia.com>2013-08-27 10:20:12 +0200
commit745cecc07cee878a5afdda40d13f8b0901a88ebd (patch)
treee8144e2ba4a3f2b0b0453978983e27a5a656f90f /drivers/gpu
parentgpu: host1x: fix an integer overflow check (diff)
downloadlinux-745cecc07cee878a5afdda40d13f8b0901a88ebd.tar.xz
linux-745cecc07cee878a5afdda40d13f8b0901a88ebd.zip
gpu: host1x: returning success instead of -ENOMEM
There is a mistake here so it returns PTR_ERR(NULL) which is success instead of -ENOMEM. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Thierry Reding <treding@nvidia.com>
Diffstat (limited to 'drivers/gpu')
-rw-r--r--drivers/gpu/host1x/job.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/gpu/host1x/job.c b/drivers/gpu/host1x/job.c
index 18a47f95e90c..c4e1050f2252 100644
--- a/drivers/gpu/host1x/job.c
+++ b/drivers/gpu/host1x/job.c
@@ -466,9 +466,8 @@ static inline int copy_gathers(struct host1x_job *job, struct device *dev)
&job->gather_copy,
GFP_KERNEL);
if (!job->gather_copy_mapped) {
- int err = PTR_ERR(job->gather_copy_mapped);
job->gather_copy_mapped = NULL;
- return err;
+ return -ENOMEM;
}
job->gather_copy_size = size;