diff options
author | Thierry Reding <treding@nvidia.com> | 2016-06-23 05:21:51 -0400 |
---|---|---|
committer | Thierry Reding <treding@nvidia.com> | 2016-06-23 05:59:27 -0400 |
commit | ebb2475c4731d9c9940db6ff6514fab723e1f489 (patch) | |
tree | cd82dc96c210ef0f2b83aec231851ae00b6c3515 | |
parent | e18e33afa4011b0670e5040eda1b8a2d0b995e4e (diff) |
gpu: host1x: cdma: Drop unnecessary local variable
The local 'pos' variable doesn't serve any purpose other than being a
shortcut for pb->pos, but the result doesn't remove much, so simply drop
the local variable.
Signed-off-by: Thierry Reding <treding@nvidia.com>
-rw-r--r-- | drivers/gpu/host1x/cdma.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/gpu/host1x/cdma.c b/drivers/gpu/host1x/cdma.c index a18db4d5347c..fe1b216b8d17 100644 --- a/drivers/gpu/host1x/cdma.c +++ b/drivers/gpu/host1x/cdma.c | |||
@@ -96,12 +96,12 @@ fail: | |||
96 | */ | 96 | */ |
97 | static void host1x_pushbuffer_push(struct push_buffer *pb, u32 op1, u32 op2) | 97 | static void host1x_pushbuffer_push(struct push_buffer *pb, u32 op1, u32 op2) |
98 | { | 98 | { |
99 | u32 pos = pb->pos; | 99 | u32 *p = (u32 *)((void *)pb->mapped + pb->pos); |
100 | u32 *p = (u32 *)((void *)pb->mapped + pos); | 100 | |
101 | WARN_ON(pos == pb->fence); | 101 | WARN_ON(pb->pos == pb->fence); |
102 | *(p++) = op1; | 102 | *(p++) = op1; |
103 | *(p++) = op2; | 103 | *(p++) = op2; |
104 | pb->pos = (pos + 8) & (pb->size_bytes - 1); | 104 | pb->pos = (pb->pos + 8) & (pb->size_bytes - 1); |
105 | } | 105 | } |
106 | 106 | ||
107 | /* | 107 | /* |