[PATCH] drm/exynos: use %pad for dma_addr_t

[Daniel Kurtz]

On Tue, Apr 22, 2014 at 1:45 PM, Jingoo Han  wrote:

> Use %pad for dma_addr_t, because a dma_addr_t type can vary
> based on build options. So, it prevents possible build warnings
> in printks.
>
> Signed-off-by: Jingoo Han 
>

Yay!

Reviewed-by: Daniel Kurtz 


> ---
>  drivers/gpu/drm/exynos/exynos_drm_dmabuf.c |2 +-
>  drivers/gpu/drm/exynos/exynos_drm_vidi.c   |2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/exynos/exynos_drm_dmabuf.c
> b/drivers/gpu/drm/exynos/exynos_drm_dmabuf.c
> index 59827cc..9cf71fa 100644
> --- a/drivers/gpu/drm/exynos/exynos_drm_dmabuf.c
> +++ b/drivers/gpu/drm/exynos/exynos_drm_dmabuf.c
> @@ -263,7 +263,7 @@ struct drm_gem_object
> *exynos_dmabuf_prime_import(struct drm_device *drm_dev,
> buffer->sgt = sgt;
> exynos_gem_obj->base.import_attach = attach;
>
> -   DRM_DEBUG_PRIME("dma_addr = 0x%x, size = 0x%lx\n",
> buffer->dma_addr,
> +   DRM_DEBUG_PRIME("dma_addr = %pad, size = 0x%lx\n",
> &buffer->dma_addr,
>
> buffer->size);
>
> return &exynos_gem_obj->base;
> diff --git a/drivers/gpu/drm/exynos/exynos_drm_vidi.c
> b/drivers/gpu/drm/exynos/exynos_drm_vidi.c
> index 3319289..e4689ef 100644
> --- a/drivers/gpu/drm/exynos/exynos_drm_vidi.c
> +++ b/drivers/gpu/drm/exynos/exynos_drm_vidi.c
> @@ -221,7 +221,7 @@ static void vidi_win_commit(struct exynos_drm_manager
> *mgr, int zpos)
>
> win_data->enabled = true;
>
> -   DRM_DEBUG_KMS("dma_addr = 0x%x\n", win_data->dma_addr);
> +   DRM_DEBUG_KMS("dma_addr = %pad\n", &win_data->dma_addr);
>
> if (ctx->vblank_on)
> schedule_work(&ctx->work);
> --
> 1.7.10.4
>
>
> ___
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
>
-- next part --
An HTML attachment was scrubbed...
URL: 

[PATCH] drm/exynos: use %pad for dma_addr_t

[Jingoo Han]

Use %pad for dma_addr_t, because a dma_addr_t type can vary
based on build options. So, it prevents possible build warnings
in printks.

Signed-off-by: Jingoo Han 
---
 drivers/gpu/drm/exynos/exynos_drm_dmabuf.c |2 +-
 drivers/gpu/drm/exynos/exynos_drm_vidi.c   |2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/exynos/exynos_drm_dmabuf.c 
b/drivers/gpu/drm/exynos/exynos_drm_dmabuf.c
index 59827cc..9cf71fa 100644
--- a/drivers/gpu/drm/exynos/exynos_drm_dmabuf.c
+++ b/drivers/gpu/drm/exynos/exynos_drm_dmabuf.c
@@ -263,7 +263,7 @@ struct drm_gem_object *exynos_dmabuf_prime_import(struct 
drm_device *drm_dev,
buffer->sgt = sgt;
exynos_gem_obj->base.import_attach = attach;

-   DRM_DEBUG_PRIME("dma_addr = 0x%x, size = 0x%lx\n", buffer->dma_addr,
+   DRM_DEBUG_PRIME("dma_addr = %pad, size = 0x%lx\n", &buffer->dma_addr,
buffer->size);

return &exynos_gem_obj->base;
diff --git a/drivers/gpu/drm/exynos/exynos_drm_vidi.c 
b/drivers/gpu/drm/exynos/exynos_drm_vidi.c
index 3319289..e4689ef 100644
--- a/drivers/gpu/drm/exynos/exynos_drm_vidi.c
+++ b/drivers/gpu/drm/exynos/exynos_drm_vidi.c
@@ -221,7 +221,7 @@ static void vidi_win_commit(struct exynos_drm_manager *mgr, 
int zpos)

win_data->enabled = true;

-   DRM_DEBUG_KMS("dma_addr = 0x%x\n", win_data->dma_addr);
+   DRM_DEBUG_KMS("dma_addr = %pad\n", &win_data->dma_addr);

if (ctx->vblank_on)
schedule_work(&ctx->work);
-- 
1.7.10.4