From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f53.google.com (mail-wr1-f53.google.com [209.85.221.53]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8469E3090CE for ; Fri, 21 Nov 2025 20:48:39 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.53 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763758121; cv=none; b=md/n9VAZ5b6UXs5pxlh+Q85U8nrXGaergTi+M1pB4FXjS9hUDRrxYdEfkl+QT67RJaaYFHe22YEH2PcWMGq/p0WHzPYhruV6hKe4mDtT0UJJFw3tIm2rI/K+5NeLnhK34Ilxye7IZnxBoz/1yxb4s/e2rXZdqSVbU0+bWplPz5s= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763758121; c=relaxed/simple; bh=+4fVew2+4u+WlL4vnVXwzj+uge05xava6pxne1RN0Mc=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=PofL6s8qpw8LTzg/GLiOKNpBVvDZng0ZEEd442wAShFUKQ+c3i05pB0Bq/pAofQ5mPBjvxWi8yAuG0729AEWWkXLYDEuWtE3bs5JfWhS8Nzj/YqHzUiilYMaFcxxqVXMY2eLSdBITmrml2MnQWtfZLjyfvZ7jEZy+CaYt69uPsc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=EDJjMLA4; arc=none smtp.client-ip=209.85.221.53 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="EDJjMLA4" Received: by mail-wr1-f53.google.com with SMTP id ffacd0b85a97d-42b55ba1e62so188057f8f.2 for ; Fri, 21 Nov 2025 12:48:39 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1763758118; x=1764362918; darn=vger.kernel.org; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date:from:to :cc:subject:date:message-id:reply-to; bh=pGOCAupp/j1gbqj8RASAzeEB0W1VfiDIcyH9885Q3ps=; b=EDJjMLA40wVDM9W+pER36MbGMbdF8dWL/IxR5thS/fbBSc8sfVvnYAUeupvkEEk86j bsuNP2hDxyc/siYw74fJ8xeZl6kj9uEWtFHbowveg8SpMPvGgHocIGeneRqLBCD1wedn 6NZIcKfiKY08awJmW1FRmITisO49Ta/TllEIgdpBQhgSOhbSPpgrAByO301lbbzSHAhN n1wX6McoGhcjw3xodRaS6Wi0rOhmupXqoHwKYBJjLSYYpkeVLv0xwFI01g3Q79I0I9B4 akDigIO4ItwuY3O6phvpS/2NwSdz5wT4zl2GZjo2cCzZZHE+QkjvFXZ78HJKR/Y5cdVN jD4A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1763758118; x=1764362918; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=pGOCAupp/j1gbqj8RASAzeEB0W1VfiDIcyH9885Q3ps=; b=c77cJOh3n3gXlhR6T4bK9o4jVi49N2e0iZ2GX7CpRvHJAXvFfvonDq/V9zOJZrVLQW eLjqHiBcCrMRdSpZHFj+3IpL4cAPnka/HjbxqYn6FxP6k5hVOncP2kWNB5+R3JNff52V 4N6GKOLVpgF6AC4LQjux1PRjzQLrk93ectU9TPpFCzfdr57J1EDLsJCX5sD+JtmmQHaq 46+6z174r8tGeY+A1r9Tv0i4D/8aBN0D938IFK046D1MdyRlCIN1iZS641dVXsf342ri er3HYin/FvbypVZi1EWajzF/2lgBsWcV+M9Aamw1cMLBjcF+5dMLD6V1dxQBhHcHKPg/ czQA== X-Forwarded-Encrypted: i=1; AJvYcCU+kcWUMyEJXlAjJwcWpOKbOKa/X8Cz1uUQyDK1SujKAPu7+HCkN9t5CAOD9+IdypA+p4mF+U6QIMc4@vger.kernel.org X-Gm-Message-State: AOJu0YzTuCX3fut3zQEx2DqgWxT6Zyx0Kg6MitPjCIdkXBTIa77IvFgC QG34hKoQ+pg1mOylHGg4ObifLgO3GzoZq1tXrBcxTrNVpkvliZvkv/3a X-Gm-Gg: ASbGnctNLbrMW7aD4HW7tC9IRDTjQwHTgNbX3Ys1y2NKUe7l97g6Y5TLL6SnDTOh3gs arIyxx6+IB7vQSW+XtBlclDfaT3QO1pRWvKzsLpDQCD6dOP6JFj3dZxAPfjm/eX5mXa5NghI/8O fOOWwVCFcKT/ypEtq5Zrb+nWaWMQjMa1T3bCsk11n6uPzeafP4IcAo0vM6L5yLaQtHdNxLy+SPR 3oUE2Tj3Zp2Qfy7TnETH4L1era13laT1b82n2z5poCikHCxoJTP2PWAcBcoP4zJWe/60Mp4ecRa rLn+G8BzkYIQS962WD+vbCHo+Fkw5ItYlYCZvOQFybMy+Uvl3yJxSkTtUE3ZzUW1dk1FFbWuWqx TkgSES0oqYF1U/js6YjkQycWpdLBIC7pcV/Ncf3tcKHgxlrHK1SsYVLHg0jq+TJUsHK9e/hjlRl DefiM= X-Google-Smtp-Source: AGHT+IEiC/MfRTvQT3tA1MrA+Mzzqzp5R6aAOe9/0it1znq1L7sttK4oGWLscrmdtQKkr6uwAxSe9g== X-Received: by 2002:a05:600c:3b9c:b0:475:d7b8:8505 with SMTP id 5b1f17b1804b1-477c01f4ab6mr23231215e9.7.1763758117718; Fri, 21 Nov 2025 12:48:37 -0800 (PST) Received: from skbuf ([2a02:2f04:d106:d600:b19f:2efa:e88a:a382]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-477bf227ae2sm60962265e9.9.2025.11.21.12.48.35 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 21 Nov 2025 12:48:37 -0800 (PST) Date: Fri, 21 Nov 2025 22:48:33 +0200 From: Vladimir Oltean To: "Lad, Prabhakar" Cc: =?utf-8?B?Q2zDqW1lbnQgTMOpZ2Vy?= , Andrew Lunn , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Simon Horman , Philipp Zabel , Russell King , Geert Uytterhoeven , Magnus Damm , linux-renesas-soc@vger.kernel.org, netdev@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Biju Das , Fabrizio Castro , Lad Prabhakar Subject: Re: [PATCH net-next 02/11] net: dsa: tag_rzn1_a5psw: Add RZ/T2H ETHSW tag protocol and register ethsw tag driver Message-ID: <20251121204833.hu4jfq6bx2kdamd3@skbuf> References: <20251121113553.2955854-1-prabhakar.mahadev-lad.rj@bp.renesas.com> <20251121113553.2955854-3-prabhakar.mahadev-lad.rj@bp.renesas.com> <20251121192700.3eg5h2eqk7bruxeu@skbuf> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: On Fri, Nov 21, 2025 at 08:29:04PM +0000, Lad, Prabhakar wrote: > On Fri, Nov 21, 2025 at 7:27 PM Vladimir Oltean wrote: > > > > On Fri, Nov 21, 2025 at 11:35:28AM +0000, Prabhakar wrote: > > > From: Lad Prabhakar > > > > > > Add an explicit tag protocol for the RZ/T2H ETHSW and register a separate > > > ethsw tag driver so the existing A5PSW tag implementation can be reused > > > for RZ/T2H without code duplication. > > > > > > The ETHSW IP on RZ/T2H shares substantial commonality with the A5PSW IP on > > > RZ/N1, and the current tag driver does not touch the register fields that > > > differ between the two blocks. > > > > Tagging protocol drivers are specifically written to not deal with > > register fields. I would like a clarification that this is a phrasing > > mistake and you mean the packet header fields that differ between the > > ETHSW and the A5PSW tag format. > > > Unlike the other drivers, tagging drivers don't have compatible > strings to match against. For the ETHSW IP, the current driver is > reused as-is. My intention with the comment was simply to point out > that, if an issue ever arises that requires us to split the paths, we > can future-proof things by using DSA_TAG_PROTO_* identifiers. The tagging protocol's name uniquely defines the layout of the DSA header and general interaction procedure required on RX and TX (for more complex things such as PTP). It doesn't have to be further namespaced by its users just because. In other words, two switch drivers using the same tagging protocol with the same name is fine (even if due to a lack of imagination, the tagging protocol's name comes just from its first user), and introducing a new name for it would be unnecessary. For example, felix_vsc9959.c, a switch different from ocelot_ext.c, uses DSA_TAG_PROTO_OCELOT because the protocol is identical. The exception would be when there exist packet headers which have different layouts - then irrespective of whether those fields are currently used or not, we should register a new driver. This is the only thing that matters. I thought that you were saying that such differences exist, but after your second reply, it seems not?