Re: [Qemu-trivial] [Qemu-devel] [PATCH] serial-pci: Set prog interface field of pci config to 16550 compatible

[BALATON Zoltan]

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 <address@hidden>
> > > ---
> > > 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