NTB: remove handling of peer_sta from amd_link_is_up
authorArindam Nath <arindam.nath@amd.com>
Wed, 5 Feb 2020 15:54:24 +0000 (21:24 +0530)
committerJon Mason <jdmason@kudzu.us>
Fri, 13 Mar 2020 14:04:20 +0000 (10:04 -0400)
amd_link_is_up() is a callback to inquire whether
the NTB link is up or not. So it should not indulge
itself into clearing the bitmasks of peer_sta.

Signed-off-by: Arindam Nath <arindam.nath@amd.com>
Signed-off-by: Jon Mason <jdmason@kudzu.us>
drivers/ntb/hw/amd/ntb_hw_amd.c

index b85af15..e964442 100644 (file)
@@ -253,17 +253,6 @@ static int amd_link_is_up(struct amd_ntb_dev *ndev)
                return 1;
        }
 
-       /* If peer_sta is reset or D0 event, the ISR has
-        * started a timer to check link status of hardware.
-        * So here just clear status bit. And if peer_sta is
-        * D3 or PME_TO, D0/reset event will be happened when
-        * system wakeup/poweron, so do nothing here.
-        */
-       if (ndev->peer_sta & AMD_PEER_RESET_EVENT)
-               ndev->peer_sta &= ~AMD_PEER_RESET_EVENT;
-       else if (ndev->peer_sta & (AMD_PEER_D0_EVENT | AMD_LINK_DOWN_EVENT))
-               ndev->peer_sta = 0;
-
        return 0;
 }