> -----Original Message-----
> From: Luca Boccassi [mailto:bl...@debian.org]
> Sent: Friday, November 2, 2018 8:19 AM
> To: dev@dpdk.org
> Cc: Lu, Wenzhuo <wenzhuo...@intel.com>; Ananyev, Konstantin
> <konstantin.anan...@intel.com>; Zhang, Qi Z <qi.z.zh...@intel.com>;
> 3ch...@gmail.com; Luca Boccassi <bl...@debian.org>; sta...@dpdk.org
> Subject: [PATCH v3 1/2] net/ixgbe: fix x550 code to handle unidentified PHY
>
> ixgbe_identify_sfp_module_X550em() was missing the code to handle
> unidentified PHY that has been there in 82599 so it was not able to complete
> initialization of ixgbe sequence if no sfp plugged in.
> Port it over to return an appropriate type and complete init sequence
> properly.
>
> Fixes: d2e72774e58c ("ixgbe/base: support X550")
> Cc: sta...@dpdk.org
>
> Signed-off-by: Luca Boccassi <bl...@debian.org>
> ---
> v2: refresh to remove merge conflict with master
> v3: coalesce fix into ixgbe_identify_sfp_module_X550em to avoid
> code duplication, improve comment
>
> drivers/net/ixgbe/base/ixgbe_x550.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/drivers/net/ixgbe/base/ixgbe_x550.c
> b/drivers/net/ixgbe/base/ixgbe_x550.c
> index f7b98af52..a88d5c86a 100644
> --- a/drivers/net/ixgbe/base/ixgbe_x550.c
> +++ b/drivers/net/ixgbe/base/ixgbe_x550.c
> @@ -1561,6 +1561,12 @@ s32 ixgbe_identify_sfp_module_X550em(struct
> ixgbe_hw *hw)
>
> status = ixgbe_identify_module_generic(hw);
>
> + /* Set PHY type none if no PHY detected to allow init without SFP */
> + if (hw->phy.type == ixgbe_phy_unknown) {
> + hw->phy.type = ixgbe_phy_none;
Set PHY type to none for a device that does have PHY looks weird.
does ixgeb_phy_generic works here?
Where is failure you met with ixgbe_phy_unknown?
> + return IXGBE_SUCCESS;
> + }
> +
> if (status != IXGBE_SUCCESS)
> return status;
>
> --
> 2.19.1
