All of lore.kernel.org
 help / color / mirror / Atom feed
* [Kernel-janitors] [PATCH] radio/radio-maestro: replace
@ 2004-07-26 22:36 Nishanth Aravamudan
  2004-07-26 22:37 ` [Kernel-janitors] [PATCH] radio/radio-maxiradio: replace Nishanth Aravamudan
  2004-08-14 18:51 ` [Kernel-janitors] [PATCH] radio/radio-maestro: replace maximilian attems
  0 siblings, 2 replies; 3+ messages in thread
From: Nishanth Aravamudan @ 2004-07-26 22:36 UTC (permalink / raw)
  To: kernel-janitors

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

I would appreciate any comments from the janitors list.

Thanks,
Nish

Applys-to: 2.6.7

Description: Replaced sleep_125ms() with msleep(125), udelay2()
with udelay(2), udelay4() with udelay(4) and udelay16() with
udelay(16) and removed the replaced functions' definitions.

Signed-off-by: Nishanth Aravamudan <nacc@us.ibm.com>


--- linux-vanilla/drivers/media/radio/radio-maestro.c	2004-06-16 05:19:36.000000000 +0000
+++ linux-dev/drivers/media/radio/radio-maestro.c	2004-07-02 20:41:43.000000000 +0000
@@ -93,27 +93,6 @@ static struct radio_device
 	struct  semaphore lock;
 } radio_unit = {0, 0, 0, 0, };
 
-static void sleep_125ms(void)
-{
-	current->state = TASK_INTERRUPTIBLE;
-	schedule_timeout(HZ >> 3);
-}
-
-static void udelay2(void)
-{
-	udelay(2);
-}
-
-static void udelay4(void)
-{
-	udelay(4);
-}
-
-static void udelay16(void)
-{
-	udelay(16);
-}
-
 static __u32 radio_bits_get(struct radio_device *dev)
 {
 	register __u16 io=dev->io, l, rdata;
@@ -122,14 +101,15 @@ static __u32 radio_bits_get(struct radio
 	omask = inw(io + IO_MASK);
 	outw(~(STR_CLK | STR_WREN), io + IO_MASK);
 	outw(0, io);
-	udelay16();
+	udelay(16);
+
 	for (l=24;l--;) {
 		outw(STR_CLK, io);		/* HI state */
-		udelay2();
+		udelay(2);
 		if(!l) 
 			dev->tuned = inw(io) & STR_MOST ? 0 : 0xffff;
 		outw(0, io);			/* LO state */
-		udelay2();
+		udelay(2);
 		data <<= 1;			/* shift data */
 		rdata = inw(io);
 		if(!l)
@@ -138,11 +118,11 @@ static __u32 radio_bits_get(struct radio
 		else
 			if(rdata & STR_DATA)
 				data++;
-		udelay2();
+		udelay(2);
 	}
 	if(dev->muted)
 		outw(STR_WREN, io);
-	udelay4();
+	udelay(4);
 	outw(omask, io + IO_MASK);
 	return data & 0x3ffe;
 }
@@ -155,23 +135,23 @@ static void radio_bits_set(struct radio_
 	odir  = (inw(io + IO_DIR) & ~STR_DATA) | (STR_CLK | STR_WREN);
 	outw(odir | STR_DATA, io + IO_DIR);
 	outw(~(STR_DATA | STR_CLK | STR_WREN), io + IO_MASK);
-	udelay16();
+	udelay(16);
 	for (l=25;l;l--) {
 		bits = ((data >> 18) & STR_DATA) | STR_WREN ;
 		data <<= 1;			/* shift data */
 		outw(bits, io);			/* start strobe */
-		udelay2();
+		udelay(2);
 		outw(bits | STR_CLK, io);	/* HI level */
-		udelay2();   
+		udelay(2);   
 		outw(bits, io);			/* LO level */
-		udelay4();
+		udelay(4);
 	}
 	if(!dev->muted)
 		outw(0, io);
-	udelay4();
+	udelay(4);
 	outw(omask, io + IO_MASK);
 	outw(odir, io + IO_DIR);
-	sleep_125ms();
+	msleep(125);
 }
 
 inline static int radio_function(struct inode *inode, struct file *file,
@@ -238,9 +218,9 @@ inline static int radio_function(struct 
 				outw(~STR_WREN, io + IO_MASK);
 				outw((card->muted = v->flags & VIDEO_AUDIO_MUTE)
 				     ? STR_WREN : 0, io);
-				udelay4();
+				udelay(4);
 				outw(omask, io + IO_MASK);
-				sleep_125ms();
+				msleep(125);
 				return 0;
 			}
 		}
@@ -315,7 +295,7 @@ inline static __u16 radio_power_on(struc
 	outw(odir, io + IO_DIR);
 	outw(~(STR_WREN | STR_CLK), io + IO_MASK);
 	outw(dev->muted ? 0 : STR_WREN, io);
-	udelay16();
+	udelay(16);
 	outw(omask, io + IO_MASK);
 	ofreq = radio_bits_get(dev);
 	if((ofreq<FREQ2BITS(FREQ_LO)) || (ofreq>FREQ2BITS(FREQ_HI)))

[-- 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

^ permalink raw reply	[flat|nested] 3+ messages in thread
* [Kernel-janitors] [PATCH] radio/radio-maxiradio: replace
  2004-07-26 22:36 [Kernel-janitors] [PATCH] radio/radio-maestro: replace Nishanth Aravamudan
@ 2004-07-26 22:37 ` Nishanth Aravamudan
  2004-08-14 18:51 ` [Kernel-janitors] [PATCH] radio/radio-maestro: replace maximilian attems
  1 sibling, 0 replies; 3+ messages in thread
From: Nishanth Aravamudan @ 2004-07-26 22:37 UTC (permalink / raw)
  To: kernel-janitors

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

I would appreciate any comments from the janitors list.

Thanks,
Nish

Applys-to: 2.6.7

Description: Replaced sleep_125ms() with msleep(125) and replaced the
replaced function's definition.

Signed-off-by: Nishanth Aravamudan <nacc@us.ibm.com>


--- linux-vanilla/drivers/media/radio/radio-maxiradio.c	2004-06-16 05:20:03.000000000 +0000
+++ linux-dev/drivers/media/radio/radio-maxiradio.c	2004-07-02 20:39:37.000000000 +0000
@@ -104,13 +104,6 @@ static struct radio_device
 } radio_unit = {0, 0, 0, 0, };
 
 
-static void sleep_125ms(void)
-{
-	current->state = TASK_INTERRUPTIBLE;
-	schedule_timeout(HZ >> 3);
-}
-
-
 static void outbit(unsigned long bit, __u16 io)
 {
 	if(bit != 0)
@@ -228,7 +221,7 @@ inline static int radio_function(struct 
 				return -EINVAL;
 			card->freq = *freq;
 			set_freq(card->io, FREQ2BITS(card->freq));
-			sleep_125ms();
+			msleep(125);
 			return 0;
 		}
 		case VIDIOCGAUDIO: {	

[-- 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

^ permalink raw reply	[flat|nested] 3+ messages in thread
* Re: [Kernel-janitors] [PATCH] radio/radio-maestro: replace
  2004-07-26 22:36 [Kernel-janitors] [PATCH] radio/radio-maestro: replace Nishanth Aravamudan
  2004-07-26 22:37 ` [Kernel-janitors] [PATCH] radio/radio-maxiradio: replace Nishanth Aravamudan
@ 2004-08-14 18:51 ` maximilian attems
  1 sibling, 0 replies; 3+ messages in thread
From: maximilian attems @ 2004-08-14 18:51 UTC (permalink / raw)
  To: kernel-janitors

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

On Mon, 26 Jul 2004, Nishanth Aravamudan wrote:

> I would appreciate any comments from the janitors list.
> 
> Thanks,
> Nish
> 
> Applys-to: 2.6.7
> 
> Description: Replaced sleep_125ms() with msleep(125), udelay2()
> with udelay(2), udelay4() with udelay(4) and udelay16() with
> udelay(16) and removed the replaced functions' definitions.
> 
> Signed-off-by: Nishanth Aravamudan <nacc@us.ibm.com>
> 
> 
> --- linux-vanilla/drivers/media/radio/radio-maestro.c	2004-06-16 05:19:36.000000000 +0000
> +++ linux-dev/drivers/media/radio/radio-maestro.c	2004-07-02 20:41:43.000000000 +0000
> @@ -93,27 +93,6 @@ static struct radio_device
>  	struct  semaphore lock;
>  } radio_unit = {0, 0, 0, 0, };
>  
> -static void sleep_125ms(void)
> -{
> -	current->state = TASK_INTERRUPTIBLE;
> -	schedule_timeout(HZ >> 3);
> -}
> -
> -static void udelay2(void)
> -{
> -	udelay(2);
> -}
> -
> -static void udelay4(void)
> -{
> -	udelay(4);
> -}
> -
> -static void udelay16(void)
> -{
> -	udelay(16);
> -}
> -
>  static __u32 radio_bits_get(struct radio_device *dev)
>  {
>  	register __u16 io=dev->io, l, rdata;
> @@ -122,14 +101,15 @@ static __u32 radio_bits_get(struct radio
>  	omask = inw(io + IO_MASK);
>  	outw(~(STR_CLK | STR_WREN), io + IO_MASK);
>  	outw(0, io);
> -	udelay16();
> +	udelay(16);
> +
>  	for (l=24;l--;) {
>  		outw(STR_CLK, io);		/* HI state */
> -		udelay2();
> +		udelay(2);
>  		if(!l) 
>  			dev->tuned = inw(io) & STR_MOST ? 0 : 0xffff;
>  		outw(0, io);			/* LO state */
> -		udelay2();
> +		udelay(2);
>  		data <<= 1;			/* shift data */
>  		rdata = inw(io);
>  		if(!l)
> @@ -138,11 +118,11 @@ static __u32 radio_bits_get(struct radio
>  		else
>  			if(rdata & STR_DATA)
>  				data++;
> -		udelay2();
> +		udelay(2);
>  	}
>  	if(dev->muted)
>  		outw(STR_WREN, io);
> -	udelay4();
> +	udelay(4);
>  	outw(omask, io + IO_MASK);
>  	return data & 0x3ffe;
>  }
> @@ -155,23 +135,23 @@ static void radio_bits_set(struct radio_
>  	odir  = (inw(io + IO_DIR) & ~STR_DATA) | (STR_CLK | STR_WREN);
>  	outw(odir | STR_DATA, io + IO_DIR);
>  	outw(~(STR_DATA | STR_CLK | STR_WREN), io + IO_MASK);
> -	udelay16();
> +	udelay(16);
>  	for (l=25;l;l--) {
>  		bits = ((data >> 18) & STR_DATA) | STR_WREN ;
>  		data <<= 1;			/* shift data */
>  		outw(bits, io);			/* start strobe */
> -		udelay2();
> +		udelay(2);
>  		outw(bits | STR_CLK, io);	/* HI level */
> -		udelay2();   
> +		udelay(2);   
here the trailing space existed already before ;)
>  		outw(bits, io);			/* LO level */
> -		udelay4();
> +		udelay(4);
>  	}
>  	if(!dev->muted)
>  		outw(0, io);
> -	udelay4();
> +	udelay(4);
>  	outw(omask, io + IO_MASK);
>  	outw(odir, io + IO_DIR);
> -	sleep_125ms();
> +	msleep(125);
>  }
>  
>  inline static int radio_function(struct inode *inode, struct file *file,
> @@ -238,9 +218,9 @@ inline static int radio_function(struct 
>  				outw(~STR_WREN, io + IO_MASK);
>  				outw((card->muted = v->flags & VIDEO_AUDIO_MUTE)
>  				     ? STR_WREN : 0, io);
> -				udelay4();
> +				udelay(4);
>  				outw(omask, io + IO_MASK);
> -				sleep_125ms();
> +				msleep(125);
>  				return 0;
>  			}
>  		}
> @@ -315,7 +295,7 @@ inline static __u16 radio_power_on(struc
>  	outw(odir, io + IO_DIR);
>  	outw(~(STR_WREN | STR_CLK), io + IO_MASK);
>  	outw(dev->muted ? 0 : STR_WREN, io);
> -	udelay16();
> +	udelay(16);
>  	outw(omask, io + IO_MASK);
>  	ofreq = radio_bits_get(dev);
>  	if((ofreq<FREQ2BITS(FREQ_LO)) || (ofreq>FREQ2BITS(FREQ_HI)))

removed for next kjt.
a++ maks


[-- 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

^ permalink raw reply	[flat|nested] 3+ messages in thread
end of thread, other threads:[~2004-08-14 18:51 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-07-26 22:36 [Kernel-janitors] [PATCH] radio/radio-maestro: replace Nishanth Aravamudan
2004-07-26 22:37 ` [Kernel-janitors] [PATCH] radio/radio-maxiradio: replace Nishanth Aravamudan
2004-08-14 18:51 ` [Kernel-janitors] [PATCH] radio/radio-maestro: replace maximilian attems

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.