DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] doc/fix: add CCM to the QAT feature list
@ 2018-09-11 17:23 Tomasz Cel
  2018-09-13 13:41 ` Kovacevic, Marko
  2018-09-19  6:27 ` [dpdk-dev] [PATCH v2] doc/crypto: fix missing CCM to " Tomasz Cel
  0 siblings, 2 replies; 6+ messages in thread
From: Tomasz Cel @ 2018-09-11 17:23 UTC (permalink / raw)
  To: dev; +Cc: Tomasz Cel

Signed-off-by: Tomasz Cel <tomaszx.cel@intel.com>
---
 doc/guides/cryptodevs/features/qat.ini | 3 +++
 doc/guides/cryptodevs/qat.rst          | 1 +
 2 files changed, 4 insertions(+)

diff --git a/doc/guides/cryptodevs/features/qat.ini b/doc/guides/cryptodevs/features/qat.ini
index 29d865e..a37a1a1 100644
--- a/doc/guides/cryptodevs/features/qat.ini
+++ b/doc/guides/cryptodevs/features/qat.ini
@@ -56,3 +56,6 @@ ZUC EIA3     = Y
 AES GCM (128) = Y
 AES GCM (192) = Y
 AES GCM (256) = Y
+AES CCM (128) = Y
+AES CCM (256) = Y
+AES CCM (192) = Y
diff --git a/doc/guides/cryptodevs/qat.rst b/doc/guides/cryptodevs/qat.rst
index af10799..b09624f 100644
--- a/doc/guides/cryptodevs/qat.rst
+++ b/doc/guides/cryptodevs/qat.rst
@@ -66,6 +66,7 @@ Hash algorithms:
 Supported AEAD algorithms:
 
 * ``RTE_CRYPTO_AEAD_AES_GCM``
+* ``RTE_CRYPTO_AEAD_AES_CCM``
 
 
 Limitations
-- 
2.7.4

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [dpdk-dev] [PATCH] doc/fix: add CCM to the QAT feature list
  2018-09-11 17:23 [dpdk-dev] [PATCH] doc/fix: add CCM to the QAT feature list Tomasz Cel
@ 2018-09-13 13:41 ` Kovacevic, Marko
  2018-09-19  6:27 ` [dpdk-dev] [PATCH v2] doc/crypto: fix missing CCM to " Tomasz Cel
  1 sibling, 0 replies; 6+ messages in thread
From: Kovacevic, Marko @ 2018-09-13 13:41 UTC (permalink / raw)
  To: Cel, TomaszX, dev; +Cc: Cel, TomaszX

> Signed-off-by: Tomasz Cel <tomaszx.cel@intel.com>
> ---
>  doc/guides/cryptodevs/features/qat.ini | 3 +++
>  doc/guides/cryptodevs/qat.rst          | 1 +
>  2 files changed, 4 insertions(+)


Hi Tomasz,

 I think the patch is ok there is just a few things as this is a fix you need to add a fixes
tag into the commit message here is a reference to the guides for it.
https://doc.dpdk.org/guides/contributing/patches.html?highlight=commit%20message

Excepted output:
-------------------------------------
doc: fix some parameter description

Update the docs, fixing description of some parameter.

Fixes: abcdefgh1234 ("doc: add some parameter")
Cc: author@example.com

Signed-off-by: Alex Smith <alex.smith@example.com>
-----------------------------------------------------------------------

Other thing is try and keep the sizes in order

 +AES CCM (128) = Y   > +AES CCM (128) = Y   
 +AES CCM (256) = Y   > +AES CCM (192) = Y
 +AES CCM (192) = Y   > +AES CCM (256) = Y



Marko K.


 

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [dpdk-dev] [PATCH v2] doc/crypto: fix missing CCM to QAT feature list
  2018-09-11 17:23 [dpdk-dev] [PATCH] doc/fix: add CCM to the QAT feature list Tomasz Cel
  2018-09-13 13:41 ` Kovacevic, Marko
@ 2018-09-19  6:27 ` Tomasz Cel
  2018-09-19  7:28   ` Trahe, Fiona
                     ` (2 more replies)
  1 sibling, 3 replies; 6+ messages in thread
From: Tomasz Cel @ 2018-09-19  6:27 UTC (permalink / raw)
  To: dev; +Cc: stable, akhil.goyal, marko.kovacevic, Tomasz Cel

Update the QAT documentation to show that it supports CCM.

Fixes: ab56c4d9ed9a ("crypto/qat: support AES-CCM")

Signed-off-by: Tomasz Cel <tomaszx.cel@intel.com>
---
 doc/guides/cryptodevs/features/qat.ini | 3 +++
 doc/guides/cryptodevs/qat.rst          | 1 +
 2 files changed, 4 insertions(+)

diff --git a/doc/guides/cryptodevs/features/qat.ini b/doc/guides/cryptodevs/features/qat.ini
index 29d865e..220291b 100644
--- a/doc/guides/cryptodevs/features/qat.ini
+++ b/doc/guides/cryptodevs/features/qat.ini
@@ -56,3 +56,6 @@ ZUC EIA3     = Y
 AES GCM (128) = Y
 AES GCM (192) = Y
 AES GCM (256) = Y
+AES CCM (128) = Y
+AES CCM (192) = Y
+AES CCM (256) = Y
diff --git a/doc/guides/cryptodevs/qat.rst b/doc/guides/cryptodevs/qat.rst
index af10799..b09624f 100644
--- a/doc/guides/cryptodevs/qat.rst
+++ b/doc/guides/cryptodevs/qat.rst
@@ -66,6 +66,7 @@ Hash algorithms:
 Supported AEAD algorithms:
 
 * ``RTE_CRYPTO_AEAD_AES_GCM``
+* ``RTE_CRYPTO_AEAD_AES_CCM``
 
 
 Limitations
-- 
2.7.4

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [dpdk-dev] [PATCH v2] doc/crypto: fix missing CCM to QAT feature list
  2018-09-19  6:27 ` [dpdk-dev] [PATCH v2] doc/crypto: fix missing CCM to " Tomasz Cel
@ 2018-09-19  7:28   ` Trahe, Fiona
  2018-09-19  8:09   ` Kovacevic, Marko
  2018-09-26 12:26   ` Akhil Goyal
  2 siblings, 0 replies; 6+ messages in thread
From: Trahe, Fiona @ 2018-09-19  7:28 UTC (permalink / raw)
  To: Cel, TomaszX, dev; +Cc: stable, akhil.goyal, Kovacevic, Marko, Cel, TomaszX



> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Tomasz Cel
> Sent: Wednesday, September 19, 2018 7:27 AM
> To: dev@dpdk.org
> Cc: stable@dpdk.org; akhil.goyal@nxp.com; Kovacevic, Marko <marko.kovacevic@intel.com>; Cel,
> TomaszX <tomaszx.cel@intel.com>
> Subject: [dpdk-dev] [PATCH v2] doc/crypto: fix missing CCM to QAT feature list
> 
> Update the QAT documentation to show that it supports CCM.
> 
> Fixes: ab56c4d9ed9a ("crypto/qat: support AES-CCM")
> 
> Signed-off-by: Tomasz Cel <tomaszx.cel@intel.com>
Acked-by: Fiona Trahe <fiona.trahe@intel.com>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [dpdk-dev] [PATCH v2] doc/crypto: fix missing CCM to QAT feature list
  2018-09-19  6:27 ` [dpdk-dev] [PATCH v2] doc/crypto: fix missing CCM to " Tomasz Cel
  2018-09-19  7:28   ` Trahe, Fiona
@ 2018-09-19  8:09   ` Kovacevic, Marko
  2018-09-26 12:26   ` Akhil Goyal
  2 siblings, 0 replies; 6+ messages in thread
From: Kovacevic, Marko @ 2018-09-19  8:09 UTC (permalink / raw)
  To: Cel, TomaszX, dev; +Cc: stable, akhil.goyal

> Update the QAT documentation to show that it supports CCM.
> 
> Fixes: ab56c4d9ed9a ("crypto/qat: support AES-CCM")
> 
> Signed-off-by: Tomasz Cel <tomaszx.cel@intel.com>
> ---
>  doc/guides/cryptodevs/features/qat.ini | 3 +++
>  doc/guides/cryptodevs/qat.rst          | 1 +
>  2 files changed, 4 insertions(+)

Acked-by: Marko Kovacevic <marko.kovacevic@intel.com>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [dpdk-dev] [PATCH v2] doc/crypto: fix missing CCM to QAT feature list
  2018-09-19  6:27 ` [dpdk-dev] [PATCH v2] doc/crypto: fix missing CCM to " Tomasz Cel
  2018-09-19  7:28   ` Trahe, Fiona
  2018-09-19  8:09   ` Kovacevic, Marko
@ 2018-09-26 12:26   ` Akhil Goyal
  2 siblings, 0 replies; 6+ messages in thread
From: Akhil Goyal @ 2018-09-26 12:26 UTC (permalink / raw)
  To: Tomasz Cel, dev; +Cc: stable, marko.kovacevic



On 9/19/2018 11:57 AM, Tomasz Cel wrote:
> Update the QAT documentation to show that it supports CCM.
>
> Fixes: ab56c4d9ed9a ("crypto/qat: support AES-CCM")
>
> Signed-off-by: Tomasz Cel <tomaszx.cel@intel.com>
> ---
>   doc/guides/cryptodevs/features/qat.ini | 3 +++
>   doc/guides/cryptodevs/qat.rst          | 1 +
>   2 files changed, 4 insertions(+)
>
Applied to dpdk-next-crypto
cc: stable@dpdk.org


Thanks

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2018-09-26 12:26 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-09-11 17:23 [dpdk-dev] [PATCH] doc/fix: add CCM to the QAT feature list Tomasz Cel
2018-09-13 13:41 ` Kovacevic, Marko
2018-09-19  6:27 ` [dpdk-dev] [PATCH v2] doc/crypto: fix missing CCM to " Tomasz Cel
2018-09-19  7:28   ` Trahe, Fiona
2018-09-19  8:09   ` Kovacevic, Marko
2018-09-26 12:26   ` Akhil Goyal

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).