Re: [net-next,v5,05/13] octeontx2-af: npc: cn20k: Allocate default MCAM indexes

[Ratheesh Kannoth]

On 2026-01-28 at 07:59:21, Jakub Kicinski (kuba@xxxxxxxxxx) wrote:
> > +chk_sanity:
> > +	dev_dbg(rvu->dev,
> > +		"%s: Default index for pcifunc=%#x, bcast=%u mcast=%u promise=%u ucast=%u cnt=%u\n",
>                                                                        ^^^^^^^
>
> Should "promise" be "promisc" here? This appears to be a typo in the debug
> message.
ACK.

>
>
> > +/**
> > + * enum npc_dft_rule_id - Default rule type
> > + *
> > + * Mcam default rule type.
> > + *
> > + * @NPC_DFT_RULE_START_ID:	Not used
> > + * @NPC_DFT_RULE_PROMISC_ID:	promiscuous rule
> > + * @NPC_DFT_RULE_MCAST_ID:	promiscuous rule
>                                 ^^^^^^^^^^^^^^^^

>
> Should this say "multicast rule" instead of "promiscuous rule"? It appears
>
ACK.

>
> > + * @NPC_DFT_RULE_BCAST_ID:	broadcast rule
> > + * @NPC_DFT_RULE_UCAST_ID:	unicast rule
> > + * @NPC_DFT_RULE_MAX_ID:	Maximum index.
> > + */
> --
> pw-bot: cr