diff mbox series

crypto/mvsam: fix build

Message ID 20240515163415.268786-1-thomas@monjalon.net (mailing list archive)
State Accepted
Delegated to: akhil goyal
Headers
Series crypto/mvsam: fix build |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/loongarch-compilation success Compilation OK
ci/loongarch-unit-testing success Unit Testing PASS
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS
ci/iol-mellanox-Performance success Performance Testing PASS
ci/intel-Functional success Functional PASS
ci/github-robot: build success github build: passed
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-sample-apps-testing success Testing PASS
ci/iol-unit-amd64-testing success Testing PASS
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-abi-testing success Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-compile-amd64-testing success Testing PASS
ci/iol-unit-arm64-testing success Testing PASS
ci/iol-broadcom-Performance success Performance Testing PASS
ci/iol-compile-arm64-testing success Testing PASS

Commit Message

Thomas Monjalon May 15, 2024, 4:34 p.m. UTC 
  When moving the alignment attribute, a semicolon was missed.

Fixes: 27595cd83053 ("drivers: move alignment attribute on types for MSVC")

Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
---
 drivers/crypto/mvsam/rte_mrvl_pmd.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Thomas Monjalon May 16, 2024, 8:29 a.m. UTC | #1 
15/05/2024 18:34, Thomas Monjalon:
> When moving the alignment attribute, a semicolon was missed.
> 
> Fixes: 27595cd83053 ("drivers: move alignment attribute on types for MSVC")
> 
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> ---
> -}
> +};

Applied
Tyler Retzlaff May 16, 2024, 4:19 p.m. UTC | #2 
On Wed, May 15, 2024 at 06:34:14PM +0200, Thomas Monjalon wrote:
> When moving the alignment attribute, a semicolon was missed.
> 
> Fixes: 27595cd83053 ("drivers: move alignment attribute on types for MSVC")
> 
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> ---
>  drivers/crypto/mvsam/rte_mrvl_pmd.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/crypto/mvsam/rte_mrvl_pmd.c b/drivers/crypto/mvsam/rte_mrvl_pmd.c
> index 193b7fb931..a824719fb0 100644
> --- a/drivers/crypto/mvsam/rte_mrvl_pmd.c
> +++ b/drivers/crypto/mvsam/rte_mrvl_pmd.c
> @@ -60,7 +60,7 @@ struct __rte_aligned(32) cipher_params_mapping {
>  struct __rte_aligned(32) auth_params_mapping {
>  	enum algo_supported supported;  /**< On/off switch */
>  	enum sam_auth_alg   auth_alg;   /**< Auth algorithm */
> -}
> +};

already applied i see, but acking anyway.

sorry for the trouble.

Acked-by: Tyler Retzlaff <roretzla@linux.microsoft.com>

>  
>  /**
>   * Map of supported cipher algorithms.
> -- 
> 2.45.0
diff mbox series

Patch

diff --git a/drivers/crypto/mvsam/rte_mrvl_pmd.c b/drivers/crypto/mvsam/rte_mrvl_pmd.c
index 193b7fb931..a824719fb0 100644
--- a/drivers/crypto/mvsam/rte_mrvl_pmd.c
+++ b/drivers/crypto/mvsam/rte_mrvl_pmd.c
@@ -60,7 +60,7 @@  struct __rte_aligned(32) cipher_params_mapping {
 struct __rte_aligned(32) auth_params_mapping {
 	enum algo_supported supported;  /**< On/off switch */
 	enum sam_auth_alg   auth_alg;   /**< Auth algorithm */
-}
+};
 
 /**
  * Map of supported cipher algorithms.