Hi,
> -----Original Message-----
> From: Sun, GuinanX <guinanx....@intel.com>
> Sent: Monday, June 22, 2020 2:46 PM
> To: dev@dpdk.org
> Cc: Guo, Jia <jia....@intel.com>; Zhao1, Wei <wei.zh...@intel.com>; Sun,
> GuinanX <guinanx....@intel.com>; Trakhtenberg, Dany
> <dany.trakhtenb...@intel.com>
> Subject: [PATCH 15/70] net/e1000/base: setup copper link function for i225
>
> A dedicated function for i225 that Configures the link for auto-neg or forced
> speed and duplex. It was taken from the
> e1000_setup_copper_link_82575 function in e1000_82575 file and was
> refactored for i225 only.
>
> Signed-off-by: Dany Trakhtenberg <dany.trakhtenb...@intel.com>
> Signed-off-by: Guinan Sun <guinanx....@intel.com>
> ---
> drivers/net/e1000/base/e1000_i225.c | 30
> +++++++++++++++++++++++++++++ drivers/net/e1000/base/e1000_phy.h
> | 13 +++++++++++++
> 2 files changed, 43 insertions(+)
>
> diff --git a/drivers/net/e1000/base/e1000_i225.c
> b/drivers/net/e1000/base/e1000_i225.c
> index 4ba6c45be..621dc6863 100644
> --- a/drivers/net/e1000/base/e1000_i225.c
> +++ b/drivers/net/e1000/base/e1000_i225.c
It seems we should commit to folder dpdk/drivers/net/igc/base not in e1000
folder for i225
> @@ -124,6 +124,36 @@ void e1000_release_swfw_sync_i225(struct
> e1000_hw *hw, u16 mask)
> e1000_put_hw_semaphore_generic(hw);
> }
>
> +/*
> + * e1000_setup_copper_link_i225 - Configure copper link settings
> + * @hw: pointer to the HW structure
> + *
> + * Configures the link for auto-neg or forced speed and duplex. Then
> +we check
> + * for link, once link is established calls to configure collision
> +distance
> + * and flow control are called.
> + */
> +s32 e1000_setup_copper_link_i225(struct e1000_hw *hw) {
> + u32 phpm_reg;
> +
