Re: [PATCH v2] Enable serial output for Oxford Semiconductor PCIe cards and fixes.

[Aravind Gopalakrishnan <aravind.gopalakrishnan@amd.com>]

On 3/7/2014 1:01 PM, Konrad Rzeszutek Wilk wrote:
> Since v1 [http://mid.gmane.org/1394040334-16278-1-git-send-email-konrad.wilk@oracle.com]:
>   - Update it per Jan and Andrew comments
>   - Tackle on two bug-fixes.
>
>
> Hey,
>
> This is the second, posting of the changes to the ns16550 driver
> to also support other chipsets - like the Oxford 16C950 one - popular on
> the inexpensive Rosewill RC-300E.
>
> There are also two bug-fixes that I discovered while testing my code.
> One of them was found through code inspection while the other was
> when I was testing my code.
>
> Of interest is:
>   [PATCH v2 3/7] serial: Support OXPCIe952 aka Oxford Semiconductor
>
>   that enables the functionality on the PCIe card I have - while the rest
> of the patches flesh out the code to support more of PCI and PCIe type cards.
>
>   xen/arch/x86/oprofile/op_model_athlon.c |    5 +-
>   xen/arch/x86/x86_64/mmconf-fam10h.c     |    1 +
>   xen/arch/x86/x86_64/mmconfig-shared.c   |    1 +
>   xen/arch/x86/x86_64/mmconfig.h          |    7 -
>   xen/drivers/char/ns16550.c              |  254 ++++++++++++++++++++++++++++--
>   xen/include/xen/pci_ids.h               |   12 ++
>   6 files changed, 253 insertions(+), 27 deletions(-)
>
> Konrad Rzeszutek Wilk (7):
>        serial: Skip over PCIe device which have no quirks (fix AMT regression).
>        serial: Fix COM1 assumption if pci_uart_config did not find the PCI serial card.
>        serial: Support OXPCIe952 aka Oxford Semiconductor Ltd Device c138 (1415:c138)
>        serial: Seperate the PCI device ids and parameters
>        serial: Use #defines for PCI vendors.
>        serial: Expand the PCI serial quirks for OXPCIe200 and OXPCIe952 1 Native UART
>        pci: Put all PCI device vendor and models in one file.
>
>

Tested the patch set with Broadcom Trumanage MMIO device just to make 
sure there are no regressions and can verify that it works fine..

Thanks,
-Aravind.