netdev - Re: [PATCH v3 net-next 2/2] net: airoha: Introduce ethernet support for EN7581 SoC

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <4a39fa50-cffc-4f0c-a442-b666b024ba34@lunn.ch>
Date: Mon, 24 Jun 2024 17:45:20 +0200
From: Andrew Lunn <andrew@...n.ch>
To: Benjamin Larsson <benjamin.larsson@...exis.eu>
Cc: Lorenzo Bianconi <lorenzo@...nel.org>, netdev@...r.kernel.org,
	nbd@....name, lorenzo.bianconi83@...il.com, davem@...emloft.net,
	edumazet@...gle.com, kuba@...nel.org, pabeni@...hat.com,
	conor@...nel.org, linux-arm-kernel@...ts.infradead.org,
	robh+dt@...nel.org, krzysztof.kozlowski+dt@...aro.org,
	conor+dt@...nel.org, devicetree@...r.kernel.org,
	catalin.marinas@....com, will@...nel.org, upstream@...oha.com,
	angelogioacchino.delregno@...labora.com, rkannoth@...vell.com,
	sgoutham@...vell.com
Subject: Re: [PATCH v3 net-next 2/2] net: airoha: Introduce ethernet support
 for EN7581 SoC

On Mon, Jun 24, 2024 at 01:01:44AM +0200, Benjamin Larsson wrote:
> Hi,
> > Code like this is making me wounder about the split between MAC
> > driver, DSA driver and DSA tag driver. Or if it should actually be a
> > pure switchdev driver?
> > 
> > If there some open architecture documentation for this device?
> > 
> > What are these ports about?
> > 
> > > +static int airoha_dev_open(struct net_device *dev)
> > > +{
> > > +	struct airoha_eth *eth = netdev_priv(dev);
> > > +	int err;
> > > +
> > > +	if (netdev_uses_dsa(dev))
> > > +		airoha_fe_set(eth, REG_GDM1_INGRESS_CFG, GDM1_STAG_EN_MASK);
> > > +	else
> > > +		airoha_fe_clear(eth, REG_GDM1_INGRESS_CFG, GDM1_STAG_EN_MASK);
> > Does that imply both instances of the GMAC are not connected to the
> > switch? Can one be used with a PHY?
> > 
> > 	Andrew
> 
> https://mirror2.openwrt.org/docs/MT7981B_Wi-Fi6_Platform_Datasheet_Open_V1.0.pdf
> 
> page 107 (text for 9.1.1 is relevant but not a complete match). In the
> EN7581 case there is a 5 port switch in the place of GMAC1 (one switch port
> is connected to GDM1).

The typical DSA architecture is that the SoC MAC is connected to a
switch MAC port. You say here, the switch is directly connected to the
GGM1. So there is no GMAC involved? If there is no MAC, you don't need
a MAC driver.

It seems more likely there is a GMAC, and the SGMII interface, or
something similar is connected to the switch?

	Andrew