Re: [PATCH 2/2] mcb: add Altera PCI ID to mcb-pci

From: Michael Moese
Date: Tue Feb 27 2018 - 08:37:32 EST


Looks good.

Reviewed-by: Michael Moese <mmoese@xxxxxxx>


On Tue, Feb 27, 2018 at 02:30:18PM +0100, Johannes Thumshirn wrote:
> Some older PCI attached MEN FPGAs use an Altera PCI Vendor ID instead
> of the MEN one. Add it to the PCI ID table so the driver automatically
> attaches to it.
>
> Signed-off-by: Johannes Thumshirn <jthumshirn@xxxxxxx>
> Reported-by: Ben Turner <ben.turner@xxxxxxxxx>
> Tested-by: Ben Turner <ben.turner@xxxxxxxxx>
> Cc: Andreas Geißler <andreas.geissler@xxxxxx>
> Cc: Michael Moese <mmoese@xxxxxxx>
> ---
> drivers/mcb/mcb-pci.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/mcb/mcb-pci.c b/drivers/mcb/mcb-pci.c
> index af4d2f26f1c6..286b32ca526f 100644
> --- a/drivers/mcb/mcb-pci.c
> +++ b/drivers/mcb/mcb-pci.c
> @@ -11,6 +11,7 @@
>
> #include <linux/module.h>
> #include <linux/pci.h>
> +#include <linux/pci_ids.h>
> #include <linux/mcb.h>
>
> #include "mcb-internal.h"
> @@ -117,6 +118,7 @@ static void mcb_pci_remove(struct pci_dev *pdev)
>
> static const struct pci_device_id mcb_pci_tbl[] = {
> { PCI_DEVICE(PCI_VENDOR_ID_MEN, PCI_DEVICE_ID_MEN_CHAMELEON) },
> + { PCI_DEVICE(PCI_VENDOR_ID_ALTERA, PCI_DEVICE_ID_MEN_CHAMELEON) },
> { 0 },
> };
> MODULE_DEVICE_TABLE(pci, mcb_pci_tbl);
> --
> 2.13.6
>

--
SUSE Linux GmbH, GF: Felix Imendörffer, Jane Smithard, Graham Norton, HRB 21284 (AG Nürnberg)