[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20210326020727.246828-1-kuba@kernel.org>
Date: Thu, 25 Mar 2021 19:07:21 -0700
From: Jakub Kicinski <kuba@...nel.org>
To: davem@...emloft.net
Cc: netdev@...r.kernel.org, ecree.xilinx@...il.com,
michael.chan@...adcom.com, paul.greenwalt@...el.com,
rajur@...lsio.com, jaroslawx.gawin@...el.com, vkochan@...vell.com,
alobakin@...me, snelson@...sando.io, shayagr@...zon.com,
ayal@...dia.com, shenjian15@...wei.com, saeedm@...dia.com,
mkubecek@...e.cz, andrew@...n.ch, roopa@...dia.com,
Jakub Kicinski <kuba@...nel.org>
Subject: [PATCH net-next v2 0/6] ethtool: clarify the ethtool FEC interface
Our FEC configuration interface is one of the more confusing.
It also lacks any error checking in the core. This certainly
shows in the varying implementations across the drivers.
Improve the documentation and add most basic checks. Sadly, it's
probably too late now to try to enforce much more uniformity.
Any thoughts & suggestions welcome. Next step is to add netlink
for FEC, then stats.
v2:
- fix patch 5
- adjust kdoc in patches 3 and 6
Jakub Kicinski (6):
ethtool: fec: fix typo in kdoc
ethtool: fec: remove long structure description
ethtool: fec: sanitize ethtool_fecparam->reserved
ethtool: fec: sanitize ethtool_fecparam->active_fec
ethtool: fec: sanitize ethtool_fecparam->fec
ethtool: clarify the ethtool FEC interface
include/uapi/linux/ethtool.h | 45 +++++++++++++++++++++++++++---------
net/ethtool/ioctl.c | 9 ++++++++
2 files changed, 43 insertions(+), 11 deletions(-)
--
2.30.2
Powered by blists - more mailing lists