> -----Original Message-----
> From: David Marchand <[email protected]>
> Sent: Tuesday, November 4, 2025 6:38 PM
> To: [email protected]
> Cc: [email protected]; Gujjar, Abhinandan S <[email protected]>;
> Jerin Jacob <[email protected]>; Akhil Goyal <[email protected]>;
> Volodymyr Fialko <[email protected]>; Naga Harish K, S V
> <[email protected]>
> Subject: [PATCH] eventdev: fix build with clang 21
> 
> Fix 16-bits formatting issues reported by clang 21 on Fedora 43:
> 
> ../lib/eventdev/rte_event_crypto_adapter.c:1461:4: error:
>       format specifies type 'unsigned char' but the argument has type
>       'uint16_t' (aka 'unsigned short') [-Werror,-Wformat]
>  1459 |     RTE_EDEV_LOG_ERR("Failed to get adapter caps dev %" PRIu8
>       |     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>  1460 |             " cdev %" PRIu8, adapter->eventdev_id,
>       |             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>  1461 |             adapter->next_cdev_id);
>       |             ^~~~~~~~~~~~~~~~~~~~~~
> 
> ../lib/eventdev/rte_event_crypto_adapter.c:1592:46: error:
>       format specifies type 'unsigned char' but the argument has type
>       'uint16_t' (aka 'unsigned short') [-Werror,-Wformat]
>  1592 |     RTE_EDEV_LOG_ERR("Invalid dev_id=%" PRIu8, cdev_id);
>       |     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~
> 
> ../lib/eventdev/rte_event_crypto_adapter.c:1613:45: error:
>       format specifies type 'unsigned char' but the argument has type
>       'uint16_t' (aka 'unsigned short') [-Werror,-Wformat]
>  1612 |     RTE_EDEV_LOG_ERR("Event vectorization is not supported,"
>       |     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>  1613 |                      "dev %" PRIu8 " cdev %" PRIu8, dev_id, cdev_id);
>       |                      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~
> 
> Fixes: c1749bc5ee10 ("eventdev: introduce event cryptodev vector type")
> Fixes: 04ed18cd41f1 ("eventdev/crypto: support runtime set/get parameters")
> Cc: [email protected]
> 
> Signed-off-by: David Marchand <[email protected]>

Acked-by: [email protected]

> ---
>  lib/eventdev/rte_event_crypto_adapter.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/lib/eventdev/rte_event_crypto_adapter.c
> b/lib/eventdev/rte_event_crypto_adapter.c
> index b827a0ffd6..3f9cc7379a 100644
> --- a/lib/eventdev/rte_event_crypto_adapter.c
> +++ b/lib/eventdev/rte_event_crypto_adapter.c
> @@ -1457,7 +1457,7 @@ crypto_adapter_cap_check(struct
> event_crypto_adapter *adapter)
>                                               &caps);
>       if (ret) {
>               RTE_EDEV_LOG_ERR("Failed to get adapter caps dev %" PRIu8
> -                     " cdev %" PRIu8, adapter->eventdev_id,
> +                     " cdev %" PRIu16, adapter->eventdev_id,
>                       adapter->next_cdev_id);
>               return ret;
>       }
> @@ -1589,7 +1589,7 @@ rte_event_crypto_adapter_vector_limits_get(
>       RTE_EVENTDEV_VALID_DEVID_OR_ERR_RET(dev_id, -EINVAL);
> 
>       if (!rte_cryptodev_is_valid_dev(cdev_id)) {
> -             RTE_EDEV_LOG_ERR("Invalid dev_id=%" PRIu8, cdev_id);
> +             RTE_EDEV_LOG_ERR("Invalid dev_id=%" PRIu16, cdev_id);
>               return -EINVAL;
>       }
> 
> @@ -1610,7 +1610,7 @@ rte_event_crypto_adapter_vector_limits_get(
> 
>       if (!(cap & RTE_EVENT_CRYPTO_ADAPTER_CAP_EVENT_VECTOR)) {
>               RTE_EDEV_LOG_ERR("Event vectorization is not supported,"
> -                              "dev %" PRIu8 " cdev %" PRIu8, dev_id,
> cdev_id);
> +                              "dev %" PRIu8 " cdev %" PRIu16, dev_id,
> cdev_id);
>               return -ENOTSUP;
>       }
> 
> --
> 2.51.0

Reply via email to