[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <20200911.173503.2013046040329976699.davem@davemloft.net>
Date: Fri, 11 Sep 2020 17:35:03 -0700 (PDT)
From: David Miller <davem@...emloft.net>
To: grygorii.strashko@...com
Cc: netdev@...r.kernel.org, kuba@...nel.org, vigneshr@...com,
m-karicheri2@...com, nsekhar@...com, linux-kernel@...r.kernel.org,
linux-omap@...r.kernel.org
Subject: Re: [PATCH net-next v3 0/9] net: ethernet: ti: ale: add static
configuration
From: Grygorii Strashko <grygorii.strashko@...com>
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.
Powered by blists - more mailing lists