[PATCH-Trivial 1/2] [media] s5p-fimc: Replace asm/* headers with linux/*

[PATCH-Trivial 1/2] [media] s5p-fimc: Replace asm/* headers with linux/*

[Sachin Kamat]

Silences the following warning:
WARNING: Use #include <linux/sizes.h> instead of <asm/sizes.h>

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 drivers/media/platform/s5p-fimc/fimc-core.h |    2 +-
 drivers/media/platform/s5p-fimc/fimc-lite.h |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/media/platform/s5p-fimc/fimc-core.h b/drivers/media/platform/s5p-fimc/fimc-core.h
index 808ccc6..e787f65 100644
--- a/drivers/media/platform/s5p-fimc/fimc-core.h
+++ b/drivers/media/platform/s5p-fimc/fimc-core.h
@@ -17,7 +17,7 @@
 #include <linux/types.h>
 #include <linux/videodev2.h>
 #include <linux/io.h>
-#include <asm/sizes.h>
+#include <linux/sizes.h>

 #include <media/media-entity.h>
 #include <media/videobuf2-core.h>
diff --git a/drivers/media/platform/s5p-fimc/fimc-lite.h b/drivers/media/platform/s5p-fimc/fimc-lite.h
index 44424ee..8dc3e9b 100644
--- a/drivers/media/platform/s5p-fimc/fimc-lite.h
+++ b/drivers/media/platform/s5p-fimc/fimc-lite.h
@@ -9,7 +9,7 @@
 #ifndef FIMC_LITE_H_
 #define FIMC_LITE_H_

-#include <asm/sizes.h>
+#include <linux/sizes.h>
 #include <linux/io.h>
 #include <linux/irqreturn.h>
 #include <linux/platform_device.h>
--
1.7.4.1

[PATCH-Trivial 2/2] [media] s5p-fimc: Add missing braces around sizeof

[Sachin Kamat]

Silences the following warning:
WARNING: sizeof *ctx should be sizeof(*ctx)

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 drivers/media/platform/s5p-fimc/fimc-capture.c |    2 +-
 drivers/media/platform/s5p-fimc/fimc-m2m.c     |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/media/platform/s5p-fimc/fimc-capture.c b/drivers/media/platform/s5p-fimc/fimc-capture.c
index 8e413dd..5283957 100644
--- a/drivers/media/platform/s5p-fimc/fimc-capture.c
+++ b/drivers/media/platform/s5p-fimc/fimc-capture.c
@@ -1593,7 +1593,7 @@ static int fimc_register_capture_device(struct fimc_dev *fimc,
 	struct vb2_queue *q;
 	int ret = -ENOMEM;

-	ctx = kzalloc(sizeof *ctx, GFP_KERNEL);
+	ctx = kzalloc(sizeof(*ctx), GFP_KERNEL);
 	if (!ctx)
 		return -ENOMEM;

diff --git a/drivers/media/platform/s5p-fimc/fimc-m2m.c b/drivers/media/platform/s5p-fimc/fimc-m2m.c
index c587011..1b1e564 100644
--- a/drivers/media/platform/s5p-fimc/fimc-m2m.c
+++ b/drivers/media/platform/s5p-fimc/fimc-m2m.c
@@ -661,7 +661,7 @@ static int fimc_m2m_open(struct file *file)
 	if (fimc->vid_cap.refcnt > 0)
 		goto unlock;

-	ctx = kzalloc(sizeof *ctx, GFP_KERNEL);
+	ctx = kzalloc(sizeof(*ctx), GFP_KERNEL);
 	if (!ctx) {
 		ret = -ENOMEM;
 		goto unlock;
--
1.7.4.1

Re: [PATCH-Trivial 1/2] [media] s5p-fimc: Replace asm/* headers with linux/*

[Sylwester Nawrocki]

Hi Sachin,

On 08/17/2012 08:28 AM, Sachin Kamat wrote:
> Silences the following warning:
> WARNING: Use #include <linux/sizes.h> instead of <asm/sizes.h>
> 
> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>

Added both to my tree.

Thanks,
Sylwester

Re: [PATCH-Trivial 1/2] [media] s5p-fimc: Replace asm/* headers with linux/*

[Sachin Kamat]

On 17 August 2012 13:51, Sylwester Nawrocki <s.nawrocki@samsung.com> wrote:
> Hi Sachin,
>
> On 08/17/2012 08:28 AM, Sachin Kamat wrote:
>> Silences the following warning:
>> WARNING: Use #include <linux/sizes.h> instead of <asm/sizes.h>
>>
>> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
>
> Added both to my tree.

Thanks Sylwester.

>
> Thanks,
> Sylwester



-- 
With warm regards,
Sachin