Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: sohzFpREqIwKpH9q3aDfqWXmj0ecrl1d4ycRrRmrgfulM88Zcp5wYRLrEAxC2uCyxxvEQvA13d
 ymDZcA4AKn6eUnvW55g3/BiWW3KE+frYjQHAXMkcdGMlB70yRJowq149y75B2A7MizykkTq4BB
 wEX3Ugq7nc+8PeBhycANf6q/Bknml6+i6pQdy56KGg9mHtgMYR4HQ8/IrkbCSBsDDIxogJTIBO
 AhtD/mGSa5jwm3wPI4BpsQMovUWAbbWSFmHaCCEE9GVHACuhs6nHPXHba/NNRFgBu+Po635WZt
 p74=
From:   <Ajay.Kathat@microchip.com>
To:     <linux-wireless@vger.kernel.org>, <kvalo@codeaurora.org>
CC:     <Claudiu.Beznea@microchip.com>, <Sripad.Balwadgi@microchip.com>,
        <Ajay.Kathat@microchip.com>
Subject: [PATCH] wilc1000: changes for SPI communication stall issue found
 with Iperf
Thread-Topic: [PATCH] wilc1000: changes for SPI communication stall issue
 found with Iperf
Thread-Index: AQHWzU4z9ZWBhWbUDEWTavke0mxVkQ==
Date:   Tue, 8 Dec 2020 10:38:00 +0000
Message-ID: <20201208103739.28597-1-ajay.kathat@microchip.com>
Accept-Language: en-US
Content-Language: en-US
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 31b61bf0-b323-44e5-67b9-08d89b655619
X-MS-Exchange-CrossTenant-originalarrivaltime: 08 Dec 2020 10:38:01.0353
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 3f4057f3-b418-4d4e-ba84-d55b4e897d88
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: 1sq6Xnzt3k2rzeHi2XO1NiZ1HUA83NcHvB4uV8tuJRQAI6VilP10Eqr6q0S/vNaFA80OIXky7vCE4eW4RU1cjDMpQySEH6vMfAQzWqyC9Z4=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR1101MB2168
Precedence: bulk
List-ID: <linux-wireless.vger.kernel.org>
X-Mailing-List: linux-wireless@vger.kernel.org

From: Ajay Singh <ajay.kathat@microchip.com>

Added retry mechanism to ensure VMM enable bit is set during the
block transfer of data between host and WILC FW.

Signed-off-by: Ajay Singh <ajay.kathat@microchip.com>
---
 drivers/net/wireless/microchip/wilc1000/spi.c | 23 +++++++++++++++++--
 1 file changed, 21 insertions(+), 2 deletions(-)

diff --git a/drivers/net/wireless/microchip/wilc1000/spi.c b/drivers/net/wi=
reless/microchip/wilc1000/spi.c
index a18dac0aa6b6..be732929322c 100644
--- a/drivers/net/wireless/microchip/wilc1000/spi.c
+++ b/drivers/net/wireless/microchip/wilc1000/spi.c
@@ -35,6 +35,7 @@ static const struct wilc_hif_func wilc_hif_spi;
 #define CMD_SINGLE_READ				0xca
 #define CMD_RESET				0xcf
=20
+#define SPI_ENABLE_VMM_RETRY_LIMIT		2
 #define DATA_PKT_SZ_256				256
 #define DATA_PKT_SZ_512				512
 #define DATA_PKT_SZ_1K				1024
@@ -856,8 +857,26 @@ static int wilc_spi_read_int(struct wilc *wilc, u32 *i=
nt_status)
=20
 static int wilc_spi_clear_int_ext(struct wilc *wilc, u32 val)
 {
-	return spi_internal_write(wilc, WILC_SPI_INT_CLEAR - WILC_SPI_REG_BASE,
-				  val);
+	int ret;
+	int retry =3D SPI_ENABLE_VMM_RETRY_LIMIT;
+	u32 check;
+
+	while (retry) {
+		ret =3D spi_internal_write(wilc,
+					 WILC_SPI_INT_CLEAR - WILC_SPI_REG_BASE,
+					 val);
+		if (ret)
+			break;
+
+		ret =3D spi_internal_read(wilc,
+					WILC_SPI_INT_CLEAR - WILC_SPI_REG_BASE,
+					&check);
+		if (ret || ((check & EN_VMM) =3D=3D (val & EN_VMM)))
+			break;
+
+		retry--;
+	}
+	return ret;
 }
=20
 static int wilc_spi_sync_ext(struct wilc *wilc, int nint)
--=20
2.24.0