Commit 7a171cdc authored by Russell King's avatar Russell King Committed by Russell King

[SERIAL] Fix two bugs in parport_serial

Steinar H. Gunderson reported:

  - For some reason, it detects the 9845 as a 9735 -- it appears this is
    simply related to the ordering in parport_serial_pci_tbl[]. If we move
    the 9845 up above the 9735, it prints out 9710:9845, but no change in
    behaviour. (We didn't find out why this was the case; we left it alone
    since it didn't affect our problem.)
  - The card has no parallel port (at least no physical ones), yet it reports
    (via its subsystem ID of 0x0014) one parallel port and four serial ports.
    The probe for the parallel port fails, and the driver just aborts. Thus,
    it doesn't find the serial ports.

Fix the debugging code to use dev_dbg, but don't bother displaying the
PCI ID of the detected board (that's accessible via other means.)

Also, arrange for parport_register() to return 0 even if it finds no
ports.
Signed-off-by: default avatarRussell King <rmk+kernel@arm.linux.org.uk>
parent 501f74f2
...@@ -312,8 +312,7 @@ static int __devinit parport_register (struct pci_dev *dev, ...@@ -312,8 +312,7 @@ static int __devinit parport_register (struct pci_dev *dev,
{ {
struct parport_pc_pci *card; struct parport_pc_pci *card;
struct parport_serial_private *priv = pci_get_drvdata (dev); struct parport_serial_private *priv = pci_get_drvdata (dev);
int i = id->driver_data, n; int n, success = 0;
int success = 0;
priv->par = cards[id->driver_data]; priv->par = cards[id->driver_data];
card = &priv->par; card = &priv->par;
...@@ -344,10 +343,8 @@ static int __devinit parport_register (struct pci_dev *dev, ...@@ -344,10 +343,8 @@ static int __devinit parport_register (struct pci_dev *dev,
"hi" as an offset (see SYBA "hi" as an offset (see SYBA
def.) */ def.) */
/* TODO: test if sharing interrupts works */ /* TODO: test if sharing interrupts works */
printk (KERN_DEBUG "PCI parallel port detected: %04x:%04x, " dev_dbg(&dev->dev, "PCI parallel port detected: I/O at "
"I/O at %#lx(%#lx)\n", "%#lx(%#lx)\n", io_lo, io_hi);
parport_serial_pci_tbl[i].vendor,
parport_serial_pci_tbl[i].device, io_lo, io_hi);
port = parport_pc_probe_port (io_lo, io_hi, PARPORT_IRQ_NONE, port = parport_pc_probe_port (io_lo, io_hi, PARPORT_IRQ_NONE,
PARPORT_DMA_NONE, dev); PARPORT_DMA_NONE, dev);
if (port) { if (port) {
...@@ -359,7 +356,7 @@ static int __devinit parport_register (struct pci_dev *dev, ...@@ -359,7 +356,7 @@ static int __devinit parport_register (struct pci_dev *dev,
if (card->postinit_hook) if (card->postinit_hook)
card->postinit_hook (dev, card, !success); card->postinit_hook (dev, card, !success);
return success ? 0 : 1; return 0;
} }
static int __devinit parport_serial_pci_probe (struct pci_dev *dev, static int __devinit parport_serial_pci_probe (struct pci_dev *dev,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment