* [Intel-wired-lan] [PATCH net-next 1/3] iavf: Remove unused declarations
2024-09-03 12:22 [Intel-wired-lan] [PATCH net-next 0/3] Cleanup intel driver declarations Yue Haibing
@ 2024-09-03 12:22 ` Yue Haibing
2024-09-03 12:22 ` [Intel-wired-lan] [PATCH net-next 2/3] igb: Cleanup " Yue Haibing
` (3 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Yue Haibing @ 2024-09-03 12:22 UTC (permalink / raw)
To: anthony.l.nguyen, przemyslaw.kitszel, davem, kuba, pabeni,
ahmed.zaki, yuehaibing, richardcochran, michal.swiatkowski,
amritha.nambiar, mateusz.polchlopek, jacob.e.keller,
maciej.fijalkowski
Cc: netdev, intel-wired-lan
There is no caller and implementation in tree.
Signed-off-by: Yue Haibing <yuehaibing@huawei.com>
---
drivers/net/ethernet/intel/iavf/iavf.h | 10 ----------
drivers/net/ethernet/intel/iavf/iavf_prototype.h | 3 ---
2 files changed, 13 deletions(-)
diff --git a/drivers/net/ethernet/intel/iavf/iavf.h b/drivers/net/ethernet/intel/iavf/iavf.h
index 48cd1d06761c..95b1d8f62304 100644
--- a/drivers/net/ethernet/intel/iavf/iavf.h
+++ b/drivers/net/ethernet/intel/iavf/iavf.h
@@ -529,22 +529,16 @@ static inline void iavf_change_state(struct iavf_adapter *adapter,
iavf_state_str(adapter->state));
}
-int iavf_up(struct iavf_adapter *adapter);
void iavf_down(struct iavf_adapter *adapter);
int iavf_process_config(struct iavf_adapter *adapter);
int iavf_parse_vf_resource_msg(struct iavf_adapter *adapter);
void iavf_schedule_reset(struct iavf_adapter *adapter, u64 flags);
void iavf_schedule_aq_request(struct iavf_adapter *adapter, u64 flags);
void iavf_schedule_finish_config(struct iavf_adapter *adapter);
-void iavf_reset(struct iavf_adapter *adapter);
void iavf_set_ethtool_ops(struct net_device *netdev);
-void iavf_update_stats(struct iavf_adapter *adapter);
void iavf_free_all_tx_resources(struct iavf_adapter *adapter);
void iavf_free_all_rx_resources(struct iavf_adapter *adapter);
-void iavf_napi_add_all(struct iavf_adapter *adapter);
-void iavf_napi_del_all(struct iavf_adapter *adapter);
-
int iavf_send_api_ver(struct iavf_adapter *adapter);
int iavf_verify_api_ver(struct iavf_adapter *adapter);
int iavf_send_vf_config_msg(struct iavf_adapter *adapter);
@@ -555,11 +549,9 @@ void iavf_set_queue_vlan_tag_loc(struct iavf_adapter *adapter);
u16 iavf_get_num_vlans_added(struct iavf_adapter *adapter);
void iavf_irq_enable(struct iavf_adapter *adapter, bool flush);
void iavf_configure_queues(struct iavf_adapter *adapter);
-void iavf_deconfigure_queues(struct iavf_adapter *adapter);
void iavf_enable_queues(struct iavf_adapter *adapter);
void iavf_disable_queues(struct iavf_adapter *adapter);
void iavf_map_queues(struct iavf_adapter *adapter);
-int iavf_request_queues(struct iavf_adapter *adapter, int num);
void iavf_add_ether_addrs(struct iavf_adapter *adapter);
void iavf_del_ether_addrs(struct iavf_adapter *adapter);
void iavf_add_vlans(struct iavf_adapter *adapter);
@@ -579,8 +571,6 @@ void iavf_virtchnl_completion(struct iavf_adapter *adapter,
enum virtchnl_ops v_opcode,
enum iavf_status v_retval, u8 *msg, u16 msglen);
int iavf_config_rss(struct iavf_adapter *adapter);
-int iavf_lan_add_device(struct iavf_adapter *adapter);
-int iavf_lan_del_device(struct iavf_adapter *adapter);
void iavf_enable_channels(struct iavf_adapter *adapter);
void iavf_disable_channels(struct iavf_adapter *adapter);
void iavf_add_cloud_filter(struct iavf_adapter *adapter);
diff --git a/drivers/net/ethernet/intel/iavf/iavf_prototype.h b/drivers/net/ethernet/intel/iavf/iavf_prototype.h
index 48c3901381b4..cac9d1a35a52 100644
--- a/drivers/net/ethernet/intel/iavf/iavf_prototype.h
+++ b/drivers/net/ethernet/intel/iavf/iavf_prototype.h
@@ -18,7 +18,6 @@
/* adminq functions */
enum iavf_status iavf_init_adminq(struct iavf_hw *hw);
enum iavf_status iavf_shutdown_adminq(struct iavf_hw *hw);
-void iavf_adminq_init_ring_data(struct iavf_hw *hw);
enum iavf_status iavf_clean_arq_element(struct iavf_hw *hw,
struct iavf_arq_event_info *e,
u16 *events_pending);
@@ -33,8 +32,6 @@ bool iavf_asq_done(struct iavf_hw *hw);
void iavf_debug_aq(struct iavf_hw *hw, enum iavf_debug_mask mask,
void *desc, void *buffer, u16 buf_len);
-void iavf_idle_aq(struct iavf_hw *hw);
-void iavf_resume_aq(struct iavf_hw *hw);
bool iavf_check_asq_alive(struct iavf_hw *hw);
enum iavf_status iavf_aq_queue_shutdown(struct iavf_hw *hw, bool unloading);
const char *iavf_aq_str(struct iavf_hw *hw, enum iavf_admin_queue_err aq_err);
--
2.34.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* [Intel-wired-lan] [PATCH net-next 2/3] igb: Cleanup unused declarations
2024-09-03 12:22 [Intel-wired-lan] [PATCH net-next 0/3] Cleanup intel driver declarations Yue Haibing
2024-09-03 12:22 ` [Intel-wired-lan] [PATCH net-next 1/3] iavf: Remove unused declarations Yue Haibing
@ 2024-09-03 12:22 ` Yue Haibing
2024-09-03 12:22 ` [Intel-wired-lan] [PATCH net-next 3/3] ice: " Yue Haibing
` (2 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Yue Haibing @ 2024-09-03 12:22 UTC (permalink / raw)
To: anthony.l.nguyen, przemyslaw.kitszel, davem, kuba, pabeni,
ahmed.zaki, yuehaibing, richardcochran, michal.swiatkowski,
amritha.nambiar, mateusz.polchlopek, jacob.e.keller,
maciej.fijalkowski
Cc: netdev, intel-wired-lan
e1000_init_function_pointers_82575() is never implemented and used since
commit 9d5c824399de ("igb: PCI-Express 82575 Gigabit Ethernet driver").
And commit 9835fd7321a6 ("igb: Add new function to read part number from
EEPROM in string format") removed igb_read_part_num() implementation.
Signed-off-by: Yue Haibing <yuehaibing@huawei.com>
---
drivers/net/ethernet/intel/igb/e1000_mac.h | 1 -
drivers/net/ethernet/intel/igb/e1000_nvm.h | 1 -
2 files changed, 2 deletions(-)
diff --git a/drivers/net/ethernet/intel/igb/e1000_mac.h b/drivers/net/ethernet/intel/igb/e1000_mac.h
index 6e110f28f922..529b7d18b662 100644
--- a/drivers/net/ethernet/intel/igb/e1000_mac.h
+++ b/drivers/net/ethernet/intel/igb/e1000_mac.h
@@ -63,6 +63,5 @@ enum e1000_mng_mode {
#define E1000_MNG_DHCP_COOKIE_STATUS_VLAN 0x2
-void e1000_init_function_pointers_82575(struct e1000_hw *hw);
#endif
diff --git a/drivers/net/ethernet/intel/igb/e1000_nvm.h b/drivers/net/ethernet/intel/igb/e1000_nvm.h
index 091cddf4ada8..4f652ab713b3 100644
--- a/drivers/net/ethernet/intel/igb/e1000_nvm.h
+++ b/drivers/net/ethernet/intel/igb/e1000_nvm.h
@@ -7,7 +7,6 @@
s32 igb_acquire_nvm(struct e1000_hw *hw);
void igb_release_nvm(struct e1000_hw *hw);
s32 igb_read_mac_addr(struct e1000_hw *hw);
-s32 igb_read_part_num(struct e1000_hw *hw, u32 *part_num);
s32 igb_read_part_string(struct e1000_hw *hw, u8 *part_num,
u32 part_num_size);
s32 igb_read_nvm_eerd(struct e1000_hw *hw, u16 offset, u16 words, u16 *data);
--
2.34.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* [Intel-wired-lan] [PATCH net-next 3/3] ice: Cleanup unused declarations
2024-09-03 12:22 [Intel-wired-lan] [PATCH net-next 0/3] Cleanup intel driver declarations Yue Haibing
2024-09-03 12:22 ` [Intel-wired-lan] [PATCH net-next 1/3] iavf: Remove unused declarations Yue Haibing
2024-09-03 12:22 ` [Intel-wired-lan] [PATCH net-next 2/3] igb: Cleanup " Yue Haibing
@ 2024-09-03 12:22 ` Yue Haibing
2024-09-03 12:35 ` [Intel-wired-lan] [PATCH net-next 0/3] Cleanup intel driver declarations Fijalkowski, Maciej
2024-09-03 21:18 ` Jacob Keller
4 siblings, 0 replies; 6+ messages in thread
From: Yue Haibing @ 2024-09-03 12:22 UTC (permalink / raw)
To: anthony.l.nguyen, przemyslaw.kitszel, davem, kuba, pabeni,
ahmed.zaki, yuehaibing, richardcochran, michal.swiatkowski,
amritha.nambiar, mateusz.polchlopek, jacob.e.keller,
maciej.fijalkowski
Cc: netdev, intel-wired-lan
Since commit fff292b47ac1 ("ice: add VF representors one by one")
ice_eswitch_configure() is not used anymore.
Commit 1b8f15b64a00 ("ice: refactor filter functions") removed
ice_vsi_cfg_mac_fltr() but leave declaration.
Commit a24b4c6e9aab ("ice: xsk: Do not convert to buff to frame for
XDP_TX") leave ice_xmit_xdp_buff() declaration.
Commit 7cab44f1c35f ("ice: Introduce ETH56G PHY model for E825C
products") declared ice_phy_cfg_{rx,tx}_offset_eth56g(),
commit a1ffafb0b4a4 ("ice: Support configuring the device to Double
VLAN Mode") declared ice_pkg_buf_get_free_space(), and
commit 8a3a565ff210 ("ice: add admin commands to access cgu
configuration") declared ice_is_pca9575_present(), but all these never
be implemented.
Signed-off-by: Yue Haibing <yuehaibing@huawei.com>
---
drivers/net/ethernet/intel/ice/ice_eswitch.h | 5 -----
drivers/net/ethernet/intel/ice/ice_flex_pipe.h | 3 ---
drivers/net/ethernet/intel/ice/ice_lib.h | 2 --
drivers/net/ethernet/intel/ice/ice_ptp_hw.h | 3 ---
drivers/net/ethernet/intel/ice/ice_txrx_lib.h | 1 -
5 files changed, 14 deletions(-)
diff --git a/drivers/net/ethernet/intel/ice/ice_eswitch.h b/drivers/net/ethernet/intel/ice/ice_eswitch.h
index 78fd39a6935d..cf4b7845f08f 100644
--- a/drivers/net/ethernet/intel/ice/ice_eswitch.h
+++ b/drivers/net/ethernet/intel/ice/ice_eswitch.h
@@ -48,11 +48,6 @@ ice_eswitch_set_target_vsi(struct sk_buff *skb,
static inline void
ice_eswitch_update_repr(unsigned long *repr_id, struct ice_vsi *vsi) { }
-static inline int ice_eswitch_configure(struct ice_pf *pf)
-{
- return 0;
-}
-
static inline int ice_eswitch_mode_get(struct devlink *devlink, u16 *mode)
{
return DEVLINK_ESWITCH_MODE_LEGACY;
diff --git a/drivers/net/ethernet/intel/ice/ice_flex_pipe.h b/drivers/net/ethernet/intel/ice/ice_flex_pipe.h
index 90b9b0993122..28b0897adf32 100644
--- a/drivers/net/ethernet/intel/ice/ice_flex_pipe.h
+++ b/drivers/net/ethernet/intel/ice/ice_flex_pipe.h
@@ -23,9 +23,6 @@ int
ice_get_sw_fv_list(struct ice_hw *hw, struct ice_prot_lkup_ext *lkups,
unsigned long *bm, struct list_head *fv_list);
int
-ice_pkg_buf_unreserve_section(struct ice_buf_build *bld, u16 count);
-u16 ice_pkg_buf_get_free_space(struct ice_buf_build *bld);
-int
ice_aq_upload_section(struct ice_hw *hw, struct ice_buf_hdr *pkg_buf,
u16 buf_size, struct ice_sq_cd *cd);
bool
diff --git a/drivers/net/ethernet/intel/ice/ice_lib.h b/drivers/net/ethernet/intel/ice/ice_lib.h
index 94ce8964dda6..77dff4ab875a 100644
--- a/drivers/net/ethernet/intel/ice/ice_lib.h
+++ b/drivers/net/ethernet/intel/ice/ice_lib.h
@@ -91,8 +91,6 @@ void ice_write_intrl(struct ice_q_vector *q_vector, u8 intrl);
void ice_write_itr(struct ice_ring_container *rc, u16 itr);
void ice_set_q_vector_intrl(struct ice_q_vector *q_vector);
-int ice_vsi_cfg_mac_fltr(struct ice_vsi *vsi, const u8 *macaddr, bool set);
-
bool ice_is_safe_mode(struct ice_pf *pf);
bool ice_is_rdma_ena(struct ice_pf *pf);
bool ice_is_dflt_vsi_in_use(struct ice_port_info *pi);
diff --git a/drivers/net/ethernet/intel/ice/ice_ptp_hw.h b/drivers/net/ethernet/intel/ice/ice_ptp_hw.h
index 0852a34ade91..67a9711d01ac 100644
--- a/drivers/net/ethernet/intel/ice/ice_ptp_hw.h
+++ b/drivers/net/ethernet/intel/ice/ice_ptp_hw.h
@@ -403,7 +403,6 @@ int ice_phy_cfg_intr_e82x(struct ice_hw *hw, u8 quad, bool ena, u8 threshold);
int ice_read_sma_ctrl_e810t(struct ice_hw *hw, u8 *data);
int ice_write_sma_ctrl_e810t(struct ice_hw *hw, u8 data);
int ice_read_pca9575_reg_e810t(struct ice_hw *hw, u8 offset, u8 *data);
-bool ice_is_pca9575_present(struct ice_hw *hw);
enum dpll_pin_type ice_cgu_get_pin_type(struct ice_hw *hw, u8 pin, bool input);
struct dpll_pin_frequency *
ice_cgu_get_pin_freq_supp(struct ice_hw *hw, u8 pin, bool input, u8 *num);
@@ -420,8 +419,6 @@ int ice_cgu_get_output_pin_state_caps(struct ice_hw *hw, u8 pin_id,
int ice_ptp_read_tx_hwtstamp_status_eth56g(struct ice_hw *hw, u32 *ts_status);
int ice_stop_phy_timer_eth56g(struct ice_hw *hw, u8 port, bool soft_reset);
int ice_start_phy_timer_eth56g(struct ice_hw *hw, u8 port);
-int ice_phy_cfg_tx_offset_eth56g(struct ice_hw *hw, u8 port);
-int ice_phy_cfg_rx_offset_eth56g(struct ice_hw *hw, u8 port);
int ice_phy_cfg_intr_eth56g(struct ice_hw *hw, u8 port, bool ena, u8 threshold);
int ice_phy_cfg_ptp_1step_eth56g(struct ice_hw *hw, u8 port);
diff --git a/drivers/net/ethernet/intel/ice/ice_txrx_lib.h b/drivers/net/ethernet/intel/ice/ice_txrx_lib.h
index afcead4baef4..79f960c6680d 100644
--- a/drivers/net/ethernet/intel/ice/ice_txrx_lib.h
+++ b/drivers/net/ethernet/intel/ice/ice_txrx_lib.h
@@ -154,7 +154,6 @@ static inline u32 ice_set_rs_bit(const struct ice_tx_ring *xdp_ring)
}
void ice_finalize_xdp_rx(struct ice_tx_ring *xdp_ring, unsigned int xdp_res, u32 first_idx);
-int ice_xmit_xdp_buff(struct xdp_buff *xdp, struct ice_tx_ring *xdp_ring);
int __ice_xmit_xdp_ring(struct xdp_buff *xdp, struct ice_tx_ring *xdp_ring,
bool frame);
void ice_release_rx_desc(struct ice_rx_ring *rx_ring, u16 val);
--
2.34.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [Intel-wired-lan] [PATCH net-next 0/3] Cleanup intel driver declarations
2024-09-03 12:22 [Intel-wired-lan] [PATCH net-next 0/3] Cleanup intel driver declarations Yue Haibing
` (2 preceding siblings ...)
2024-09-03 12:22 ` [Intel-wired-lan] [PATCH net-next 3/3] ice: " Yue Haibing
@ 2024-09-03 12:35 ` Fijalkowski, Maciej
2024-09-03 21:18 ` Jacob Keller
4 siblings, 0 replies; 6+ messages in thread
From: Fijalkowski, Maciej @ 2024-09-03 12:35 UTC (permalink / raw)
To: Yue Haibing, Nguyen, Anthony L, Kitszel, Przemyslaw,
davem@davemloft.net, kuba@kernel.org, pabeni@redhat.com,
Zaki, Ahmed, richardcochran@gmail.com,
michal.swiatkowski@linux.intel.com, Nambiar, Amritha,
Polchlopek, Mateusz, Keller, Jacob E
Cc: netdev@vger.kernel.org, intel-wired-lan@lists.osuosl.org
Hi Yue,
I know these patches are obvious but one sentence in cover letter
wouldn't hurt 😊
Anyways, for the content:
Reviewed-by: Maciej Fijalkowski <maciej.fijalkowski@intel.com>
> Yue Haibing (3):
> iavf: Remove unused declarations
> igb: Cleanup unused declarations
> ice: Cleanup unused declarations
>
> drivers/net/ethernet/intel/iavf/iavf.h | 10 ----------
> drivers/net/ethernet/intel/iavf/iavf_prototype.h | 3 ---
> drivers/net/ethernet/intel/ice/ice_eswitch.h | 5 -----
> drivers/net/ethernet/intel/ice/ice_flex_pipe.h | 3 ---
> drivers/net/ethernet/intel/ice/ice_lib.h | 2 --
> drivers/net/ethernet/intel/ice/ice_ptp_hw.h | 3 ---
> drivers/net/ethernet/intel/ice/ice_txrx_lib.h | 1 -
> drivers/net/ethernet/intel/igb/e1000_mac.h | 1 -
> drivers/net/ethernet/intel/igb/e1000_nvm.h | 1 -
> 9 files changed, 29 deletions(-)
>
> --
> 2.34.1
>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [Intel-wired-lan] [PATCH net-next 0/3] Cleanup intel driver declarations
2024-09-03 12:22 [Intel-wired-lan] [PATCH net-next 0/3] Cleanup intel driver declarations Yue Haibing
` (3 preceding siblings ...)
2024-09-03 12:35 ` [Intel-wired-lan] [PATCH net-next 0/3] Cleanup intel driver declarations Fijalkowski, Maciej
@ 2024-09-03 21:18 ` Jacob Keller
4 siblings, 0 replies; 6+ messages in thread
From: Jacob Keller @ 2024-09-03 21:18 UTC (permalink / raw)
To: Yue Haibing, anthony.l.nguyen, przemyslaw.kitszel, davem, kuba,
pabeni, ahmed.zaki, richardcochran, michal.swiatkowski,
amritha.nambiar, mateusz.polchlopek, maciej.fijalkowski
Cc: netdev, intel-wired-lan
On 9/3/2024 5:22 AM, Yue Haibing wrote:
>
> Yue Haibing (3):
> iavf: Remove unused declarations
> igb: Cleanup unused declarations
> ice: Cleanup unused declarations
>
It would be interesting to see when the code for these declarations was
removed (if it was ever present in the kernel to begin with?)
^ permalink raw reply [flat|nested] 6+ messages in thread