[dpdk-dev] crypto/qat: make PCI device id struct const

Message ID 20170126140744.21626-1-ferruh.yigit@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Pablo de Lara Guarch
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel compilation success Compilation OK

Commit Message

Ferruh Yigit Jan. 26, 2017, 2:07 p.m. UTC
  Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
---
 drivers/crypto/qat/rte_qat_cryptodev.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Shreyansh Jain Jan. 27, 2017, 7:10 a.m. UTC | #1
On Thursday 26 January 2017 07:37 PM, Ferruh Yigit wrote:
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> ---
>  drivers/crypto/qat/rte_qat_cryptodev.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/crypto/qat/rte_qat_cryptodev.c b/drivers/crypto/qat/rte_qat_cryptodev.c
> index 9594067..a492dc0 100644
> --- a/drivers/crypto/qat/rte_qat_cryptodev.c
> +++ b/drivers/crypto/qat/rte_qat_cryptodev.c
> @@ -67,7 +67,7 @@ static struct rte_cryptodev_ops crypto_qat_ops = {
>   * The set of PCI devices this driver supports
>   */
>
> -static struct rte_pci_id pci_id_qat_map[] = {
> +static const struct rte_pci_id pci_id_qat_map[] = {
>  		{
>  			RTE_PCI_DEVICE(0x8086, 0x0443),
>  		},
>

Acked-by: Shreyansh Jain <shreyansh.jain@nxp.com>
  
De Lara Guarch, Pablo Jan. 27, 2017, 8:37 a.m. UTC | #2
> -----Original Message-----
> From: Shreyansh Jain [mailto:shreyansh.jain@nxp.com]
> Sent: Friday, January 27, 2017 7:11 AM
> To: Yigit, Ferruh
> Cc: dev@dpdk.org; Thomas Monjalon; De Lara Guarch, Pablo
> Subject: Re: [PATCH] crypto/qat: make PCI device id struct const
> 
> On Thursday 26 January 2017 07:37 PM, Ferruh Yigit wrote:
> > Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> > ---
> >  drivers/crypto/qat/rte_qat_cryptodev.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/crypto/qat/rte_qat_cryptodev.c
> b/drivers/crypto/qat/rte_qat_cryptodev.c
> > index 9594067..a492dc0 100644
> > --- a/drivers/crypto/qat/rte_qat_cryptodev.c
> > +++ b/drivers/crypto/qat/rte_qat_cryptodev.c
> > @@ -67,7 +67,7 @@ static struct rte_cryptodev_ops crypto_qat_ops = {
> >   * The set of PCI devices this driver supports
> >   */
> >
> > -static struct rte_pci_id pci_id_qat_map[] = {
> > +static const struct rte_pci_id pci_id_qat_map[] = {
> >  		{
> >  			RTE_PCI_DEVICE(0x8086, 0x0443),
> >  		},
> >
> 
> Acked-by: Shreyansh Jain <shreyansh.jain@nxp.com>

Applied to dpdk-next-crypto.
Thanks,

Pablo
  

Patch

diff --git a/drivers/crypto/qat/rte_qat_cryptodev.c b/drivers/crypto/qat/rte_qat_cryptodev.c
index 9594067..a492dc0 100644
--- a/drivers/crypto/qat/rte_qat_cryptodev.c
+++ b/drivers/crypto/qat/rte_qat_cryptodev.c
@@ -67,7 +67,7 @@  static struct rte_cryptodev_ops crypto_qat_ops = {
  * The set of PCI devices this driver supports
  */
 
-static struct rte_pci_id pci_id_qat_map[] = {
+static const struct rte_pci_id pci_id_qat_map[] = {
 		{
 			RTE_PCI_DEVICE(0x8086, 0x0443),
 		},