CHROMIUM: drm/exynos: gem: Remove useless size field
authorDaniel Kurtz <djkurtz@chromium.org>
Mon, 10 Dec 2012 05:53:11 +0000 (13:53 +0800)
committerGerrit <chrome-bot@google.com>
Tue, 11 Dec 2012 01:55:14 +0000 (17:55 -0800)
exynos_drm_gem_obj, drm_gem_object, the dma_buf and the exynos_drm_gem_buf
all store a buffer's size.  This seems a bit excessive.
Remove the exynos_drm_gem_obj copy, since it is written to but never read.

Signed-off-by: Daniel Kurtz <djkurtz@chromium.org>
BUG=chromium-os:37097
TEST=compile test

Change-Id: I5bf057964351d08f40146f0e928e6eb8414f164f
Reviewed-on: https://gerrit.chromium.org/gerrit/39475
Commit-Ready: Daniel Kurtz <djkurtz@chromium.org>
Reviewed-by: Daniel Kurtz <djkurtz@chromium.org>
Tested-by: Daniel Kurtz <djkurtz@chromium.org>
drivers/gpu/drm/exynos/exynos_drm_gem.c
drivers/gpu/drm/exynos/exynos_drm_gem.h

index 9430804..c7ee566 100644 (file)
@@ -318,7 +318,6 @@ struct exynos_drm_gem_obj *exynos_drm_gem_init(struct drm_device *dev,
                return NULL;
        }
 
-       exynos_gem_obj->size = size;
        obj = &exynos_gem_obj->base;
 
        ret = drm_gem_object_init(dev, obj, size);
index 5f38a45..ea69d12 100644 (file)
@@ -63,7 +63,6 @@ struct exynos_drm_gem_buf {
  *     by user request or at framebuffer creation.
  *     continuous memory region allocated by user request
  *     or at framebuffer creation.
- * @size: total memory size to physically non-continuous memory region.
  * @flags: indicate memory type to allocated buffer and cache attribute.
  *
  * P.S. this object would be transfered to user as kms_bo.handle so
@@ -72,7 +71,6 @@ struct exynos_drm_gem_buf {
 struct exynos_drm_gem_obj {
        struct drm_gem_object           base;
        struct exynos_drm_gem_buf       *buffer;
-       unsigned long                   size;
        unsigned int                    flags;
 };