On Mon, Jul 5, 2021 at 2:27 PM Thomas Monjalon <thomas@monjalon.net> wrote:

> Please make the title start with an imperative verb.
>
During the merge, modified the title to:
add shared TCAM region support.



> One more nit below.
>
> 13/06/2021 02:06, Ajit Khaparde:
> > From: Farah Smith <farah.smith@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.smith@broadcom.com>
> > Signed-off-by: Jay Ding <jay.ding@broadcom.com>
> > Signed-off-by: Randy Schacher <stuart.schacher@broadcom.com>
> > Signed-off-by: Venkat Duvvuru <venkatkumar.duvvuru@broadcom.com>
> > Reviewed-by: Ajit Khaparde <ajit.khaparde@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.
>
Removed.

Updated patch merged in dpdk-next-net-brcm. Thanks