From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0538F34EF1C for ; Tue, 5 May 2026 08:59:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=170.10.133.124 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777971560; cv=none; b=RTIfJnzh7SLRWEft/4ceabTYemESHOP/5KSC2Ij+xQvUmmBM0cnHaZzBWXfqPGxmrd3ghUEAr8Dz1TeFzcACnIhvwb/ykLxQ5AnhMWw2OvrpArMhdGaB3nFf5Bf8xSiueKd9JURBD0xRTIQe2k9jorYbNtyVDaNrlU+RrNjFp3w= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777971560; c=relaxed/simple; bh=CHfzsbBBWkt442uo449ObgtnuSTM8h9Xz+UeThmzFOg=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=IHqxThW5yP/iclnjENdqXuLuTnGyv5S1ShSLbKsSV4aalH8ZEehQDwTxunboiqfT9H97BuvFeC7bk0IDfTE/VtH8DIFNQuLWo4Gbd5QQYXXbNL76JZdUkMyB54RvvY/FYnSBsgtlcNzAGO2XQwbKE/GvtVyhisq1jt1DrVhV63E= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com; spf=pass smtp.mailfrom=redhat.com; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b=gAd/ep8C; dkim=pass (2048-bit key) header.d=redhat.com header.i=@redhat.com header.b=OGTbNZVA; arc=none smtp.client-ip=170.10.133.124 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=redhat.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="gAd/ep8C"; dkim=pass (2048-bit key) header.d=redhat.com header.i=@redhat.com header.b="OGTbNZVA" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1777971558; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=F0r85uYEBvFZwjw3nY1w+QDsZrIZzp0BkVyYOYWi2dY=; b=gAd/ep8CUNH7HQwq8zUjoU7MeyNPlL6R4xPaaiV3HAQq8fi/pAHNW7rhwHYc+jGxovA+uF giaTaxTFHFFks0NPoyadeXRkM5t3Z8png268IMxGRzWwHfZDDadJePFDaN8zzdCI9bwSiC O53+kdG7zTrA0dGBT9QwLROC9OzBrTo= Received: from mail-wm1-f70.google.com (mail-wm1-f70.google.com [209.85.128.70]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-266-xnHkxMOyPf68STxWGP_baA-1; Tue, 05 May 2026 04:59:17 -0400 X-MC-Unique: xnHkxMOyPf68STxWGP_baA-1 X-Mimecast-MFC-AGG-ID: xnHkxMOyPf68STxWGP_baA_1777971556 Received: by mail-wm1-f70.google.com with SMTP id 5b1f17b1804b1-488d1b5bca0so26184275e9.2 for ; Tue, 05 May 2026 01:59:16 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=google; t=1777971555; x=1778576355; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=F0r85uYEBvFZwjw3nY1w+QDsZrIZzp0BkVyYOYWi2dY=; b=OGTbNZVAbvXa0QCpzGF/gEd1atRftGjJVX9OniclUCDeDPa7vxtlr/RAgg7DuJkzgy GfD6vMwMjCk0T2G+e3EEavpbEsUJbRlgulmg2vfLouNsDwV054jTkRlGz9DPz0xTJQNa yXcciymXL4/9Qh2BeCA5Q/R8cGs0jMZhS/TbG+bs5KiZloB6194W/zwIfobTSgK0eVpG oaAJ4IOfr2paYH85vjhdOMb6EMrrOZ7kl6hgk3Ns7BRJbE5NtgQzAhVTBz1xKNcUP294 PGuF1M4rTT1G49ngdsSEz6BFaDCe1piheWYhB68UFz4Wnt8OkW0o/eaxpPuWav1zP0wD 3BJg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1777971555; x=1778576355; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=F0r85uYEBvFZwjw3nY1w+QDsZrIZzp0BkVyYOYWi2dY=; b=UKrZ3ewVIlUnWEh20t40vfcxjXl1SZyRdh4zfqlsC8ctt8669fvDYCJi1ohYjiJpYk pLYlcjlTDIWe4+GNT+4CKWiPWkolisSRWLwxNRTHWJ4ddFFTPuluTh+zoznR2c1KEjUp rJXnqj2z4WlrnMezQGepLgXv3qFdNUK0DwLeRVLOrSOT9N4MGA/tWX2gGClnIvA62LOi Tfsss1Lx8rsDYd/sS6PFWdLKv4EKzoqe7virrXhHTo7v3NOfr9IKE7qbxBhoIvUwiaUc 3137ESxboAFDgjNY0kE299GsKYXZMiBnNl2Q7klv95r9Q62sCsJzTCKQS+fQJnychGuc cqRw== X-Forwarded-Encrypted: i=1; AFNElJ88yrXCkTl5K3PPBv3m8r+sI1nc47wGVwkjUZUM3JVR3WQVtX/H4dGvpRv23cbOfo8DZWAs+15vLwrVKTM=@vger.kernel.org X-Gm-Message-State: AOJu0Yw4Hbo+pgB31tcF8iykHFlT+8pNM5aGynttAMGyMGx6Oq0cWuJ6 kkNFZ7j/Vq875fwyes38Qt4RBdMYNdTg4sL5swBnqz2WQIUASiBu64I1BmQIDPphxpOKQg5Gk2w FLJ7Lm4z6aJTo9T7sDom+/KT0n74fILo2UVj/D1Ksbtg1hR3WHNWz/AMwWJOhRsfh7Q== X-Gm-Gg: AeBDiesHFobQet+36VTePZsFKDNdGot5mEUYUKZhKbSIbFl7n00xPEqAtJtw3AaVcqT IrkC999qk4+9YUjXjD38yD6TQ28vcqc4T5qM+mIcN8S7nigQLqnSRerWq7qxYpIWb4BVrUT2Qkg 6uvqBGf6wiV8hQfTnAFgZ3Q5xsGdhUr/GgZKQGMcul0qmolrYm7gFsziNEYHz++dyge5H78mZiJ Ml3Qje3xbqdkZTjd+n9iC73as/DscHiBp6MoB6kNGiQIekdiwm0roFE/rKikcPv6nfFPlIxJdPc 5r599pJcBI32DXt0oj8YV8H8RrdyvXh+nKslt2AkkLUbBKTuPheoii89bknZR0DBAOG3CAeO1hL GXabEDcSLTRuFDXYZR2bVusmqd+/Pg0KkcoRn3TIKC2IH3Suj2rO3liOnjA7Cf1mBMvQ= X-Received: by 2002:a05:600c:5308:b0:48a:5565:ec3d with SMTP id 5b1f17b1804b1-48d03b401f5mr161071205e9.22.1777971555615; Tue, 05 May 2026 01:59:15 -0700 (PDT) X-Received: by 2002:a05:600c:5308:b0:48a:5565:ec3d with SMTP id 5b1f17b1804b1-48d03b401f5mr161070775e9.22.1777971555194; Tue, 05 May 2026 01:59:15 -0700 (PDT) Received: from [192.168.88.32] ([212.105.155.47]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-48d184ed3b0sm12329145e9.34.2026.05.05.01.59.13 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 05 May 2026 01:59:14 -0700 (PDT) Message-ID: <6324783e-b5c8-462d-8cf7-f6cc9c01ea89@redhat.com> Date: Tue, 5 May 2026 10:59:12 +0200 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v5 net-next 04/15] net: enetc: add basic operations to the FDB table To: Wei Fang , claudiu.manoil@nxp.com, vladimir.oltean@nxp.com, xiaoning.wang@nxp.com, andrew+netdev@lunn.ch, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, f.fainelli@gmail.com, frank.li@nxp.com, chleroy@kernel.org, horms@kernel.org, linux@armlinux.org.uk Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-arm-kernel@lists.infradead.org, imx@lists.linux.dev References: <20260430024945.3413973-1-wei.fang@nxp.com> <20260430024945.3413973-5-wei.fang@nxp.com> Content-Language: en-US From: Paolo Abeni In-Reply-To: <20260430024945.3413973-5-wei.fang@nxp.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 4/30/26 4:49 AM, Wei Fang wrote: > The FDB table is used for MAC learning lookups and MAC forwarding lookups. > Each table entry includes information such as a FID and MAC address that > may be unicast or multicast and a forwarding destination field containing > a port bitmap identifying the associated port(s) with the MAC address. > FDB table entries can be static or dynamic. Static entries are added from > software whereby dynamic entries are added either by software or by the > hardware as MAC addresses are learned in the datapath. > > The FDB table can only be managed by the command BD ring using table > management protocol version 2.0. Table management command operations Add, > Delete, Update and Query are supported. And the FDB table supports three > access methods: Entry ID, Exact Match Key Element and Search. This patch > adds the following basic supports to the FDB table. > > ntmp_fdbt_update_entry() - update the configuration element data of a > specified FDB entry > > ntmp_fdbt_delete_entry() - delete a specified FDB entry > > ntmp_fdbt_add_entry() - add an entry into the FDB table > > ntmp_fdbt_search_port_entry() - Search the FDB entry on the specified > port based on RESUME_ENTRY_ID. > > Signed-off-by: Wei Fang > --- > drivers/net/ethernet/freescale/enetc/ntmp.c | 203 +++++++++++++++++- > .../ethernet/freescale/enetc/ntmp_private.h | 61 +++++- > include/linux/fsl/ntmp.h | 44 +++- > 3 files changed, 305 insertions(+), 3 deletions(-) > > diff --git a/drivers/net/ethernet/freescale/enetc/ntmp.c b/drivers/net/ethernet/freescale/enetc/ntmp.c > index c94a928622fd..4ed8d783a9a2 100644 > --- a/drivers/net/ethernet/freescale/enetc/ntmp.c > +++ b/drivers/net/ethernet/freescale/enetc/ntmp.c > @@ -1,7 +1,7 @@ > // SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause) > /* > * NETC NTMP (NETC Table Management Protocol) 2.0 Library > - * Copyright 2025 NXP > + * Copyright 2025-2026 NXP > */ > > #include > @@ -21,11 +21,15 @@ > /* Define NTMP Table ID */ > #define NTMP_MAFT_ID 1 > #define NTMP_RSST_ID 3 > +#define NTMP_FDBT_ID 15 > > /* Generic Update Actions for most tables */ > #define NTMP_GEN_UA_CFGEU BIT(0) > #define NTMP_GEN_UA_STSEU BIT(1) > > +/* Query Action: 0: Full query, 1: Only query entry ID */ > +#define NTMP_QA_ENTRY_ID 1 Sashiko noted that the above comments looks inconsistent with the update code, where NTMP_QA_ENTRY_ID apparently uses a full query, and 0 just the entry ID. If you have to repost for other reasons, please fix this. Note that you should reply on the ML to sashiko reviews ruling out invalid comments. Thanks, Paolo