Skip to contentSkip to conversation

fix four OSS u16 comparisons

PATCH New Adrian Bunk 1 participant 1 message Jun 21, 2003

lore
Share & tools
lore

Adrian Bunk

author
#0 lore raw

PATCH 1/1 fix four OSS u16 comparisons

The patch below fixees four OSS drivers that compared an u16 with 0xFFFFFF ...

cu
Adrian

drivers/sound/ymfpci.c renamed from drivers/sound/ymfpci.c.old +1 -1
@@ -2460,7 +2460,7 @@
 	}
 
 	eid = ymfpci_codec_read(codec, AC97_EXTENDED_ID);
-	if (eid==0xFFFFFF) {
+	if (eid==0xFFFF) {
 		printk(KERN_WARNING "ymfpci: no codec attached ?\n");
 		goto out_kfree;
 	}
drivers/sound/i810_audio.c renamed from drivers/sound/i810_audio.c.old +1 -1
@@ -2921,7 +2921,7 @@
 		/* Don't attempt to get eid until powerup is complete */
 		eid = i810_ac97_get(codec, AC97_EXTENDED_ID);
 
-		if(eid==0xFFFFFF)
+		if(eid==0xFFFF)
 		{
 			printk(KERN_WARNING "i810_audio: no codec attached ?\n");
 			kfree(codec);
drivers/sound/cs46xx.c renamed from drivers/sound/cs46xx.c.old +1 -1
@@ -4260,7 +4260,7 @@
 
 		eid = cs_ac97_get(codec, AC97_EXTENDED_ID);
 		
-		if(eid==0xFFFFFF)
+		if(eid==0xFFFF)
 		{
 			printk(KERN_WARNING "cs46xx: codec %d not present\n",num_ac97);
 			kfree(codec);
drivers/sound/ali5455.c renamed from drivers/sound/ali5455.c.old +1 -1
@@ -3277,7 +3277,7 @@
 		card->ac97_status = 0;
 		/* Don't attempt to get eid until powerup is complete */
 		eid = ali_ac97_get(codec, AC97_EXTENDED_ID);
-		if (eid == 0xFFFFFF) {
+		if (eid == 0xFFFF) {
 			printk(KERN_ERR "ali_audio: no codec attached ?\n");
 			kfree(codec);
 			break;