diff options
author | Thomas Hellstrom <thellstrom@vmware.com> | 2015-08-04 09:34:14 -0400 |
---|---|---|
committer | Thomas Hellstrom <thellstrom@vmware.com> | 2015-08-05 08:01:11 -0400 |
commit | 2e3cc8cff629c9697ac27c95a89dda7c7785b6b4 (patch) | |
tree | 2ff99d3282e0e81f641ff334fc8d06a29f5544c3 | |
parent | b9eb1a6174e58eb8beea664ffc20d152230d8004 (diff) |
drm/vmwgfx: Fix compiler warning with 32-bit dma_addr_t
When the size of dma_addr_t was 32 bits, the compiler warned
about the size of the 32 bit shift being larger than the size
of the data type.
Reported by Intel's kbuild robot.
Signed-off-by: Thomas Hellstrom <thellstrom@vmware.com>
Reviewed-by: Sinclair Yeh <syeh@vmware.com>
Reviewed-by: Brian Paul <brianp@vmware.com>
-rw-r--r-- | drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf.c b/drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf.c index 32ec52eaedd8..afc6d1df47d7 100644 --- a/drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf.c +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf.c | |||
@@ -293,8 +293,12 @@ static int vmw_cmdbuf_header_submit(struct vmw_cmdbuf_header *header) | |||
293 | struct vmw_cmdbuf_man *man = header->man; | 293 | struct vmw_cmdbuf_man *man = header->man; |
294 | u32 val; | 294 | u32 val; |
295 | 295 | ||
296 | val = (header->handle >> 32); | 296 | if (sizeof(header->handle) > 4) |
297 | val = (header->handle >> 32); | ||
298 | else | ||
299 | val = 0; | ||
297 | vmw_write(man->dev_priv, SVGA_REG_COMMAND_HIGH, val); | 300 | vmw_write(man->dev_priv, SVGA_REG_COMMAND_HIGH, val); |
301 | |||
298 | val = (header->handle & 0xFFFFFFFFULL); | 302 | val = (header->handle & 0xFFFFFFFFULL); |
299 | val |= header->cb_context & SVGA_CB_CONTEXT_MASK; | 303 | val |= header->cb_context & SVGA_CB_CONTEXT_MASK; |
300 | vmw_write(man->dev_priv, SVGA_REG_COMMAND_LOW, val); | 304 | vmw_write(man->dev_priv, SVGA_REG_COMMAND_LOW, val); |