netdev - Re: [PATCH net-next,V2] Add LAN9352 Ethernet Driver

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: <20160216221536.GB6033@lunn.ch>
Date:	Tue, 16 Feb 2016 23:15:36 +0100
From:	Andrew Lunn <andrew@...n.ch>
To:	Bryan.Whitehead@...rochip.com
Cc:	davem@...emloft.net, f.fainelli@...il.com, netdev@...r.kernel.org
Subject: Re: [PATCH net-next,V2] Add LAN9352 Ethernet Driver

> I just spoke with my manager, and we would like to change the target
> device from LAN9352 to LAN9250. The LAN9250 is the same as the
> LAN9352 but without the switch. It has one mac and one phy.

It is not so easy to get an overview from the website, but it looks
like:

LAN9250 - one port MAC/PHY
LAN9352 - 10/100 2-Port Managed Ethernet Switch
LAN9353 - 10/100 3-Port Managed Ethernet with Dual RMII or Single MII/RMII/Turbo MII
LAN9354 - 10/100 3-Port Managed Ethernet Switch with Single RMII
LAN9355 - 10/100 3-Port Managed Ethernet Switch with Dual MII/RMII/Turbo MII

So i get the feeling this product line is for switches, and the
LAN9250 is an oddball in the series.

You are already 1/2 way to a DSA driver, since you have a MAC
driver. So i agree with David, do it right and add a simple DSA
driver.

	Andrew