Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 13 May 2016 07:05:40 +0000 (UTC)
From:      Andrew Rybchenko <arybchik@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r299603 - in head/sys: dev/sfxge/common modules/sfxge
Message-ID:  <201605130705.u4D75e5S037012@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: arybchik
Date: Fri May 13 07:05:40 2016
New Revision: 299603
URL: https://svnweb.freebsd.org/changeset/base/299603

Log:
  sfxge(4): move ef10_phy_* to ef10_phy.c
  
  Submitted by:   Andy Moreton <amoreton at solarflare.com>
  Sponsored by:   Solarflare Communications, Inc.
  MFC after:      1 week

Added:
  head/sys/dev/sfxge/common/ef10_phy.c
     - copied, changed from r299592, head/sys/dev/sfxge/common/hunt_phy.c
Modified:
  head/sys/dev/sfxge/common/hunt_phy.c
  head/sys/modules/sfxge/Makefile

Copied and modified: head/sys/dev/sfxge/common/ef10_phy.c (from r299592, head/sys/dev/sfxge/common/hunt_phy.c)
==============================================================================
--- head/sys/dev/sfxge/common/hunt_phy.c	Fri May 13 06:26:42 2016	(r299592, copy source)
+++ head/sys/dev/sfxge/common/ef10_phy.c	Fri May 13 07:05:40 2016	(r299603)
@@ -34,7 +34,7 @@ __FBSDID("$FreeBSD$");
 #include "efx.h"
 #include "efx_impl.h"
 
-#if EFSYS_OPT_HUNTINGTON
+#if EFSYS_OPT_HUNTINGTON || EFSYS_OPT_MEDFORD
 
 static			void
 mcdi_phy_decode_cap(
@@ -515,155 +515,4 @@ ef10_phy_prop_set(
 
 #endif	/* EFSYS_OPT_PHY_PROPS */
 
-#if EFSYS_OPT_BIST
-
-	__checkReturn		efx_rc_t
-hunt_bist_enable_offline(
-	__in			efx_nic_t *enp)
-{
-	efx_rc_t rc;
-
-	if ((rc = efx_mcdi_bist_enable_offline(enp)) != 0)
-		goto fail1;
-
-	return (0);
-
-fail1:
-	EFSYS_PROBE1(fail1, efx_rc_t, rc);
-
-	return (rc);
-}
-
-	__checkReturn		efx_rc_t
-hunt_bist_start(
-	__in			efx_nic_t *enp,
-	__in			efx_bist_type_t type)
-{
-	efx_rc_t rc;
-
-	if ((rc = efx_mcdi_bist_start(enp, type)) != 0)
-		goto fail1;
-
-	return (0);
-
-fail1:
-	EFSYS_PROBE1(fail1, efx_rc_t, rc);
-
-	return (rc);
-}
-
-	__checkReturn		efx_rc_t
-hunt_bist_poll(
-	__in			efx_nic_t *enp,
-	__in			efx_bist_type_t type,
-	__out			efx_bist_result_t *resultp,
-	__out_opt __drv_when(count > 0, __notnull)
-	uint32_t *value_maskp,
-	__out_ecount_opt(count)	__drv_when(count > 0, __notnull)
-	unsigned long *valuesp,
-	__in			size_t count)
-{
-	efx_nic_cfg_t *encp = &(enp->en_nic_cfg);
-	efx_mcdi_req_t req;
-	uint8_t payload[MAX(MC_CMD_POLL_BIST_IN_LEN,
-			    MCDI_CTL_SDU_LEN_MAX)];
-	uint32_t value_mask = 0;
-	uint32_t result;
-	efx_rc_t rc;
-
-	(void) memset(payload, 0, sizeof (payload));
-	req.emr_cmd = MC_CMD_POLL_BIST;
-	req.emr_in_buf = payload;
-	req.emr_in_length = MC_CMD_POLL_BIST_IN_LEN;
-	req.emr_out_buf = payload;
-	req.emr_out_length = MCDI_CTL_SDU_LEN_MAX;
-
-	efx_mcdi_execute(enp, &req);
-
-	if (req.emr_rc != 0) {
-		rc = req.emr_rc;
-		goto fail1;
-	}
-
-	if (req.emr_out_length_used < MC_CMD_POLL_BIST_OUT_RESULT_OFST + 4) {
-		rc = EMSGSIZE;
-		goto fail2;
-	}
-
-	if (count > 0)
-		(void) memset(valuesp, '\0', count * sizeof (unsigned long));
-
-	result = MCDI_OUT_DWORD(req, POLL_BIST_OUT_RESULT);
-
-	if (result == MC_CMD_POLL_BIST_FAILED &&
-	    req.emr_out_length >= MC_CMD_POLL_BIST_OUT_MEM_LEN &&
-	    count > EFX_BIST_MEM_ECC_FATAL) {
-		if (valuesp != NULL) {
-			valuesp[EFX_BIST_MEM_TEST] =
-			    MCDI_OUT_DWORD(req, POLL_BIST_OUT_MEM_TEST);
-			valuesp[EFX_BIST_MEM_ADDR] =
-			    MCDI_OUT_DWORD(req, POLL_BIST_OUT_MEM_ADDR);
-			valuesp[EFX_BIST_MEM_BUS] =
-			    MCDI_OUT_DWORD(req, POLL_BIST_OUT_MEM_BUS);
-			valuesp[EFX_BIST_MEM_EXPECT] =
-			    MCDI_OUT_DWORD(req, POLL_BIST_OUT_MEM_EXPECT);
-			valuesp[EFX_BIST_MEM_ACTUAL] =
-			    MCDI_OUT_DWORD(req, POLL_BIST_OUT_MEM_ACTUAL);
-			valuesp[EFX_BIST_MEM_ECC] =
-			    MCDI_OUT_DWORD(req, POLL_BIST_OUT_MEM_ECC);
-			valuesp[EFX_BIST_MEM_ECC_PARITY] =
-			    MCDI_OUT_DWORD(req, POLL_BIST_OUT_MEM_ECC_PARITY);
-			valuesp[EFX_BIST_MEM_ECC_FATAL] =
-			    MCDI_OUT_DWORD(req, POLL_BIST_OUT_MEM_ECC_FATAL);
-		}
-		value_mask |= (1 << EFX_BIST_MEM_TEST) |
-		    (1 << EFX_BIST_MEM_ADDR) |
-		    (1 << EFX_BIST_MEM_BUS) |
-		    (1 << EFX_BIST_MEM_EXPECT) |
-		    (1 << EFX_BIST_MEM_ACTUAL) |
-		    (1 << EFX_BIST_MEM_ECC) |
-		    (1 << EFX_BIST_MEM_ECC_PARITY) |
-		    (1 << EFX_BIST_MEM_ECC_FATAL);
-	} else if (result == MC_CMD_POLL_BIST_FAILED &&
-	    encp->enc_phy_type == EFX_PHY_XFI_FARMI &&
-	    req.emr_out_length >= MC_CMD_POLL_BIST_OUT_MRSFP_LEN &&
-	    count > EFX_BIST_FAULT_CODE) {
-		if (valuesp != NULL)
-			valuesp[EFX_BIST_FAULT_CODE] =
-			    MCDI_OUT_DWORD(req, POLL_BIST_OUT_MRSFP_TEST);
-		value_mask |= 1 << EFX_BIST_FAULT_CODE;
-	}
-
-	if (value_maskp != NULL)
-		*value_maskp = value_mask;
-
-	EFSYS_ASSERT(resultp != NULL);
-	if (result == MC_CMD_POLL_BIST_RUNNING)
-		*resultp = EFX_BIST_RESULT_RUNNING;
-	else if (result == MC_CMD_POLL_BIST_PASSED)
-		*resultp = EFX_BIST_RESULT_PASSED;
-	else
-		*resultp = EFX_BIST_RESULT_FAILED;
-
-	return (0);
-
-fail2:
-	EFSYS_PROBE(fail2);
-fail1:
-	EFSYS_PROBE1(fail1, efx_rc_t, rc);
-
-	return (rc);
-}
-
-			void
-hunt_bist_stop(
-	__in		efx_nic_t *enp,
-	__in		efx_bist_type_t type)
-{
-	/* There is no way to stop BIST on Huntinton. */
-	_NOTE(ARGUNUSED(enp, type))
-}
-
-#endif	/* EFSYS_OPT_BIST */
-
-#endif	/* EFSYS_OPT_HUNTINGTON */
+#endif	/* EFSYS_OPT_HUNTINGTON || EFSYS_OPT_MEDFORD */

Modified: head/sys/dev/sfxge/common/hunt_phy.c
==============================================================================
--- head/sys/dev/sfxge/common/hunt_phy.c	Fri May 13 07:04:28 2016	(r299602)
+++ head/sys/dev/sfxge/common/hunt_phy.c	Fri May 13 07:05:40 2016	(r299603)
@@ -36,485 +36,6 @@ __FBSDID("$FreeBSD$");
 
 #if EFSYS_OPT_HUNTINGTON
 
-static			void
-mcdi_phy_decode_cap(
-	__in		uint32_t mcdi_cap,
-	__out		uint32_t *maskp)
-{
-	uint32_t mask;
-
-	mask = 0;
-	if (mcdi_cap & (1 << MC_CMD_PHY_CAP_10HDX_LBN))
-		mask |= (1 << EFX_PHY_CAP_10HDX);
-	if (mcdi_cap & (1 << MC_CMD_PHY_CAP_10FDX_LBN))
-		mask |= (1 << EFX_PHY_CAP_10FDX);
-	if (mcdi_cap & (1 << MC_CMD_PHY_CAP_100HDX_LBN))
-		mask |= (1 << EFX_PHY_CAP_100HDX);
-	if (mcdi_cap & (1 << MC_CMD_PHY_CAP_100FDX_LBN))
-		mask |= (1 << EFX_PHY_CAP_100FDX);
-	if (mcdi_cap & (1 << MC_CMD_PHY_CAP_1000HDX_LBN))
-		mask |= (1 << EFX_PHY_CAP_1000HDX);
-	if (mcdi_cap & (1 << MC_CMD_PHY_CAP_1000FDX_LBN))
-		mask |= (1 << EFX_PHY_CAP_1000FDX);
-	if (mcdi_cap & (1 << MC_CMD_PHY_CAP_10000FDX_LBN))
-		mask |= (1 << EFX_PHY_CAP_10000FDX);
-	if (mcdi_cap & (1 << MC_CMD_PHY_CAP_40000FDX_LBN))
-		mask |= (1 << EFX_PHY_CAP_40000FDX);
-	if (mcdi_cap & (1 << MC_CMD_PHY_CAP_PAUSE_LBN))
-		mask |= (1 << EFX_PHY_CAP_PAUSE);
-	if (mcdi_cap & (1 << MC_CMD_PHY_CAP_ASYM_LBN))
-		mask |= (1 << EFX_PHY_CAP_ASYM);
-	if (mcdi_cap & (1 << MC_CMD_PHY_CAP_AN_LBN))
-		mask |= (1 << EFX_PHY_CAP_AN);
-
-	*maskp = mask;
-}
-
-static			void
-mcdi_phy_decode_link_mode(
-	__in		efx_nic_t *enp,
-	__in		uint32_t link_flags,
-	__in		unsigned int speed,
-	__in		unsigned int fcntl,
-	__out		efx_link_mode_t *link_modep,
-	__out		unsigned int *fcntlp)
-{
-	boolean_t fd = !!(link_flags &
-		    (1 << MC_CMD_GET_LINK_OUT_FULL_DUPLEX_LBN));
-	boolean_t up = !!(link_flags &
-		    (1 << MC_CMD_GET_LINK_OUT_LINK_UP_LBN));
-
-	_NOTE(ARGUNUSED(enp))
-
-	if (!up)
-		*link_modep = EFX_LINK_DOWN;
-	else if (speed == 40000 && fd)
-		*link_modep = EFX_LINK_40000FDX;
-	else if (speed == 10000 && fd)
-		*link_modep = EFX_LINK_10000FDX;
-	else if (speed == 1000)
-		*link_modep = fd ? EFX_LINK_1000FDX : EFX_LINK_1000HDX;
-	else if (speed == 100)
-		*link_modep = fd ? EFX_LINK_100FDX : EFX_LINK_100HDX;
-	else if (speed == 10)
-		*link_modep = fd ? EFX_LINK_10FDX : EFX_LINK_10HDX;
-	else
-		*link_modep = EFX_LINK_UNKNOWN;
-
-	if (fcntl == MC_CMD_FCNTL_OFF)
-		*fcntlp = 0;
-	else if (fcntl == MC_CMD_FCNTL_RESPOND)
-		*fcntlp = EFX_FCNTL_RESPOND;
-	else if (fcntl == MC_CMD_FCNTL_GENERATE)
-		*fcntlp = EFX_FCNTL_GENERATE;
-	else if (fcntl == MC_CMD_FCNTL_BIDIR)
-		*fcntlp = EFX_FCNTL_RESPOND | EFX_FCNTL_GENERATE;
-	else {
-		EFSYS_PROBE1(mc_pcol_error, int, fcntl);
-		*fcntlp = 0;
-	}
-}
-
-
-			void
-ef10_phy_link_ev(
-	__in		efx_nic_t *enp,
-	__in		efx_qword_t *eqp,
-	__out		efx_link_mode_t *link_modep)
-{
-	efx_port_t *epp = &(enp->en_port);
-	unsigned int link_flags;
-	unsigned int speed;
-	unsigned int fcntl;
-	efx_link_mode_t link_mode;
-	uint32_t lp_cap_mask;
-
-	/*
-	 * Convert the LINKCHANGE speed enumeration into mbit/s, in the
-	 * same way as GET_LINK encodes the speed
-	 */
-	switch (MCDI_EV_FIELD(eqp, LINKCHANGE_SPEED)) {
-	case MCDI_EVENT_LINKCHANGE_SPEED_100M:
-		speed = 100;
-		break;
-	case MCDI_EVENT_LINKCHANGE_SPEED_1G:
-		speed = 1000;
-		break;
-	case MCDI_EVENT_LINKCHANGE_SPEED_10G:
-		speed = 10000;
-		break;
-	case MCDI_EVENT_LINKCHANGE_SPEED_40G:
-		speed = 40000;
-		break;
-	default:
-		speed = 0;
-		break;
-	}
-
-	link_flags = MCDI_EV_FIELD(eqp, LINKCHANGE_LINK_FLAGS);
-	mcdi_phy_decode_link_mode(enp, link_flags, speed,
-				    MCDI_EV_FIELD(eqp, LINKCHANGE_FCNTL),
-				    &link_mode, &fcntl);
-	mcdi_phy_decode_cap(MCDI_EV_FIELD(eqp, LINKCHANGE_LP_CAP),
-			    &lp_cap_mask);
-
-	/*
-	 * It's safe to update ep_lp_cap_mask without the driver's port lock
-	 * because presumably any concurrently running efx_port_poll() is
-	 * only going to arrive at the same value.
-	 *
-	 * ep_fcntl has two meanings. It's either the link common fcntl
-	 * (if the PHY supports AN), or it's the forced link state. If
-	 * the former, it's safe to update the value for the same reason as
-	 * for ep_lp_cap_mask. If the latter, then just ignore the value,
-	 * because we can race with efx_mac_fcntl_set().
-	 */
-	epp->ep_lp_cap_mask = lp_cap_mask;
-	epp->ep_fcntl = fcntl;
-
-	*link_modep = link_mode;
-}
-
-	__checkReturn	efx_rc_t
-ef10_phy_power(
-	__in		efx_nic_t *enp,
-	__in		boolean_t power)
-{
-	efx_rc_t rc;
-
-	if (!power)
-		return (0);
-
-	/* Check if the PHY is a zombie */
-	if ((rc = ef10_phy_verify(enp)) != 0)
-		goto fail1;
-
-	enp->en_reset_flags |= EFX_RESET_PHY;
-
-	return (0);
-
-fail1:
-	EFSYS_PROBE1(fail1, efx_rc_t, rc);
-
-	return (rc);
-}
-
-	__checkReturn	efx_rc_t
-ef10_phy_get_link(
-	__in		efx_nic_t *enp,
-	__out		ef10_link_state_t *elsp)
-{
-	efx_mcdi_req_t req;
-	uint8_t payload[MAX(MC_CMD_GET_LINK_IN_LEN,
-			    MC_CMD_GET_LINK_OUT_LEN)];
-	efx_rc_t rc;
-
-	(void) memset(payload, 0, sizeof (payload));
-	req.emr_cmd = MC_CMD_GET_LINK;
-	req.emr_in_buf = payload;
-	req.emr_in_length = MC_CMD_GET_LINK_IN_LEN;
-	req.emr_out_buf = payload;
-	req.emr_out_length = MC_CMD_GET_LINK_OUT_LEN;
-
-	efx_mcdi_execute(enp, &req);
-
-	if (req.emr_rc != 0) {
-		rc = req.emr_rc;
-		goto fail1;
-	}
-
-	if (req.emr_out_length_used < MC_CMD_GET_LINK_OUT_LEN) {
-		rc = EMSGSIZE;
-		goto fail2;
-	}
-
-	mcdi_phy_decode_cap(MCDI_OUT_DWORD(req, GET_LINK_OUT_CAP),
-			    &elsp->els_adv_cap_mask);
-	mcdi_phy_decode_cap(MCDI_OUT_DWORD(req, GET_LINK_OUT_LP_CAP),
-			    &elsp->els_lp_cap_mask);
-
-	mcdi_phy_decode_link_mode(enp, MCDI_OUT_DWORD(req, GET_LINK_OUT_FLAGS),
-			    MCDI_OUT_DWORD(req, GET_LINK_OUT_LINK_SPEED),
-			    MCDI_OUT_DWORD(req, GET_LINK_OUT_FCNTL),
-			    &elsp->els_link_mode, &elsp->els_fcntl);
-
-#if EFSYS_OPT_LOOPBACK
-	/* Assert the MC_CMD_LOOPBACK and EFX_LOOPBACK namespace agree */
-	EFX_STATIC_ASSERT(MC_CMD_LOOPBACK_NONE == EFX_LOOPBACK_OFF);
-	EFX_STATIC_ASSERT(MC_CMD_LOOPBACK_DATA == EFX_LOOPBACK_DATA);
-	EFX_STATIC_ASSERT(MC_CMD_LOOPBACK_GMAC == EFX_LOOPBACK_GMAC);
-	EFX_STATIC_ASSERT(MC_CMD_LOOPBACK_XGMII == EFX_LOOPBACK_XGMII);
-	EFX_STATIC_ASSERT(MC_CMD_LOOPBACK_XGXS == EFX_LOOPBACK_XGXS);
-	EFX_STATIC_ASSERT(MC_CMD_LOOPBACK_XAUI == EFX_LOOPBACK_XAUI);
-	EFX_STATIC_ASSERT(MC_CMD_LOOPBACK_GMII == EFX_LOOPBACK_GMII);
-	EFX_STATIC_ASSERT(MC_CMD_LOOPBACK_SGMII == EFX_LOOPBACK_SGMII);
-	EFX_STATIC_ASSERT(MC_CMD_LOOPBACK_XGBR == EFX_LOOPBACK_XGBR);
-	EFX_STATIC_ASSERT(MC_CMD_LOOPBACK_XFI == EFX_LOOPBACK_XFI);
-	EFX_STATIC_ASSERT(MC_CMD_LOOPBACK_XAUI_FAR == EFX_LOOPBACK_XAUI_FAR);
-	EFX_STATIC_ASSERT(MC_CMD_LOOPBACK_GMII_FAR == EFX_LOOPBACK_GMII_FAR);
-	EFX_STATIC_ASSERT(MC_CMD_LOOPBACK_SGMII_FAR == EFX_LOOPBACK_SGMII_FAR);
-	EFX_STATIC_ASSERT(MC_CMD_LOOPBACK_XFI_FAR == EFX_LOOPBACK_XFI_FAR);
-	EFX_STATIC_ASSERT(MC_CMD_LOOPBACK_GPHY == EFX_LOOPBACK_GPHY);
-	EFX_STATIC_ASSERT(MC_CMD_LOOPBACK_PHYXS == EFX_LOOPBACK_PHY_XS);
-	EFX_STATIC_ASSERT(MC_CMD_LOOPBACK_PCS == EFX_LOOPBACK_PCS);
-	EFX_STATIC_ASSERT(MC_CMD_LOOPBACK_PMAPMD == EFX_LOOPBACK_PMA_PMD);
-
-	elsp->els_loopback = MCDI_OUT_DWORD(req, GET_LINK_OUT_LOOPBACK_MODE);
-#endif	/* EFSYS_OPT_LOOPBACK */
-
-	elsp->els_mac_up = MCDI_OUT_DWORD(req, GET_LINK_OUT_MAC_FAULT) == 0;
-
-	return (0);
-
-fail2:
-	EFSYS_PROBE(fail2);
-fail1:
-	EFSYS_PROBE1(fail1, efx_rc_t, rc);
-
-	return (rc);
-}
-
-	__checkReturn	efx_rc_t
-ef10_phy_reconfigure(
-	__in		efx_nic_t *enp)
-{
-	efx_nic_cfg_t *encp = &(enp->en_nic_cfg);
-	efx_port_t *epp = &(enp->en_port);
-	efx_mcdi_req_t req;
-	uint8_t payload[MAX(MC_CMD_SET_LINK_IN_LEN,
-			    MC_CMD_SET_LINK_OUT_LEN)];
-	uint32_t cap_mask;
-	unsigned int led_mode;
-	unsigned int speed;
-	efx_rc_t rc;
-
-	if (~encp->enc_func_flags & EFX_NIC_FUNC_LINKCTRL)
-		goto out;
-
-	(void) memset(payload, 0, sizeof (payload));
-	req.emr_cmd = MC_CMD_SET_LINK;
-	req.emr_in_buf = payload;
-	req.emr_in_length = MC_CMD_SET_LINK_IN_LEN;
-	req.emr_out_buf = payload;
-	req.emr_out_length = MC_CMD_SET_LINK_OUT_LEN;
-
-	cap_mask = epp->ep_adv_cap_mask;
-	MCDI_IN_POPULATE_DWORD_10(req, SET_LINK_IN_CAP,
-		PHY_CAP_10HDX, (cap_mask >> EFX_PHY_CAP_10HDX) & 0x1,
-		PHY_CAP_10FDX, (cap_mask >> EFX_PHY_CAP_10FDX) & 0x1,
-		PHY_CAP_100HDX, (cap_mask >> EFX_PHY_CAP_100HDX) & 0x1,
-		PHY_CAP_100FDX, (cap_mask >> EFX_PHY_CAP_100FDX) & 0x1,
-		PHY_CAP_1000HDX, (cap_mask >> EFX_PHY_CAP_1000HDX) & 0x1,
-		PHY_CAP_1000FDX, (cap_mask >> EFX_PHY_CAP_1000FDX) & 0x1,
-		PHY_CAP_10000FDX, (cap_mask >> EFX_PHY_CAP_10000FDX) & 0x1,
-		PHY_CAP_PAUSE, (cap_mask >> EFX_PHY_CAP_PAUSE) & 0x1,
-		PHY_CAP_ASYM, (cap_mask >> EFX_PHY_CAP_ASYM) & 0x1,
-		PHY_CAP_AN, (cap_mask >> EFX_PHY_CAP_AN) & 0x1);
-	/* Too many fields for for POPULATE macros, so insert this afterwards */
-	MCDI_IN_SET_DWORD_FIELD(req, SET_LINK_IN_CAP,
-	    PHY_CAP_40000FDX, (cap_mask >> EFX_PHY_CAP_40000FDX) & 0x1);
-
-#if EFSYS_OPT_LOOPBACK
-	MCDI_IN_SET_DWORD(req, SET_LINK_IN_LOOPBACK_MODE,
-		    epp->ep_loopback_type);
-	switch (epp->ep_loopback_link_mode) {
-	case EFX_LINK_100FDX:
-		speed = 100;
-		break;
-	case EFX_LINK_1000FDX:
-		speed = 1000;
-		break;
-	case EFX_LINK_10000FDX:
-		speed = 10000;
-		break;
-	case EFX_LINK_40000FDX:
-		speed = 40000;
-		break;
-	default:
-		speed = 0;
-	}
-#else
-	MCDI_IN_SET_DWORD(req, SET_LINK_IN_LOOPBACK_MODE, MC_CMD_LOOPBACK_NONE);
-	speed = 0;
-#endif	/* EFSYS_OPT_LOOPBACK */
-	MCDI_IN_SET_DWORD(req, SET_LINK_IN_LOOPBACK_SPEED, speed);
-
-#if EFSYS_OPT_PHY_FLAGS
-	MCDI_IN_SET_DWORD(req, SET_LINK_IN_FLAGS, epp->ep_phy_flags);
-#else
-	MCDI_IN_SET_DWORD(req, SET_LINK_IN_FLAGS, 0);
-#endif	/* EFSYS_OPT_PHY_FLAGS */
-
-	efx_mcdi_execute(enp, &req);
-
-	if (req.emr_rc != 0) {
-		rc = req.emr_rc;
-		goto fail1;
-	}
-
-	/* And set the blink mode */
-	(void) memset(payload, 0, sizeof (payload));
-	req.emr_cmd = MC_CMD_SET_ID_LED;
-	req.emr_in_buf = payload;
-	req.emr_in_length = MC_CMD_SET_ID_LED_IN_LEN;
-	req.emr_out_buf = payload;
-	req.emr_out_length = MC_CMD_SET_ID_LED_OUT_LEN;
-
-#if EFSYS_OPT_PHY_LED_CONTROL
-	switch (epp->ep_phy_led_mode) {
-	case EFX_PHY_LED_DEFAULT:
-		led_mode = MC_CMD_LED_DEFAULT;
-		break;
-	case EFX_PHY_LED_OFF:
-		led_mode = MC_CMD_LED_OFF;
-		break;
-	case EFX_PHY_LED_ON:
-		led_mode = MC_CMD_LED_ON;
-		break;
-	default:
-		EFSYS_ASSERT(0);
-		led_mode = MC_CMD_LED_DEFAULT;
-	}
-
-	MCDI_IN_SET_DWORD(req, SET_ID_LED_IN_STATE, led_mode);
-#else
-	MCDI_IN_SET_DWORD(req, SET_ID_LED_IN_STATE, MC_CMD_LED_DEFAULT);
-#endif	/* EFSYS_OPT_PHY_LED_CONTROL */
-
-	efx_mcdi_execute(enp, &req);
-
-	if (req.emr_rc != 0) {
-		rc = req.emr_rc;
-		goto fail2;
-	}
-out:
-	return (0);
-
-fail2:
-	EFSYS_PROBE(fail2);
-fail1:
-	EFSYS_PROBE1(fail1, efx_rc_t, rc);
-
-	return (rc);
-}
-
-	__checkReturn	efx_rc_t
-ef10_phy_verify(
-	__in		efx_nic_t *enp)
-{
-	efx_mcdi_req_t req;
-	uint8_t payload[MAX(MC_CMD_GET_PHY_STATE_IN_LEN,
-			    MC_CMD_GET_PHY_STATE_OUT_LEN)];
-	uint32_t state;
-	efx_rc_t rc;
-
-	(void) memset(payload, 0, sizeof (payload));
-	req.emr_cmd = MC_CMD_GET_PHY_STATE;
-	req.emr_in_buf = payload;
-	req.emr_in_length = MC_CMD_GET_PHY_STATE_IN_LEN;
-	req.emr_out_buf = payload;
-	req.emr_out_length = MC_CMD_GET_PHY_STATE_OUT_LEN;
-
-	efx_mcdi_execute(enp, &req);
-
-	if (req.emr_rc != 0) {
-		rc = req.emr_rc;
-		goto fail1;
-	}
-
-	if (req.emr_out_length_used < MC_CMD_GET_PHY_STATE_OUT_LEN) {
-		rc = EMSGSIZE;
-		goto fail2;
-	}
-
-	state = MCDI_OUT_DWORD(req, GET_PHY_STATE_OUT_STATE);
-	if (state != MC_CMD_PHY_STATE_OK) {
-		if (state != MC_CMD_PHY_STATE_ZOMBIE)
-			EFSYS_PROBE1(mc_pcol_error, int, state);
-		rc = ENOTACTIVE;
-		goto fail3;
-	}
-
-	return (0);
-
-fail3:
-	EFSYS_PROBE(fail3);
-fail2:
-	EFSYS_PROBE(fail2);
-fail1:
-	EFSYS_PROBE1(fail1, efx_rc_t, rc);
-
-	return (rc);
-}
-
-	__checkReturn	efx_rc_t
-ef10_phy_oui_get(
-	__in		efx_nic_t *enp,
-	__out		uint32_t *ouip)
-{
-	_NOTE(ARGUNUSED(enp, ouip))
-
-	return (ENOTSUP);
-}
-
-#if EFSYS_OPT_PHY_STATS
-
-	__checkReturn				efx_rc_t
-ef10_phy_stats_update(
-	__in					efx_nic_t *enp,
-	__in					efsys_mem_t *esmp,
-	__inout_ecount(EFX_PHY_NSTATS)		uint32_t *stat)
-{
-	/* TBD: no stats support in firmware yet */
-	_NOTE(ARGUNUSED(enp, esmp))
-	memset(stat, 0, EFX_PHY_NSTATS * sizeof (*stat));
-
-	return (0);
-}
-
-#endif	/* EFSYS_OPT_PHY_STATS */
-
-#if EFSYS_OPT_PHY_PROPS
-
-#if EFSYS_OPT_NAMES
-
-		const char *
-ef10_phy_prop_name(
-	__in	efx_nic_t *enp,
-	__in	unsigned int id)
-{
-	_NOTE(ARGUNUSED(enp, id))
-
-	return (NULL);
-}
-
-#endif	/* EFSYS_OPT_NAMES */
-
-	__checkReturn	efx_rc_t
-ef10_phy_prop_get(
-	__in		efx_nic_t *enp,
-	__in		unsigned int id,
-	__in		uint32_t flags,
-	__out		uint32_t *valp)
-{
-	_NOTE(ARGUNUSED(enp, id, flags, valp))
-
-	return (ENOTSUP);
-}
-
-	__checkReturn	efx_rc_t
-ef10_phy_prop_set(
-	__in		efx_nic_t *enp,
-	__in		unsigned int id,
-	__in		uint32_t val)
-{
-	_NOTE(ARGUNUSED(enp, id, val))
-
-	return (ENOTSUP);
-}
-
-#endif	/* EFSYS_OPT_PHY_PROPS */
-
 #if EFSYS_OPT_BIST
 
 	__checkReturn		efx_rc_t

Modified: head/sys/modules/sfxge/Makefile
==============================================================================
--- head/sys/modules/sfxge/Makefile	Fri May 13 07:04:28 2016	(r299602)
+++ head/sys/modules/sfxge/Makefile	Fri May 13 07:05:40 2016	(r299603)
@@ -30,7 +30,7 @@ SRCS+=	siena_sram.c siena_vpd.c
 SRCS+=	siena_flash.h siena_impl.h
 
 SRCS+=	ef10_ev.c ef10_filter.c ef10_intr.c ef10_mac.c ef10_mcdi.c ef10_nic.c
-SRCS+=	ef10_nvram.c
+SRCS+=	ef10_nvram.c ef10_phy.c
 SRCS+=	ef10_impl.h
 
 SRCS+=	hunt_nic.c



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201605130705.u4D75e5S037012>