diff options
author | Markus Elfring <elfring@users.sourceforge.net> | 2015-12-19 06:48:31 -0500 |
---|---|---|
committer | Thierry Reding <treding@nvidia.com> | 2016-03-16 08:45:44 -0400 |
commit | 341917fe2b62876599315a537b8e248e31bd1366 (patch) | |
tree | 7b003685900ddb07d8ebcc7315d14ea8e332ef76 | |
parent | 93ec302976c902a77f3163e192d73b0006e3575c (diff) |
gpu: host1x: Use a signed return type for do_relocs()
The return type "unsigned int" was used by the do_relocs() function
despite the fact that it will eventually return a negative error code.
Use a signed integer instead to accomodate for error codes.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Signed-off-by: Thierry Reding <treding@nvidia.com>
-rw-r--r-- | drivers/gpu/host1x/job.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/host1x/job.c b/drivers/gpu/host1x/job.c index 63bd63f3c7df..1919aab88c3f 100644 --- a/drivers/gpu/host1x/job.c +++ b/drivers/gpu/host1x/job.c | |||
@@ -225,7 +225,7 @@ unpin: | |||
225 | return 0; | 225 | return 0; |
226 | } | 226 | } |
227 | 227 | ||
228 | static unsigned int do_relocs(struct host1x_job *job, struct host1x_bo *cmdbuf) | 228 | static int do_relocs(struct host1x_job *job, struct host1x_bo *cmdbuf) |
229 | { | 229 | { |
230 | int i = 0; | 230 | int i = 0; |
231 | u32 last_page = ~0; | 231 | u32 last_page = ~0; |