Re: [PATCH net-next v3 0/9] net: ethernet: ti: ale: add staticconfiguration

From: David Miller
Date: Fri Sep 11 2020 - 20:35:25 EST


From: Grygorii Strashko <grygorii.strashko@xxxxxx>
Date: Thu, 10 Sep 2020 23:27:58 +0300

> As existing, as newly introduced CPSW ALE versions have differences in
> supported features and ALE table formats. Especially it's actual for the
> recent AM65x/J721E/J7200 and future AM64x SoCs, which supports more
> features like: auto-aging, classifiers, Link aggregation, additional HW
> filtering, etc.
>
> The existing ALE configuration interface is not practical in terms of
> adding new features and requires consumers to program a lot static
> parameters. And any attempt to add new features will case endless adding
> and maintaining different combination of flags and options. Because CPSW
> ALE configuration is static and fixed for SoC (or set of SoC), It is
> reasonable to add support for static ALE configurations inside ALE module.
>
> This series introduces static ALE configuration table for different ALE
> variants and provides option for consumers to select required ALE
> configuration by providing ALE const char *dev_id identifier (Patch 2).
> And all existing driver have been switched to use new approach (Patches 3-6).
>
> After this ALE HW auto-ageing feature can be enabled for AM65x CPSW ALE
> variant (Patch 7).
>
> Finally, Patches 8-9 introduces tables to describe the ALE VLAN entries
> fields as the ALE VLAN entries are too much differ between different TI
> CPSW ALE versions. So, handling them using flags, defines and get/set
> functions are became over-complicated.
...

Series applied, thank you.