diff options
| author | Thierry Reding <treding@nvidia.com> | 2016-06-23 11:21:51 +0200 |
|---|---|---|
| committer | Thierry Reding <treding@nvidia.com> | 2016-06-23 11:59:27 +0200 |
| commit | ebb2475c4731d9c9940db6ff6514fab723e1f489 (patch) | |
| tree | cd82dc96c210ef0f2b83aec231851ae00b6c3515 /drivers/gpu/host1x | |
| 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>
Diffstat (limited to 'drivers/gpu/host1x')
| -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: */ static void host1x_pushbuffer_push(struct push_buffer *pb, u32 op1, u32 op2) { - u32 pos = pb->pos; - u32 *p = (u32 *)((void *)pb->mapped + pos); - WARN_ON(pos == pb->fence); + u32 *p = (u32 *)((void *)pb->mapped + pb->pos); + + WARN_ON(pb->pos == pb->fence); *(p++) = op1; *(p++) = op2; - pb->pos = (pos + 8) & (pb->size_bytes - 1); + pb->pos = (pb->pos + 8) & (pb->size_bytes - 1); } /* |