Skip to contentSkip to conversation

[2.5] Fix compilation of ip2main (fwd)

PATCH In review Adrian Bunk 2 participants 2 messages Jun 24, 2003 Char and misc drivers

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 [2.5] Fix compilation of ip2main (fwd)

Could anyone comment on this patch?

This patch by Eduardo Pereira Habkost <[email protected]> or something similar is still needed in 2.5.73 to fix the compilation of ip2main.c .

TIA
Adrian

----- Forwarded message from Eduardo Pereira Habkost <[email protected]> -----

Date: Wed, 11 Jun 2003 19:32:21 -0300
From: Eduardo Pereira Habkost <[email protected]>
To: "Michael H. Warfield" <[email protected]>
Cc: [email protected]
Subject: [PATCH][2.5] Fix compilation of ip2main

The following patch fix compilation of drivers/char/ip2main.c. It was broken by the removal of pci_present().

It just adds open and closing braces around the code that declares the pci_dev_i variable. The rest of the patch just change the indentation.

--
Eduardo

drivers/char/ip2main.c +31 -29
diff -Nru a/drivers/char/ip2main.c b/drivers/char/ip2main.c
@@ -691,40 +691,42 @@
 				} 
 			} 
 #else /* LINUX_VERSION_CODE > 2.1.99 */
-			struct pci_dev *pci_dev_i = NULL;
-			pci_dev_i = pci_find_device(PCI_VENDOR_ID_COMPUTONE,
-						  PCI_DEVICE_ID_COMPUTONE_IP2EX, pci_dev_i);
-			if (pci_dev_i != NULL) {
-				unsigned int addr;
-				unsigned char pci_irq;
+			{
+				struct pci_dev *pci_dev_i = NULL;
+				pci_dev_i = pci_find_device(PCI_VENDOR_ID_COMPUTONE,
+							  PCI_DEVICE_ID_COMPUTONE_IP2EX, pci_dev_i);
+				if (pci_dev_i != NULL) {
+					unsigned int addr;
+					unsigned char pci_irq;
 
-				ip2config.type[i] = PCI;
-				status =
-				pci_read_config_dword(pci_dev_i, PCI_BASE_ADDRESS_1, &addr);
-				if ( addr & 1 ) {
-					ip2config.addr[i]=(USHORT)(addr&0xfffe);
-				} else {
-					printk( KERN_ERR "IP2: PCI I/O address error\n");
-				}
-				status =
-				pci_read_config_byte(pci_dev_i, PCI_INTERRUPT_LINE, &pci_irq);
+					ip2config.type[i] = PCI;
+					status =
+					pci_read_config_dword(pci_dev_i, PCI_BASE_ADDRESS_1, &addr);
+					if ( addr & 1 ) {
+						ip2config.addr[i]=(USHORT)(addr&0xfffe);
+					} else {
+						printk( KERN_ERR "IP2: PCI I/O address error\n");
+					}
+					status =
+					pci_read_config_byte(pci_dev_i, PCI_INTERRUPT_LINE, &pci_irq);
 
 //		If the PCI BIOS assigned it, lets try and use it.  If we
 //		can't acquire it or it screws up, deal with it then.
 
-//				if (!is_valid_irq(pci_irq)) {
-//					printk( KERN_ERR "IP2: Bad PCI BIOS IRQ(%d)\n",pci_irq);
-//					pci_irq = 0;
-//				}
-				ip2config.irq[i] = pci_irq;
-			} else {	// ann error
-				ip2config.addr[i] = 0;
-				if (status == PCIBIOS_DEVICE_NOT_FOUND) {
-					printk( KERN_ERR "IP2: PCI board %d not found\n", i );
-				} else {
-					pcibios_strerror(status);
-				}
-			} 
+//					if (!is_valid_irq(pci_irq)) {
+//						printk( KERN_ERR "IP2: Bad PCI BIOS IRQ(%d)\n",pci_irq);
+//						pci_irq = 0;
+//					}
+					ip2config.irq[i] = pci_irq;
+				} else {	// ann error
+					ip2config.addr[i] = 0;
+					if (status == PCIBIOS_DEVICE_NOT_FOUND) {
+						printk( KERN_ERR "IP2: PCI board %d not found\n", i );
+					} else {
+						pcibios_strerror(status);
+					}
+				} 
+			}
 #endif	/* ! 2_0_X */
 #else
 			printk( KERN_ERR "IP2: PCI card specified but PCI support not\n");

----- End forwarded message -----

#1 lore raw
Quoted 5 lines · Adrian Bunk

Could anyone comment on this patch?

This patch by Eduardo Pereira Habkost <[email protected]> or something similar is still needed in 2.5.73 to fix the compilation of ip2main.c .

Bleah, yeah, that's my fault. I'll apply this and forward it on to Linus.

Sorry about that.

greg k-h