From: Michal Swiatkowski <michal.swiatkowski@linux.intel.com>
To: intel-wired-lan@lists.osuosl.org
Cc: netdev@vger.kernel.org, aleksander.lobakin@intel.com,
przemyslaw.kitszel@intel.com, piotr.kwapulinski@intel.com,
aleksandr.loktionov@intel.com, jedrzej.jagielski@intel.com,
larysa.zaremba@intel.com, anthony.l.nguyen@intel.com
Subject: [iwl-next v2 0/8] libie: commonize adminq structure
Date: Thu, 10 Apr 2025 12:01:13 +0200 [thread overview]
Message-ID: <20250410100121.2353754-1-michal.swiatkowski@linux.intel.com> (raw)
Hi,
It is a prework to allow reusing some specific Intel code (eq. fwlog).
Move common *_aq_desc structure to libie header and changing
it in ice, ixgbe, i40e and iavf.
Only generic adminq commands can be easily moved to common header, as
rest is slightly different. Format remains the same. It will be better
to correctly move it when it will be needed to commonize other part of
the code.
Move *_aq_str() to new libie module (libie_adminq) and use it across
drivers. The functions are exactly the same in each driver. Some more
adminq helpers/functions can be moved to libie_adminq when needed.
v1 --> v2: [1]
* add short descriptions in kdoc (patch 1, 5)
* handle all error types in switch to allow clean build (patch 3)
[1] https://lore.kernel.org/netdev/20250312062426.2544608-1-michal.swiatkowski@linux.intel.com/
Michal Swiatkowski (8):
ice, libie: move generic adminq descriptors to lib
ixgbe: use libie adminq descriptors
i40e: use libie adminq descriptors
iavf: use libie adminq descriptors
libie: add adminq helper for converting err to str
ice: use libie_aq_str
iavf: use libie_aq_str
i40e: use libie_aq_str
drivers/net/ethernet/intel/Kconfig | 3 +
drivers/net/ethernet/intel/libie/Kconfig | 6 +
drivers/net/ethernet/intel/libie/Makefile | 4 +
drivers/net/ethernet/intel/i40e/i40e_adminq.h | 12 +-
.../net/ethernet/intel/i40e/i40e_adminq_cmd.h | 155 +---
.../net/ethernet/intel/i40e/i40e_prototype.h | 15 +-
drivers/net/ethernet/intel/i40e/i40e_type.h | 6 +-
drivers/net/ethernet/intel/iavf/iavf_adminq.h | 12 +-
.../net/ethernet/intel/iavf/iavf_adminq_cmd.h | 83 +-
.../net/ethernet/intel/iavf/iavf_prototype.h | 3 +-
drivers/net/ethernet/intel/iavf/iavf_type.h | 2 +-
drivers/net/ethernet/intel/ice/ice.h | 1 -
.../net/ethernet/intel/ice/ice_adminq_cmd.h | 269 +------
drivers/net/ethernet/intel/ice/ice_common.h | 6 +-
drivers/net/ethernet/intel/ice/ice_controlq.h | 8 +-
drivers/net/ethernet/intel/ixgbe/ixgbe_e610.h | 12 +-
.../ethernet/intel/ixgbe/ixgbe_type_e610.h | 226 +-----
include/linux/net/intel/libie/adminq.h | 306 ++++++++
drivers/net/ethernet/intel/i40e/i40e_adminq.c | 68 +-
drivers/net/ethernet/intel/i40e/i40e_client.c | 7 +-
drivers/net/ethernet/intel/i40e/i40e_common.c | 730 ++++++++----------
drivers/net/ethernet/intel/i40e/i40e_dcb.c | 10 +-
drivers/net/ethernet/intel/i40e/i40e_dcb_nl.c | 8 +-
.../net/ethernet/intel/i40e/i40e_debugfs.c | 46 +-
.../net/ethernet/intel/i40e/i40e_ethtool.c | 36 +-
drivers/net/ethernet/intel/i40e/i40e_main.c | 240 +++---
drivers/net/ethernet/intel/i40e/i40e_nvm.c | 18 +-
.../ethernet/intel/i40e/i40e_virtchnl_pf.c | 27 +-
drivers/net/ethernet/intel/iavf/iavf_adminq.c | 62 +-
drivers/net/ethernet/intel/iavf/iavf_common.c | 110 +--
drivers/net/ethernet/intel/iavf/iavf_main.c | 5 +-
.../net/ethernet/intel/iavf/iavf_virtchnl.c | 2 +-
.../net/ethernet/intel/ice/devlink/devlink.c | 10 +-
.../net/ethernet/intel/ice/devlink/health.c | 6 +-
drivers/net/ethernet/intel/ice/ice_common.c | 388 +++++-----
drivers/net/ethernet/intel/ice/ice_controlq.c | 53 +-
drivers/net/ethernet/intel/ice/ice_dcb.c | 36 +-
drivers/net/ethernet/intel/ice/ice_dcb_lib.c | 2 +-
drivers/net/ethernet/intel/ice/ice_ddp.c | 47 +-
drivers/net/ethernet/intel/ice/ice_dpll.c | 20 +-
drivers/net/ethernet/intel/ice/ice_ethtool.c | 12 +-
.../net/ethernet/intel/ice/ice_fw_update.c | 38 +-
drivers/net/ethernet/intel/ice/ice_fwlog.c | 16 +-
drivers/net/ethernet/intel/ice/ice_lag.c | 4 +-
drivers/net/ethernet/intel/ice/ice_lib.c | 10 +-
drivers/net/ethernet/intel/ice/ice_main.c | 63 +-
drivers/net/ethernet/intel/ice/ice_nvm.c | 38 +-
drivers/net/ethernet/intel/ice/ice_ptp_hw.c | 20 +-
drivers/net/ethernet/intel/ice/ice_sched.c | 18 +-
drivers/net/ethernet/intel/ice/ice_sriov.c | 4 +-
drivers/net/ethernet/intel/ice/ice_switch.c | 55 +-
drivers/net/ethernet/intel/ice/ice_vf_mbx.c | 6 +-
drivers/net/ethernet/intel/ice/ice_virtchnl.c | 6 +-
.../net/ethernet/intel/ice/ice_vlan_mode.c | 6 +-
.../net/ethernet/intel/ice/ice_vsi_vlan_lib.c | 24 +-
drivers/net/ethernet/intel/ixgbe/ixgbe_e610.c | 272 +++----
.../ethernet/intel/ixgbe/ixgbe_fw_update.c | 4 +-
drivers/net/ethernet/intel/libie/adminq.c | 50 ++
58 files changed, 1570 insertions(+), 2136 deletions(-)
create mode 100644 include/linux/net/intel/libie/adminq.h
create mode 100644 drivers/net/ethernet/intel/libie/adminq.c
--
2.42.0
next reply other threads:[~2025-04-10 10:01 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-04-10 10:01 Michal Swiatkowski [this message]
2025-04-10 10:01 ` [iwl-next v2 1/8] ice, libie: move generic adminq descriptors to lib Michal Swiatkowski
2025-04-21 10:58 ` Simon Horman
2025-04-10 10:01 ` [iwl-next v2 2/8] ixgbe: use libie adminq descriptors Michal Swiatkowski
2025-04-10 10:01 ` [iwl-next v2 3/8] i40e: " Michal Swiatkowski
2025-04-10 10:01 ` [iwl-next v2 4/8] iavf: " Michal Swiatkowski
2025-04-10 10:01 ` [iwl-next v2 5/8] libie: add adminq helper for converting err to str Michal Swiatkowski
2025-04-10 10:01 ` [iwl-next v2 6/8] ice: use libie_aq_str Michal Swiatkowski
2025-04-10 10:01 ` [iwl-next v2 7/8] iavf: " Michal Swiatkowski
2025-04-10 12:50 ` Loktionov, Aleksandr
2025-04-10 10:01 ` [iwl-next v2 8/8] i40e: " Michal Swiatkowski
2025-04-15 22:59 ` [iwl-next v2 0/8] libie: commonize adminq structure Tony Nguyen
2025-04-16 8:38 ` Michal Swiatkowski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20250410100121.2353754-1-michal.swiatkowski@linux.intel.com \
--to=michal.swiatkowski@linux.intel.com \
--cc=aleksander.lobakin@intel.com \
--cc=aleksandr.loktionov@intel.com \
--cc=anthony.l.nguyen@intel.com \
--cc=intel-wired-lan@lists.osuosl.org \
--cc=jedrzej.jagielski@intel.com \
--cc=larysa.zaremba@intel.com \
--cc=netdev@vger.kernel.org \
--cc=piotr.kwapulinski@intel.com \
--cc=przemyslaw.kitszel@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).