[PATCH v2 2/5] baseband/acc100: modify validation code for ACC101

Tom Rix trix at redhat.com
Sun May 8 15:07:20 CEST 2022


On 4/27/22 11:17 AM, Nicolas Chautru wrote:
> The validation requirement is different for the two
> devices.
>
> Signed-off-by: Nicolas Chautru <nicolas.chautru at intel.com>
> ---
>   drivers/baseband/acc100/rte_acc100_pmd.c | 40 ++++++++++++++++++++++----------
>   1 file changed, 28 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/baseband/acc100/rte_acc100_pmd.c b/drivers/baseband/acc100/rte_acc100_pmd.c
> index fca27ef..daf2ce0 100644
> --- a/drivers/baseband/acc100/rte_acc100_pmd.c
> +++ b/drivers/baseband/acc100/rte_acc100_pmd.c
> @@ -1293,6 +1293,14 @@
>   	return (q->d->device_variant == ACC100_VARIANT);
>   }
>   
> +#ifdef RTE_LIBRTE_BBDEV_DEBUG
> +static inline bool
> +validate_op_required(struct acc100_queue *q)

There isn't an #else case so this will fail to build.

This i believe could be another function in private data fops i 
suggested in the first patch.

Tom

> +{
> +	return is_acc100(q);
> +}
> +#endif
> +



More information about the dev mailing list