Please make the title start with an imperative verb.
One more nit below.

13/06/2021 02:06, Ajit Khaparde:
> From: Farah Smith <farah.sm...@broadcom.com>
> 
> - switch to single slice management on Wh+
> - Support of shared session WC_TCAM_HIGH and WC_TCAM_LOW regions
> - Enable/disable using TF_TCAM_SHARED flag in tf_core.h
> - Fix empty session module DBs in the case that none are
>   allocated for a given module type
> 
> Signed-off-by: Farah Smith <farah.sm...@broadcom.com>
> Signed-off-by: Jay Ding <jay.d...@broadcom.com>
> Signed-off-by: Randy Schacher <stuart.schac...@broadcom.com>
> Signed-off-by: Venkat Duvvuru <venkatkumar.duvv...@broadcom.com>
> Reviewed-by: Ajit Khaparde <ajit.khapa...@broadcom.com>
[...]
> +++ b/drivers/net/bnxt/tf_core/tf_tcam_shared.c
> @@ -0,0 +1,744 @@
> +/* SPDX-License-Identifier: BSD-3-Clause
> + * Copyright(c) 2019-2021 Broadcom
> + * All rights reserved.
> + */
> +
> +#include <string.h>
> +#include <rte_common.h>
> +
> +#include "tf_tcam_shared.h"
> +#include "tf_tcam.h"
> +#include "tf_common.h"
> +#include "tf_util.h"
> +#include "tf_rm.h"
> +#include "tf_device.h"
> +#include "tfp.h"
> +#include "tf_session.h"
> +#include "tf_msg.h"
> +#include "bitalloc.h"
> +#include "tf_core.h"
> +#include "tf_rm.h"

tf_rm.h is already included.


Reply via email to