[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20191019211234.GH25148@lunn.ch>
Date: Sat, 19 Oct 2019 23:12:34 +0200
From: Andrew Lunn <andrew@...n.ch>
To: Jiri Pirko <jiri@...nulli.us>
Cc: David Miller <davem@...emloft.net>,
netdev <netdev@...r.kernel.org>,
Vivien Didelot <vivien.didelot@...il.com>
Subject: Re: [PATCH net-next v4 2/2] net: dsa: mv88e6xxx: Add devlink param
for ATU hash algorithm.
> Could you please follow the rest of the existing params?
Why are params special? Devlink resources can and do have upper case
characters. So we get into inconsistencies within devlink,
particularly if there is a link between a parameter and a resources.
And i will soon be adding a resource, and it will be upper case, since
that is allowed. And it will be related to the ATU.
Andrew
Powered by blists - more mailing lists