diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2013-08-23 12:19:11 +0200 |
|---|---|---|
| committer | Thierry Reding <treding@nvidia.com> | 2013-08-27 10:20:12 +0200 |
| commit | 745cecc07cee878a5afdda40d13f8b0901a88ebd (patch) | |
| tree | e8144e2ba4a3f2b0b0453978983e27a5a656f90f /drivers/gpu/host1x | |
| parent | gpu: host1x: fix an integer overflow check (diff) | |
| download | linux-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/host1x')
| -rw-r--r-- | drivers/gpu/host1x/job.c | 3 |
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; |