On 9/18/2017 4:17 PM, Ajit Khaparde wrote:
> This patch adds support for flow validate/create/destroy/flush ops.
> Also update doc/guides/nics/features/bnxt.ini to indicate this.
> 
> Signed-off-by: Ajit Khaparde <ajit.khapa...@broadcom.com>

<...>

> +int bnxt_hwrm_set_ntuple_filter(struct bnxt *bp,
> +                      uint16_t dst_id,
> +                      struct bnxt_filter_info *filter)
> +{
> +     int rc = 0;
> +     struct hwrm_cfa_ntuple_filter_alloc_input req = {.req_type = 0 };
> +     struct hwrm_cfa_ntuple_filter_alloc_output *resp =
> +                                             bp->hwrm_cmd_resp_addr;
> +     uint32_t enables = 0;
> +
> +     if (filter->fw_ntuple_filter_id != UINT64_MAX) {
> +             RTE_LOG(ERR, PMD,
> +                     "NTUPLE id %lx\n", filter->fw_ntuple_filter_id);

This breaks the 32bit build:

.../drivers/net/bnxt/bnxt_hwrm.c:3270:50: error: format ‘%lx’ expects
argument of type ‘long unsigned int’, but argument 4 has type ‘uint64_t
{aka long long unsigned int}’ [-Werror=format=]
    "NTUPLE id %lx\n", filter->fw_ntuple_filter_id);
                       ~~~~~~~~~                  ^

<...>

Reply via email to