tg3: Don't send APE events for NCSI firmware

NCSI firmware does not accept APE events.  It relies on a "driver state"
location in shared memory to tell it what the driver's current state is.

This patch pivots the code to use the new driver state scheme.

Reviewed-by: Benjamin Li <benli@broadcom.com>
Reviewed-by: Michael Chan <mchan@broadcom.com>
Signed-off-by: Matt Carlson <mcarlson@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

1 files changed, 20 insertions, 2 deletions

diff --git a/drivers/net/tg3.c b/drivers/net/tg3.c
index 38f1b19b776b..fc9706c8e3c6 100644
--- a/drivers/net/tg3.c
+++ b/drivers/net/tg3.c
@@ -6596,6 +6596,10 @@ static void tg3_ape_send_event(struct tg3 *tp, u32 event)
         int i;
         u32 apedata;
 
+        /* NCSI does not support APE events */
+        if (tp->tg3_flags3 & TG3_FLG3_APE_HAS_NCSI)
+                return;
+
         apedata = tg3_ape_read32(tp, TG3_APE_SEG_SIG);
         if (apedata != APE_SEG_SIG_MAGIC)
                 return;
@@ -6647,6 +6651,8 @@ static void tg3_ape_driver_state_change(struct tg3 *tp, int kind)
                         APE_HOST_DRIVER_ID_MAGIC(TG3_MAJ_NUM, TG3_MIN_NUM));
                 tg3_ape_write32(tp, TG3_APE_HOST_BEHAVIOR,
                                 APE_HOST_BEHAV_NO_PHYLOCK);
+                tg3_ape_write32(tp, TG3_APE_HOST_DRVR_STATE,
+                                    TG3_APE_HOST_DRVR_STATE_START);
 
                 event = APE_EVENT_STATUS_STATE_START;
                 break;
@@ -6658,6 +6664,16 @@ static void tg3_ape_driver_state_change(struct tg3 *tp, int kind)
                  */
                 tg3_ape_write32(tp, TG3_APE_HOST_SEG_SIG, 0x0);
 
+                if (device_may_wakeup(&tp->pdev->dev) &&
+                    (tp->tg3_flags & TG3_FLAG_WOL_ENABLE)) {
+                        tg3_ape_write32(tp, TG3_APE_HOST_WOL_SPEED,
+                                            TG3_APE_HOST_WOL_SPEED_AUTO);
+                        apedata = TG3_APE_HOST_DRVR_STATE_WOL;
+                } else
+                        apedata = TG3_APE_HOST_DRVR_STATE_UNLOAD;
+
+                tg3_ape_write32(tp, TG3_APE_HOST_DRVR_STATE, apedata);
+
                 event = APE_EVENT_STATUS_STATE_UNLOAD;
                 break;
         case RESET_KIND_SUSPEND:
@@ -12739,10 +12755,12 @@ static void __devinit tg3_read_dash_ver(struct tg3 *tp)
 
         apedata = tg3_ape_read32(tp, TG3_APE_FW_VERSION);
 
-        if (tg3_ape_read32(tp, TG3_APE_FW_FEATURES) & TG3_APE_FW_FEATURE_NCSI)
+        if (tg3_ape_read32(tp, TG3_APE_FW_FEATURES) & TG3_APE_FW_FEATURE_NCSI) {
+                tp->tg3_flags3 |= TG3_FLG3_APE_HAS_NCSI;
                 fwtype = "NCSI";
-        else
+        } else {
                 fwtype = "DASH";
+        }
 
         vlen = strlen(tp->fw_ver);