diff mbox

[v3,01/32] drm/exynos: Remove useless slab.h include

Message ID 1383063198-10526-2-git-send-email-seanpaul@chromium.org (mailing list archive)
State New, archived
Headers show

Commit Message

Sean Paul Oct. 29, 2013, 4:12 p.m. UTC
From: Stéphane Marchesin <marcheu@chromium.org>

Signed-off-by: Stéphane Marchesin <marcheu@chromium.org>
Signed-off-by: Sean Paul <seanpaul@chromium.org>
---

Changes in v2: None
Changes in v3: None

 drivers/video/exynos/exynos_dp_core.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Inki Dae Oct. 31, 2013, 10:24 a.m. UTC | #1
CCing Jingoo,

> -----Original Message-----
> From: Sean Paul [mailto:seanpaul@chromium.org]
> Sent: Wednesday, October 30, 2013 1:13 AM
> To: dri-devel@lists.freedesktop.org; inki.dae@samsung.com
> Cc: airlied@linux.ie; tomasz.figa@gmail.com; marcheu@chromium.org; Sean
> Paul
> Subject: [PATCH v3 01/32] drm/exynos: Remove useless slab.h include
> 
> From: Stéphane Marchesin <marcheu@chromium.org>
> 
> Signed-off-by: Stéphane Marchesin <marcheu@chromium.org>
> Signed-off-by: Sean Paul <seanpaul@chromium.org>
> ---
> 
> Changes in v2: None
> Changes in v3: None
> 
>  drivers/video/exynos/exynos_dp_core.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/video/exynos/exynos_dp_core.c
> b/drivers/video/exynos/exynos_dp_core.c
> index 12bbede..089ae22 100644
> --- a/drivers/video/exynos/exynos_dp_core.c
> +++ b/drivers/video/exynos/exynos_dp_core.c
> @@ -12,7 +12,6 @@
> 
>  #include <linux/module.h>
>  #include <linux/platform_device.h>
> -#include <linux/slab.h>
>  #include <linux/err.h>
>  #include <linux/clk.h>
>  #include <linux/io.h>
> --
> 1.8.4
Tomasz Figa Oct. 31, 2013, 11:32 p.m. UTC | #2
Hi Sean, Stéphane,

On Tuesday 29 of October 2013 12:12:47 Sean Paul wrote:
> From: Stéphane Marchesin <marcheu@chromium.org>
> 
> Signed-off-by: Stéphane Marchesin <marcheu@chromium.org>
> Signed-off-by: Sean Paul <seanpaul@chromium.org>
> ---
> 
> Changes in v2: None
> Changes in v3: None
> 
>  drivers/video/exynos/exynos_dp_core.c | 1 -
>  1 file changed, 1 deletion(-)

I'm not sure if this patch should really be a part of this series.

Furthermore, is this header inclusion really useless? I believe it is the 
place where kmalloc() and friends are declared.

Best regards,
Tomasz
diff mbox

Patch

diff --git a/drivers/video/exynos/exynos_dp_core.c b/drivers/video/exynos/exynos_dp_core.c
index 12bbede..089ae22 100644
--- a/drivers/video/exynos/exynos_dp_core.c
+++ b/drivers/video/exynos/exynos_dp_core.c
@@ -12,7 +12,6 @@ 
 
 #include <linux/module.h>
 #include <linux/platform_device.h>
-#include <linux/slab.h>
 #include <linux/err.h>
 #include <linux/clk.h>
 #include <linux/io.h>