diff options
author | Arindam Nath <arindam.nath@amd.com> | 2020-02-05 16:54:26 +0100 |
---|---|---|
committer | Jon Mason <jdmason@kudzu.us> | 2020-03-13 15:04:20 +0100 |
commit | 673dd0c24779fa9233fd600555132b5ffc1e19e4 (patch) | |
tree | ecc391355cb2fc0b37f77dd35d4e9736e303876e | |
parent | NTB: handle link down event correctly (diff) | |
download | linux-673dd0c24779fa9233fd600555132b5ffc1e19e4.tar.xz linux-673dd0c24779fa9233fd600555132b5ffc1e19e4.zip |
NTB: handle link up, D0 and D3 events correctly
Just like for Link-Down event, Link-Up and D3 events
are also mutually exclusive to Link-Down and D0 events
respectively. So we clear the bitmasks in peer_sta
depending on event type.
Signed-off-by: Arindam Nath <arindam.nath@amd.com>
Signed-off-by: Jon Mason <jdmason@kudzu.us>
-rw-r--r-- | drivers/ntb/hw/amd/ntb_hw_amd.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/drivers/ntb/hw/amd/ntb_hw_amd.c b/drivers/ntb/hw/amd/ntb_hw_amd.c index d933a1dffdc6..a1c4a21c58c3 100644 --- a/drivers/ntb/hw/amd/ntb_hw_amd.c +++ b/drivers/ntb/hw/amd/ntb_hw_amd.c @@ -568,6 +568,11 @@ static void amd_handle_event(struct amd_ntb_dev *ndev, int vec) case AMD_PEER_PMETO_EVENT: case AMD_LINK_UP_EVENT: ndev->peer_sta |= status; + if (status == AMD_LINK_UP_EVENT) + ndev->peer_sta &= ~AMD_LINK_DOWN_EVENT; + else if (status == AMD_PEER_D3_EVENT) + ndev->peer_sta &= ~AMD_PEER_D0_EVENT; + amd_ack_smu(ndev, status); /* link down */ @@ -582,6 +587,7 @@ static void amd_handle_event(struct amd_ntb_dev *ndev, int vec) dev_info(dev, "Wakeup is done.\n"); ndev->peer_sta |= AMD_PEER_D0_EVENT; + ndev->peer_sta &= ~AMD_PEER_D3_EVENT; amd_ack_smu(ndev, AMD_PEER_D0_EVENT); /* start a timer to poll link status */ |