Re: [PATCH v1 4/4] net: dsa: add support for Atheros AR9331 build-in switch

From: Oleksij Rempel
Date: Fri Oct 18 2019 - 01:08:37 EST

On Thu, Oct 17, 2019 at 11:35:48AM -0700, Florian Fainelli wrote:
> On 10/13/2019 11:15 PM, Oleksij Rempel wrote:
> > Provide basic support for Atheros AR9331 build-in switch. So far it
> > works as port multiplexer without any hardware offloading support.
> I glanced through the functional parts of the code, and it looks pretty
> straight forward, since there is no offloading done so far, do you plan
> on adding bridge offload eventually if nothing more?

Currently not. There are following reasons:
- I do it for the Freifunk project. It is currently not clear, what
functionality has higher priority.
- there are not many ar9331 based devices with enough RAM to run any
thing modern. There is even less devices using more then one switch
- IPv6 support is important for this project, but old Atheros switches have some
known issues with IPv6 packages in hardware bridge mode. So, this
functionality will need more testing.

> When you submit v2, I would suggest splitting the tagger code from the
> switch driver code, just to make them easier to review.


Pengutronix e.K. | |
Industrial Linux Solutions | |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |