From: Eli Billauer <eli.billauer@gmail.com>
To: Johannes Thumshirn <jthumshirn@suse.de>
Cc: Greg KH <gregkh@linuxfoundation.org>,
Bjorn Helgaas <helgaas@kernel.org>,
Linux Kernel Mailinglist <linux-kernel@vger.kernel.org>,
Michael Moese <mmoese@suse.de>,
Andy Shevchenko <andy.shevchenko@gmail.com>,
linux-pci@vger.kernel.org, Bjorn Helgaas <bhelgaas@google.com>,
Anatolij Gustschin <agust@denx.de>
Subject: Re: [PATCH v2 1/2] PCI: add vendor id for Altera
Date: Wed, 28 Feb 2018 13:59:35 +0200 [thread overview]
Message-ID: <5A9699A7.6060801@gmail.com> (raw)
In-Reply-To: <20180228084418.7566-2-jthumshirn@suse.de>
Thanks.
Acked-by: Eli Billauer <eli.billauer@gmail.com>
On 28/02/18 10:44, Johannes Thumshirn wrote:
> Add the Altera PCI Vendor id to pci_ids.h and remove the private
> definitions from xillybus_pcie.c and altera-cvp.c.
>
> Signed-off-by: Johannes Thumshirn<jthumshirn@suse.de>
> Cc: Bjorn Helgaas<bhelgaas@google.com>
> Cc: Eli Billauer<eli.billauer@gmail.com>
> Cc: Anatolij Gustschin<agust@denx.de>
>
> ---
> Changes to v1:
> - Remove redundant pci_ids.h include (Andy)
> ---
> drivers/char/xillybus/xillybus_pcie.c | 1 -
> drivers/fpga/altera-cvp.c | 2 --
> include/linux/pci_ids.h | 2 ++
> 3 files changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/char/xillybus/xillybus_pcie.c b/drivers/char/xillybus/xillybus_pcie.c
> index dff2d1538164..05e5324f60bd 100644
> --- a/drivers/char/xillybus/xillybus_pcie.c
> +++ b/drivers/char/xillybus/xillybus_pcie.c
> @@ -24,7 +24,6 @@ MODULE_LICENSE("GPL v2");
>
> #define PCI_DEVICE_ID_XILLYBUS 0xebeb
>
> -#define PCI_VENDOR_ID_ALTERA 0x1172
> #define PCI_VENDOR_ID_ACTEL 0x11aa
> #define PCI_VENDOR_ID_LATTICE 0x1204
>
> diff --git a/drivers/fpga/altera-cvp.c b/drivers/fpga/altera-cvp.c
> index 00e73d28077c..77b04e4b3254 100644
> --- a/drivers/fpga/altera-cvp.c
> +++ b/drivers/fpga/altera-cvp.c
> @@ -384,8 +384,6 @@ static int altera_cvp_probe(struct pci_dev *pdev,
> const struct pci_device_id *dev_id);
> static void altera_cvp_remove(struct pci_dev *pdev);
>
> -#define PCI_VENDOR_ID_ALTERA 0x1172
> -
> static struct pci_device_id altera_cvp_id_tbl[] = {
> { PCI_VDEVICE(ALTERA, PCI_ANY_ID) },
> { }
> diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h
> index a6b30667a331..6a96a70fb462 100644
> --- a/include/linux/pci_ids.h
> +++ b/include/linux/pci_ids.h
> @@ -1561,6 +1561,8 @@
> #define PCI_DEVICE_ID_SERVERWORKS_CSB6LPC 0x0227
> #define PCI_DEVICE_ID_SERVERWORKS_HT1100LD 0x0408
>
> +#define PCI_VENDOR_ID_ALTERA 0x1172
> +
> #define PCI_VENDOR_ID_SBE 0x1176
> #define PCI_DEVICE_ID_SBE_WANXL100 0x0301
> #define PCI_DEVICE_ID_SBE_WANXL200 0x0302
>
next prev parent reply other threads:[~2018-02-28 11:59 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-02-28 8:44 [PATCH v2 0/2] MCB Update for v4.17 Johannes Thumshirn
2018-02-28 8:44 ` [PATCH v2 1/2] PCI: add vendor id for Altera Johannes Thumshirn
2018-02-28 11:59 ` Eli Billauer [this message]
2018-02-28 19:53 ` Bjorn Helgaas
2018-02-28 8:44 ` [PATCH v2 2/2] mcb: add Altera PCI ID to mcb-pci Johannes Thumshirn
2018-02-28 14:43 ` [PATCH v2 0/2] MCB Update for v4.17 Andy Shevchenko
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5A9699A7.6060801@gmail.com \
--to=eli.billauer@gmail.com \
--cc=agust@denx.de \
--cc=andy.shevchenko@gmail.com \
--cc=bhelgaas@google.com \
--cc=gregkh@linuxfoundation.org \
--cc=helgaas@kernel.org \
--cc=jthumshirn@suse.de \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=mmoese@suse.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.