On Fri, 14 Mar 2014, Michael Tokarev wrote:
10.03.2014 22:40, BALATON Zoltan wrote:
Ping!
http://patchwork.ozlabs.org/patch/324674/
On Thu, 27 Feb 2014, BALATON Zoltan wrote:
Signed-off-by: BALATON Zoltan <bala...@eik.bme.hu>
---
hw/char/serial-pci.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/hw/char/serial-pci.c b/hw/char/serial-pci.c
index 991c99f..e662b77 100644
--- a/hw/char/serial-pci.c
+++ b/hw/char/serial-pci.c
@@ -60,6 +60,7 @@ static int serial_pci_init(PCIDevice *dev)
return -1;
}
+ pci->dev.config[PCI_CLASS_PROG] = 0x02; /* 16550 compatible */
pci->dev.config[PCI_INTERRUPT_PIN] = 0x01;
s->irq = pci_allocate_irq(&pci->dev);
Hm. While this is an one-liner indeed, I'm not sure it's the right
value. From my pci.ids:
C 07 Communication controller
00 Serial controller
00 8250
01 16450
02 16550
03 16650
04 16750
05 16850
06 16950
so it seems correct thing to do.
What about multi_serial_pci_init() -- should it too set prog-if
like this?
I'm not completely sure what the correct value should be for
multi_serial_pci_init so I've left that alone. There's also 07 02 00 that
means multiport serial controller that might apply for that case. For the
simple case I've sent the patch for I had a client expecting this value so
I think it is correct for that at least but I don't know about the
multiport case.
Regards,
BALATON Zoltan
