[Kernel-janitors] Re: [PATCH 2.6.9-rc2 8/17] media/ovcamchip_core:

[Kernel-janitors] Re: [PATCH 2.6.9-rc2 8/17] media/ovcamchip_core:

[Mark McClelland]

[-- Attachment #1: Type: text/plain, Size: 1531 bytes --]

Nishanth Aravamudan wrote:

>Any comments would be appreciated.
>
>Description: Use msleep() instead of schedule_timeout() to
>guarantee the task delays as expected.
>  
>

Looks good to me. Thank you for doing this.

Should I send this on to Andrew/Linus, or were you (or some automated 
janitor-patch-bot) planning to do that?

Thanks,
- Mark

>Signed-off-by: Nishanth Aravamudan <nacc@us.ibm.com>
>
>--- 2.6.9-rc2-vanilla/drivers/media/video/ovcamchip/ovcamchip_core.c	2004-09-13 17:15:39.000000000 -0700
>+++ 2.6.9-rc2/drivers/media/video/ovcamchip/ovcamchip_core.c	2004-09-20 14:53:01.000000000 -0700
>@@ -15,6 +15,7 @@
> #include <linux/module.h>
> #include <linux/moduleparam.h>
> #include <linux/slab.h>
>+#include <linux/delay.h>
> #include "ovcamchip_priv.h"
> 
> #define DRIVER_VERSION "v2.27 for Linux 2.6"
>@@ -128,8 +129,7 @@ static int init_camchip(struct i2c_clien
> 	ov_write(c, 0x12, 0x80);
> 
> 	/* Wait for it to initialize */
>-	set_current_state(TASK_UNINTERRUPTIBLE);
>-	schedule_timeout(1 + 150 * HZ / 1000);
>+	msleep(150);
> 
> 	for (i = 0, success = 0; i < I2C_DETECT_RETRIES && !success; i++) {
> 		if (ov_read(c, GENERIC_REG_ID_HIGH, &high) >= 0) {
>@@ -145,8 +145,7 @@ static int init_camchip(struct i2c_clien
> 		ov_write(c, 0x12, 0x80);
> 
> 		/* Wait for it to initialize */
>-		set_current_state(TASK_UNINTERRUPTIBLE);
>-		schedule_timeout(1 + 150 * HZ / 1000);
>+		msleep(150);
> 
> 		/* Dummy read to sync I2C */
> 		ov_read(c, 0x00, &low);
>  
>


-- 
Mark McClelland
mark AT alpha.dyndns.org


[-- Attachment #2: Type: text/plain, Size: 167 bytes --]

_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@lists.osdl.org
http://lists.osdl.org/mailman/listinfo/kernel-janitors

[Kernel-janitors] Re: [PATCH 2.6.9-rc2 8/17] media/ovcamchip_core:

[Nishanth Aravamudan]

[-- Attachment #1: Type: text/plain, Size: 553 bytes --]

On Wed, Sep 22, 2004 at 05:38:32AM -0700, Mark McClelland wrote:
> Nishanth Aravamudan wrote:
> 
> >Any comments would be appreciated.
> >
> >Description: Use msleep() instead of schedule_timeout() to
> >guarantee the task delays as expected.
> > 
> >
> 
> Looks good to me. Thank you for doing this.
> 
> Should I send this on to Andrew/Linus, or were you (or some automated 
> janitor-patch-bot) planning to do that?

If you don't send it to Andrew/Linus, then Max (the kernel-janitors
maintainer) will take care of it, from what I understand.

-Nish

[-- Attachment #2: Type: text/plain, Size: 167 bytes --]

_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@lists.osdl.org
http://lists.osdl.org/mailman/listinfo/kernel-janitors