Re: [PATCH] fix saa7146 compilation on 2.6.8-rc1

From: Michael Hunold
Date: Mon Jul 12 2004 - 14:48:04 EST


Hello Linus, Andrew,

On 07/12/04 10:25, colin@xxxxxxxxxx wrote:
this patch fixes a compilation error on 2.6.8-rc1. Here's the error:
drivers/media/common/saa7146_video.c:3: conflicting types for `memory'
include/asm-m68k/setup.h:365: previous declaration of `memory'
make[3]: *** [drivers/media/common/saa7146_video.o] Error 1

Colin's patch is fine, an updated version of the patch with an additional signed-off line is attached.

Please apply. Thanks!

CU
Michael.
Signed-off-by: Colin Leroy <colin@xxxxxxxxxx>
Signed-off-by: Michael Hunold <hunold@xxxxxxxxxxx>

--- a/drivers/media/common/saa7146_video.c 2004-07-12 10:15:51.833352344 +0200
+++ b/drivers/media/common/saa7146_video.c 2004-07-12 10:16:21.209886432 +0200
@@ -1,9 +1,9 @@
#include <media/saa7146_vv.h>

-static int memory = 32;
+static int max_memory = 32;

-MODULE_PARM(memory,"i");
-MODULE_PARM_DESC(memory, "maximum memory usage for capture buffers (default: 32Mb)");
+MODULE_PARM(max_memory,"i");
+MODULE_PARM_DESC(max_memory, "maximum memory usage for capture buffers (default: 32Mb)");

#define IS_CAPTURE_ACTIVE(fh) \
(((vv->video_status & STATUS_CAPTURE) != 0) && (vv->video_fh == fh))
@@ -1331,9 +1331,9 @@

*size = fh->video_fmt.sizeimage;

- /* check if we exceed the "memory" parameter */
- if( (*count * *size) > (memory*1048576) ) {
- *count = (memory*1048576) / *size;
+ /* check if we exceed the "max_memory" parameter */
+ if( (*count * *size) > (max_memory*1048576) ) {
+ *count = (max_memory*1048576) / *size;
}

DEB_CAP(("%d buffers, %d bytes each.\n",*count,*size));