On 7/12/2021 7:49 PM, ramesh.thomas(a)intel.com wrote:
From: Ramesh Thomas <ramesh.thomas(a)intel.com>
A default value will be set by driver for hw versions < 0x200
Signed-off-by: Ramesh Thomas <ramesh.thomas(a)intel.com>
---
test/libaccfg.c | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/test/libaccfg.c b/test/libaccfg.c
index c37b71c..1d68781 100644
--- a/test/libaccfg.c
+++ b/test/libaccfg.c
@@ -186,6 +186,10 @@ static int config_group(struct accfg_ctx *ctx, struct accfg_device
*device,
group_param->tokens_allowed));
SET_ERR(rc, accfg_group_set_use_token_limit(group,
group_param->use_token_limit));
+
+ if (accfg_device_get_version(device) < 0x200)
Define magic number. Otherwise
it looks fine.
> + return rc;
> +
> SET_ERR(rc, accfg_group_set_traffic_class_a(group,
> group_param->traffic_class_a));
> SET_ERR(rc, accfg_group_set_traffic_class_b(group,
> @@ -217,6 +221,9 @@ static int check_group(struct accfg_ctx *ctx, struct accfg_device
*device,
> return -EINVAL;
> }
>
> + if (accfg_device_get_version(device) < 0x200)
> + return 0;
> +
> if (group_param->traffic_class_a !=
> accfg_group_get_traffic_class_a(group)) {
> fprintf(stderr, "%s failed on traffic_class_a\n", __func__);