Re: [v4l-dvb-maintainer] [PATCH 1/2] ivtv: Some general fixes

2007-12-03 Thread Mauro Carvalho Chehab 

Em Dom, 2007-12-02 às 20:49 +0100, Hans Verkuil escreveu:
> On Sunday 02 December 2007 18:46, Richard Knutsson wrote:
> > Fix "warning: Using plain integer as NULL pointer".
> 
> Signed-off-by: Hans Verkuil <[EMAIL PROTECTED]>
> 
> > Remove a gcc-2.95 requirement.
> 
> NACK! The main v4l-dvb repository that contains this driver can also be 
> compiled on older systems. I'd like to keep the option that the driver 
> can be compiled with an older gcc version, unless Mauro thinks 
> otherwise. Other than the removal of one comment and one space there 
> are no other benefits of this change, so I'd prefer to keep it.

>From my POV, I don't see any reason to remove the required space to keep
gcc-2.95 compatibility. 

The required extra whitespace is a small CodingStyle violation, but may
help to use the same driver code for systems that still use gcc-2.95.
So, it seems valid to keep this whitespace. Besides that, it is properly
commented.
 
Cheers,
Mauro

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [v4l-dvb-maintainer] [PATCH 1/2] ivtv: Some general fixes

2007-12-03 Thread Mauro Carvalho Chehab 

Em Dom, 2007-12-02 às 20:49 +0100, Hans Verkuil escreveu:
 On Sunday 02 December 2007 18:46, Richard Knutsson wrote:
  Fix warning: Using plain integer as NULL pointer.
 
 Signed-off-by: Hans Verkuil [EMAIL PROTECTED]
 
  Remove a gcc-2.95 requirement.
 
 NACK! The main v4l-dvb repository that contains this driver can also be 
 compiled on older systems. I'd like to keep the option that the driver 
 can be compiled with an older gcc version, unless Mauro thinks 
 otherwise. Other than the removal of one comment and one space there 
 are no other benefits of this change, so I'd prefer to keep it.

From my POV, I don't see any reason to remove the required space to keep
gcc-2.95 compatibility. 

The required extra whitespace is a small CodingStyle violation, but may
help to use the same driver code for systems that still use gcc-2.95.
So, it seems valid to keep this whitespace. Besides that, it is properly
commented.
 
Cheers,
Mauro

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [v4l-dvb-maintainer] [PATCH 1/2] ivtv: Some general fixes

2007-12-02 Thread Richard Knutsson 

Hans Verkuil wrote:

On Sunday 02 December 2007 18:46, Richard Knutsson wrote:
  

Fix "warning: Using plain integer as NULL pointer".



Signed-off-by: Hans Verkuil <[EMAIL PROTECTED]>

  

Remove a gcc-2.95 requirement.



NACK! The main v4l-dvb repository that contains this driver can also be 
compiled on older systems. I'd like to keep the option that the driver 
can be compiled with an older gcc version, unless Mauro thinks 
otherwise. Other than the removal of one comment and one space there 
are no other benefits of this change, so I'd prefer to keep it.


  
Oh alright, thought it was just a leftover since the support for the 
2.95 ended.
Maybe add something to the text to let others know the status? (Since I 
found it by doing a 'grep' for "gcc 2.95"'s...)

Convert 'x < y ? x : y' to use min() instead.



Signed-off-by: Hans Verkuil <[EMAIL PROTECTED]>

Thanks,

Hans

  

Thanks for the fast turnaround
Richard

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [v4l-dvb-maintainer] [PATCH 1/2] ivtv: Some general fixes

2007-12-02 Thread Hans Verkuil 
On Sunday 02 December 2007 18:46, Richard Knutsson wrote:
> Fix "warning: Using plain integer as NULL pointer".

Signed-off-by: Hans Verkuil <[EMAIL PROTECTED]>

> Remove a gcc-2.95 requirement.

NACK! The main v4l-dvb repository that contains this driver can also be 
compiled on older systems. I'd like to keep the option that the driver 
can be compiled with an older gcc version, unless Mauro thinks 
otherwise. Other than the removal of one comment and one space there 
are no other benefits of this change, so I'd prefer to keep it.

> Convert 'x < y ? x : y' to use min() instead.

Signed-off-by: Hans Verkuil <[EMAIL PROTECTED]>

Thanks,

Hans


>
> Signed-off-by: Richard Knutsson <[EMAIL PROTECTED]>
> ---
> Compile-tested on i386 with "allyesconfig" and "allmodconfig".
>
>  ivtv-driver.c  |2 +-
>  ivtv-driver.h  |4 +---
>  ivtv-ioctl.c   |2 +-
>  ivtv-irq.c |4 ++--
>  ivtv-streams.c |4 ++--
>  ivtvfb.c   |2 +-
>  6 files changed, 8 insertions(+), 10 deletions(-)
>
>
> diff --git a/drivers/media/video/ivtv/ivtv-driver.c
> b/drivers/media/video/ivtv/ivtv-driver.c index 6d2dd87..96f340c
> 100644
> --- a/drivers/media/video/ivtv/ivtv-driver.c
> +++ b/drivers/media/video/ivtv/ivtv-driver.c
> @@ -979,7 +979,7 @@ static int __devinit ivtv_probe(struct pci_dev
> *dev, }
>
>   itv = kzalloc(sizeof(struct ivtv), GFP_ATOMIC);
> - if (itv == 0) {
> + if (itv == NULL) {
>   spin_unlock(_cards_lock);
>   return -ENOMEM;
>   }
> diff --git a/drivers/media/video/ivtv/ivtv-driver.h
> b/drivers/media/video/ivtv/ivtv-driver.h index 49ce14d..5c4956c
> 100644
> --- a/drivers/media/video/ivtv/ivtv-driver.h
> +++ b/drivers/media/video/ivtv/ivtv-driver.h
> @@ -132,12 +132,10 @@ extern int ivtv_debug;
>  /* Flag to turn on high volume debugging */
>  #define IVTV_DBGFLG_HIGHVOL (1 << 10)
>
> -/* NOTE: extra space before comma in 'itv->num , ## args' is
> required for -   gcc-2.95, otherwise it won't compile. */
>  #define IVTV_DEBUG(x, type, fmt, args...) \
>   do { \
>   if ((x) & ivtv_debug) \
> - printk(KERN_INFO "ivtv%d " type ": " fmt, itv->num , ## 
> args); \
> + printk(KERN_INFO "ivtv%d " type ": " fmt, itv->num, ## 
> args); \
>   } while (0)
>  #define IVTV_DEBUG_WARN(fmt, args...)  IVTV_DEBUG(IVTV_DBGFLG_WARN, 
> "warn",  fmt , ## args) #define IVTV_DEBUG_INFO(fmt, args...) 
> IVTV_DEBUG(IVTV_DBGFLG_INFO,  "info",  fmt , ## args) diff --git
> a/drivers/media/video/ivtv/ivtv-ioctl.c
> b/drivers/media/video/ivtv/ivtv-ioctl.c index fd6826f..24270de 100644
> --- a/drivers/media/video/ivtv/ivtv-ioctl.c
> +++ b/drivers/media/video/ivtv/ivtv-ioctl.c
> @@ -688,7 +688,7 @@ static int ivtv_debug_ioctls(struct file *filp,
> unsigned int cmd, void *arg) ivtv_reset_ir_gpio(itv);
>   }
>   if (val & 0x02) {
> - itv->video_dec_func(itv, cmd, 0);
> + itv->video_dec_func(itv, cmd, NULL);
>   }
>   break;
>   }
> diff --git a/drivers/media/video/ivtv/ivtv-irq.c
> b/drivers/media/video/ivtv/ivtv-irq.c index fd1688e..1384615 100644
> --- a/drivers/media/video/ivtv/ivtv-irq.c
> +++ b/drivers/media/video/ivtv/ivtv-irq.c
> @@ -204,7 +204,7 @@ static int stream_enc_dma_append(struct
> ivtv_stream *s, u32 data[CX2341X_MBOX_MA s->sg_pending[idx].dst =
> buf->dma_handle;
>   s->sg_pending[idx].src = offset;
>   s->sg_pending[idx].size = s->buf_size;
> - buf->bytesused = (size < s->buf_size) ? size : s->buf_size;
> + buf->bytesused = min(size, s->buf_size);
>   buf->dma_xfer_cnt = s->dma_xfer_cnt;
>
>   s->q_predma.bytesused += buf->bytesused;
> @@ -705,7 +705,7 @@ static void ivtv_irq_dec_data_req(struct ivtv
> *itv) s = >streams[IVTV_DEC_STREAM_TYPE_YUV];
>   }
>   else {
> - itv->dma_data_req_size = data[2] >= 0x1 ? 0x1 : data[2];
> + itv->dma_data_req_size = min_t(u32, data[2], 0x1);
>   itv->dma_data_req_offset = data[1];
>   s = >streams[IVTV_DEC_STREAM_TYPE_MPG];
>   }
> diff --git a/drivers/media/video/ivtv/ivtv-streams.c
> b/drivers/media/video/ivtv/ivtv-streams.c index aa03e61..0e9e7d0
> 100644
> --- a/drivers/media/video/ivtv/ivtv-streams.c
> +++ b/drivers/media/video/ivtv/ivtv-streams.c
> @@ -572,10 +572,10 @@ int ivtv_start_v4l2_encode_stream(struct
> ivtv_stream *s) clear_bit(IVTV_F_I_EOS, >i_flags);
>
>   /* Initialize Digitizer for Capture */
> - itv->video_dec_func(itv, VIDIOC_STREAMOFF, 0);
> + itv->video_dec_func(itv, VIDIOC_STREAMOFF, NULL);
>   ivtv_msleep_timeout(300, 1);
>   ivtv_vapi(itv, CX2341X_ENC_INITIALIZE_INPUT, 0);
> - itv->video_dec_func(itv, VIDIOC_STREAMON, 0);
> + itv->video_dec_func(itv, VIDIOC_STREAMON, NULL);
>   }
>
>   /* begin_capture */
> diff

[PATCH 1/2] ivtv: Some general fixes

2007-12-02 Thread Richard Knutsson 
Fix "warning: Using plain integer as NULL pointer".
Remove a gcc-2.95 requirement.
Convert 'x < y ? x : y' to use min() instead.

Signed-off-by: Richard Knutsson <[EMAIL PROTECTED]>
---
Compile-tested on i386 with "allyesconfig" and "allmodconfig".

 ivtv-driver.c  |2 +-
 ivtv-driver.h  |4 +---
 ivtv-ioctl.c   |2 +-
 ivtv-irq.c |4 ++--
 ivtv-streams.c |4 ++--
 ivtvfb.c   |2 +-
 6 files changed, 8 insertions(+), 10 deletions(-)


diff --git a/drivers/media/video/ivtv/ivtv-driver.c 
b/drivers/media/video/ivtv/ivtv-driver.c
index 6d2dd87..96f340c 100644
--- a/drivers/media/video/ivtv/ivtv-driver.c
+++ b/drivers/media/video/ivtv/ivtv-driver.c
@@ -979,7 +979,7 @@ static int __devinit ivtv_probe(struct pci_dev *dev,
}
 
itv = kzalloc(sizeof(struct ivtv), GFP_ATOMIC);
-   if (itv == 0) {
+   if (itv == NULL) {
spin_unlock(_cards_lock);
return -ENOMEM;
}
diff --git a/drivers/media/video/ivtv/ivtv-driver.h 
b/drivers/media/video/ivtv/ivtv-driver.h
index 49ce14d..5c4956c 100644
--- a/drivers/media/video/ivtv/ivtv-driver.h
+++ b/drivers/media/video/ivtv/ivtv-driver.h
@@ -132,12 +132,10 @@ extern int ivtv_debug;
 /* Flag to turn on high volume debugging */
 #define IVTV_DBGFLG_HIGHVOL (1 << 10)
 
-/* NOTE: extra space before comma in 'itv->num , ## args' is required for
-   gcc-2.95, otherwise it won't compile. */
 #define IVTV_DEBUG(x, type, fmt, args...) \
do { \
if ((x) & ivtv_debug) \
-   printk(KERN_INFO "ivtv%d " type ": " fmt, itv->num , ## 
args); \
+   printk(KERN_INFO "ivtv%d " type ": " fmt, itv->num, ## 
args); \
} while (0)
 #define IVTV_DEBUG_WARN(fmt, args...)  IVTV_DEBUG(IVTV_DBGFLG_WARN,  "warn",  
fmt , ## args)
 #define IVTV_DEBUG_INFO(fmt, args...)  IVTV_DEBUG(IVTV_DBGFLG_INFO,  "info",  
fmt , ## args)
diff --git a/drivers/media/video/ivtv/ivtv-ioctl.c 
b/drivers/media/video/ivtv/ivtv-ioctl.c
index fd6826f..24270de 100644
--- a/drivers/media/video/ivtv/ivtv-ioctl.c
+++ b/drivers/media/video/ivtv/ivtv-ioctl.c
@@ -688,7 +688,7 @@ static int ivtv_debug_ioctls(struct file *filp, unsigned 
int cmd, void *arg)
ivtv_reset_ir_gpio(itv);
}
if (val & 0x02) {
-   itv->video_dec_func(itv, cmd, 0);
+   itv->video_dec_func(itv, cmd, NULL);
}
break;
}
diff --git a/drivers/media/video/ivtv/ivtv-irq.c 
b/drivers/media/video/ivtv/ivtv-irq.c
index fd1688e..1384615 100644
--- a/drivers/media/video/ivtv/ivtv-irq.c
+++ b/drivers/media/video/ivtv/ivtv-irq.c
@@ -204,7 +204,7 @@ static int stream_enc_dma_append(struct ivtv_stream *s, u32 
data[CX2341X_MBOX_MA
s->sg_pending[idx].dst = buf->dma_handle;
s->sg_pending[idx].src = offset;
s->sg_pending[idx].size = s->buf_size;
-   buf->bytesused = (size < s->buf_size) ? size : s->buf_size;
+   buf->bytesused = min(size, s->buf_size);
buf->dma_xfer_cnt = s->dma_xfer_cnt;
 
s->q_predma.bytesused += buf->bytesused;
@@ -705,7 +705,7 @@ static void ivtv_irq_dec_data_req(struct ivtv *itv)
s = >streams[IVTV_DEC_STREAM_TYPE_YUV];
}
else {
-   itv->dma_data_req_size = data[2] >= 0x1 ? 0x1 : data[2];
+   itv->dma_data_req_size = min_t(u32, data[2], 0x1);
itv->dma_data_req_offset = data[1];
s = >streams[IVTV_DEC_STREAM_TYPE_MPG];
}
diff --git a/drivers/media/video/ivtv/ivtv-streams.c 
b/drivers/media/video/ivtv/ivtv-streams.c
index aa03e61..0e9e7d0 100644
--- a/drivers/media/video/ivtv/ivtv-streams.c
+++ b/drivers/media/video/ivtv/ivtv-streams.c
@@ -572,10 +572,10 @@ int ivtv_start_v4l2_encode_stream(struct ivtv_stream *s)
clear_bit(IVTV_F_I_EOS, >i_flags);
 
/* Initialize Digitizer for Capture */
-   itv->video_dec_func(itv, VIDIOC_STREAMOFF, 0);
+   itv->video_dec_func(itv, VIDIOC_STREAMOFF, NULL);
ivtv_msleep_timeout(300, 1);
ivtv_vapi(itv, CX2341X_ENC_INITIALIZE_INPUT, 0);
-   itv->video_dec_func(itv, VIDIOC_STREAMON, 0);
+   itv->video_dec_func(itv, VIDIOC_STREAMON, NULL);
}
 
/* begin_capture */
diff --git a/drivers/media/video/ivtv/ivtvfb.c 
b/drivers/media/video/ivtv/ivtvfb.c
index 52ffd15..f73ce98 100644
--- a/drivers/media/video/ivtv/ivtvfb.c
+++ b/drivers/media/video/ivtv/ivtvfb.c
@@ -1053,7 +1053,7 @@ static int ivtvfb_init_card(struct ivtv *itv)
}
 
itv->osd_info = kzalloc(sizeof(struct osd_info), GFP_ATOMIC);
-   if (itv->osd_info == 0) {
+   if (itv->osd_info == NULL) {
IVTVFB_ERR("Failed to allocate memory for osd_info\n");
return -ENOMEM;
}
--
To unsubscribe from this list:

[PATCH 1/2] ivtv: Some general fixes

2007-12-02 Thread Richard Knutsson 
Fix warning: Using plain integer as NULL pointer.
Remove a gcc-2.95 requirement.
Convert 'x  y ? x : y' to use min() instead.

Signed-off-by: Richard Knutsson [EMAIL PROTECTED]
---
Compile-tested on i386 with allyesconfig and allmodconfig.

 ivtv-driver.c  |2 +-
 ivtv-driver.h  |4 +---
 ivtv-ioctl.c   |2 +-
 ivtv-irq.c |4 ++--
 ivtv-streams.c |4 ++--
 ivtvfb.c   |2 +-
 6 files changed, 8 insertions(+), 10 deletions(-)


diff --git a/drivers/media/video/ivtv/ivtv-driver.c 
b/drivers/media/video/ivtv/ivtv-driver.c
index 6d2dd87..96f340c 100644
--- a/drivers/media/video/ivtv/ivtv-driver.c
+++ b/drivers/media/video/ivtv/ivtv-driver.c
@@ -979,7 +979,7 @@ static int __devinit ivtv_probe(struct pci_dev *dev,
}
 
itv = kzalloc(sizeof(struct ivtv), GFP_ATOMIC);
-   if (itv == 0) {
+   if (itv == NULL) {
spin_unlock(ivtv_cards_lock);
return -ENOMEM;
}
diff --git a/drivers/media/video/ivtv/ivtv-driver.h 
b/drivers/media/video/ivtv/ivtv-driver.h
index 49ce14d..5c4956c 100644
--- a/drivers/media/video/ivtv/ivtv-driver.h
+++ b/drivers/media/video/ivtv/ivtv-driver.h
@@ -132,12 +132,10 @@ extern int ivtv_debug;
 /* Flag to turn on high volume debugging */
 #define IVTV_DBGFLG_HIGHVOL (1  10)
 
-/* NOTE: extra space before comma in 'itv-num , ## args' is required for
-   gcc-2.95, otherwise it won't compile. */
 #define IVTV_DEBUG(x, type, fmt, args...) \
do { \
if ((x)  ivtv_debug) \
-   printk(KERN_INFO ivtv%d  type :  fmt, itv-num , ## 
args); \
+   printk(KERN_INFO ivtv%d  type :  fmt, itv-num, ## 
args); \
} while (0)
 #define IVTV_DEBUG_WARN(fmt, args...)  IVTV_DEBUG(IVTV_DBGFLG_WARN,  warn,  
fmt , ## args)
 #define IVTV_DEBUG_INFO(fmt, args...)  IVTV_DEBUG(IVTV_DBGFLG_INFO,  info,  
fmt , ## args)
diff --git a/drivers/media/video/ivtv/ivtv-ioctl.c 
b/drivers/media/video/ivtv/ivtv-ioctl.c
index fd6826f..24270de 100644
--- a/drivers/media/video/ivtv/ivtv-ioctl.c
+++ b/drivers/media/video/ivtv/ivtv-ioctl.c
@@ -688,7 +688,7 @@ static int ivtv_debug_ioctls(struct file *filp, unsigned 
int cmd, void *arg)
ivtv_reset_ir_gpio(itv);
}
if (val  0x02) {
-   itv-video_dec_func(itv, cmd, 0);
+   itv-video_dec_func(itv, cmd, NULL);
}
break;
}
diff --git a/drivers/media/video/ivtv/ivtv-irq.c 
b/drivers/media/video/ivtv/ivtv-irq.c
index fd1688e..1384615 100644
--- a/drivers/media/video/ivtv/ivtv-irq.c
+++ b/drivers/media/video/ivtv/ivtv-irq.c
@@ -204,7 +204,7 @@ static int stream_enc_dma_append(struct ivtv_stream *s, u32 
data[CX2341X_MBOX_MA
s-sg_pending[idx].dst = buf-dma_handle;
s-sg_pending[idx].src = offset;
s-sg_pending[idx].size = s-buf_size;
-   buf-bytesused = (size  s-buf_size) ? size : s-buf_size;
+   buf-bytesused = min(size, s-buf_size);
buf-dma_xfer_cnt = s-dma_xfer_cnt;
 
s-q_predma.bytesused += buf-bytesused;
@@ -705,7 +705,7 @@ static void ivtv_irq_dec_data_req(struct ivtv *itv)
s = itv-streams[IVTV_DEC_STREAM_TYPE_YUV];
}
else {
-   itv-dma_data_req_size = data[2] = 0x1 ? 0x1 : data[2];
+   itv-dma_data_req_size = min_t(u32, data[2], 0x1);
itv-dma_data_req_offset = data[1];
s = itv-streams[IVTV_DEC_STREAM_TYPE_MPG];
}
diff --git a/drivers/media/video/ivtv/ivtv-streams.c 
b/drivers/media/video/ivtv/ivtv-streams.c
index aa03e61..0e9e7d0 100644
--- a/drivers/media/video/ivtv/ivtv-streams.c
+++ b/drivers/media/video/ivtv/ivtv-streams.c
@@ -572,10 +572,10 @@ int ivtv_start_v4l2_encode_stream(struct ivtv_stream *s)
clear_bit(IVTV_F_I_EOS, itv-i_flags);
 
/* Initialize Digitizer for Capture */
-   itv-video_dec_func(itv, VIDIOC_STREAMOFF, 0);
+   itv-video_dec_func(itv, VIDIOC_STREAMOFF, NULL);
ivtv_msleep_timeout(300, 1);
ivtv_vapi(itv, CX2341X_ENC_INITIALIZE_INPUT, 0);
-   itv-video_dec_func(itv, VIDIOC_STREAMON, 0);
+   itv-video_dec_func(itv, VIDIOC_STREAMON, NULL);
}
 
/* begin_capture */
diff --git a/drivers/media/video/ivtv/ivtvfb.c 
b/drivers/media/video/ivtv/ivtvfb.c
index 52ffd15..f73ce98 100644
--- a/drivers/media/video/ivtv/ivtvfb.c
+++ b/drivers/media/video/ivtv/ivtvfb.c
@@ -1053,7 +1053,7 @@ static int ivtvfb_init_card(struct ivtv *itv)
}
 
itv-osd_info = kzalloc(sizeof(struct osd_info), GFP_ATOMIC);
-   if (itv-osd_info == 0) {
+   if (itv-osd_info == NULL) {
IVTVFB_ERR(Failed to allocate memory for osd_info\n);
return -ENOMEM;
}
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the

Re: [v4l-dvb-maintainer] [PATCH 1/2] ivtv: Some general fixes

2007-12-02 Thread Hans Verkuil 
On Sunday 02 December 2007 18:46, Richard Knutsson wrote:
 Fix warning: Using plain integer as NULL pointer.

Signed-off-by: Hans Verkuil [EMAIL PROTECTED]

 Remove a gcc-2.95 requirement.

NACK! The main v4l-dvb repository that contains this driver can also be 
compiled on older systems. I'd like to keep the option that the driver 
can be compiled with an older gcc version, unless Mauro thinks 
otherwise. Other than the removal of one comment and one space there 
are no other benefits of this change, so I'd prefer to keep it.

 Convert 'x  y ? x : y' to use min() instead.

Signed-off-by: Hans Verkuil [EMAIL PROTECTED]

Thanks,

Hans



 Signed-off-by: Richard Knutsson [EMAIL PROTECTED]
 ---
 Compile-tested on i386 with allyesconfig and allmodconfig.

  ivtv-driver.c  |2 +-
  ivtv-driver.h  |4 +---
  ivtv-ioctl.c   |2 +-
  ivtv-irq.c |4 ++--
  ivtv-streams.c |4 ++--
  ivtvfb.c   |2 +-
  6 files changed, 8 insertions(+), 10 deletions(-)


 diff --git a/drivers/media/video/ivtv/ivtv-driver.c
 b/drivers/media/video/ivtv/ivtv-driver.c index 6d2dd87..96f340c
 100644
 --- a/drivers/media/video/ivtv/ivtv-driver.c
 +++ b/drivers/media/video/ivtv/ivtv-driver.c
 @@ -979,7 +979,7 @@ static int __devinit ivtv_probe(struct pci_dev
 *dev, }

   itv = kzalloc(sizeof(struct ivtv), GFP_ATOMIC);
 - if (itv == 0) {
 + if (itv == NULL) {
   spin_unlock(ivtv_cards_lock);
   return -ENOMEM;
   }
 diff --git a/drivers/media/video/ivtv/ivtv-driver.h
 b/drivers/media/video/ivtv/ivtv-driver.h index 49ce14d..5c4956c
 100644
 --- a/drivers/media/video/ivtv/ivtv-driver.h
 +++ b/drivers/media/video/ivtv/ivtv-driver.h
 @@ -132,12 +132,10 @@ extern int ivtv_debug;
  /* Flag to turn on high volume debugging */
  #define IVTV_DBGFLG_HIGHVOL (1  10)

 -/* NOTE: extra space before comma in 'itv-num , ## args' is
 required for -   gcc-2.95, otherwise it won't compile. */
  #define IVTV_DEBUG(x, type, fmt, args...) \
   do { \
   if ((x)  ivtv_debug) \
 - printk(KERN_INFO ivtv%d  type :  fmt, itv-num , ## 
 args); \
 + printk(KERN_INFO ivtv%d  type :  fmt, itv-num, ## 
 args); \
   } while (0)
  #define IVTV_DEBUG_WARN(fmt, args...)  IVTV_DEBUG(IVTV_DBGFLG_WARN, 
 warn,  fmt , ## args) #define IVTV_DEBUG_INFO(fmt, args...) 
 IVTV_DEBUG(IVTV_DBGFLG_INFO,  info,  fmt , ## args) diff --git
 a/drivers/media/video/ivtv/ivtv-ioctl.c
 b/drivers/media/video/ivtv/ivtv-ioctl.c index fd6826f..24270de 100644
 --- a/drivers/media/video/ivtv/ivtv-ioctl.c
 +++ b/drivers/media/video/ivtv/ivtv-ioctl.c
 @@ -688,7 +688,7 @@ static int ivtv_debug_ioctls(struct file *filp,
 unsigned int cmd, void *arg) ivtv_reset_ir_gpio(itv);
   }
   if (val  0x02) {
 - itv-video_dec_func(itv, cmd, 0);
 + itv-video_dec_func(itv, cmd, NULL);
   }
   break;
   }
 diff --git a/drivers/media/video/ivtv/ivtv-irq.c
 b/drivers/media/video/ivtv/ivtv-irq.c index fd1688e..1384615 100644
 --- a/drivers/media/video/ivtv/ivtv-irq.c
 +++ b/drivers/media/video/ivtv/ivtv-irq.c
 @@ -204,7 +204,7 @@ static int stream_enc_dma_append(struct
 ivtv_stream *s, u32 data[CX2341X_MBOX_MA s-sg_pending[idx].dst =
 buf-dma_handle;
   s-sg_pending[idx].src = offset;
   s-sg_pending[idx].size = s-buf_size;
 - buf-bytesused = (size  s-buf_size) ? size : s-buf_size;
 + buf-bytesused = min(size, s-buf_size);
   buf-dma_xfer_cnt = s-dma_xfer_cnt;

   s-q_predma.bytesused += buf-bytesused;
 @@ -705,7 +705,7 @@ static void ivtv_irq_dec_data_req(struct ivtv
 *itv) s = itv-streams[IVTV_DEC_STREAM_TYPE_YUV];
   }
   else {
 - itv-dma_data_req_size = data[2] = 0x1 ? 0x1 : data[2];
 + itv-dma_data_req_size = min_t(u32, data[2], 0x1);
   itv-dma_data_req_offset = data[1];
   s = itv-streams[IVTV_DEC_STREAM_TYPE_MPG];
   }
 diff --git a/drivers/media/video/ivtv/ivtv-streams.c
 b/drivers/media/video/ivtv/ivtv-streams.c index aa03e61..0e9e7d0
 100644
 --- a/drivers/media/video/ivtv/ivtv-streams.c
 +++ b/drivers/media/video/ivtv/ivtv-streams.c
 @@ -572,10 +572,10 @@ int ivtv_start_v4l2_encode_stream(struct
 ivtv_stream *s) clear_bit(IVTV_F_I_EOS, itv-i_flags);

   /* Initialize Digitizer for Capture */
 - itv-video_dec_func(itv, VIDIOC_STREAMOFF, 0);
 + itv-video_dec_func(itv, VIDIOC_STREAMOFF, NULL);
   ivtv_msleep_timeout(300, 1);
   ivtv_vapi(itv, CX2341X_ENC_INITIALIZE_INPUT, 0);
 - itv-video_dec_func(itv, VIDIOC_STREAMON, 0);
 + itv-video_dec_func(itv, VIDIOC_STREAMON, NULL);
   }

   /* begin_capture */
 diff --git a/drivers/media/video/ivtv/ivtvfb.c
 b/drivers/media/video/ivtv/ivtvfb.c index 52ffd15..f73ce98 100644
 --- a/drivers/media/video/ivtv/ivtvfb.c

Re: [v4l-dvb-maintainer] [PATCH 1/2] ivtv: Some general fixes

2007-12-02 Thread Richard Knutsson 

Hans Verkuil wrote:

On Sunday 02 December 2007 18:46, Richard Knutsson wrote:
  

Fix warning: Using plain integer as NULL pointer.



Signed-off-by: Hans Verkuil [EMAIL PROTECTED]

  

Remove a gcc-2.95 requirement.



NACK! The main v4l-dvb repository that contains this driver can also be 
compiled on older systems. I'd like to keep the option that the driver 
can be compiled with an older gcc version, unless Mauro thinks 
otherwise. Other than the removal of one comment and one space there 
are no other benefits of this change, so I'd prefer to keep it.


  
Oh alright, thought it was just a leftover since the support for the 
2.95 ended.
Maybe add something to the text to let others know the status? (Since I 
found it by doing a 'grep' for gcc 2.95's...)

Convert 'x  y ? x : y' to use min() instead.



Signed-off-by: Hans Verkuil [EMAIL PROTECTED]

Thanks,

Hans

  

Thanks for the fast turnaround
Richard

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/