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: QzLtriVhQP0ELAdEbSe85xAVbiApWJx1M/07qCcxMiydRvnka9ZIq0EIrjpHFhQpRV6ismss0U
 /8BHglTyzvhFmOhUbQJhcQqnELUu+1T5y6/BHcoii58sduMopgDK2dTwS+IWkC+p6Ox3sSJw4h
 7TsDZqABWRjpfFBiLHOY7ecJC8rgUnPa5wS6wRnjkgt//CGsCULXdp3XSr0zgkulWBNgaWX5dm
 iILedT59isHpjsFtkc7ahw69TV6L1v+xODkoPclEz8ojgyO9imHZU1/jDrIPkhtUuqDZHBYuHE
 msgAum7uixchZy/Zsigs0/7E
From:   <Ajay.Kathat@microchip.com>
To:     <linux-wireless@vger.kernel.org>
CC:     <Claudiu.Beznea@microchip.com>, <Ajay.Kathat@microchip.com>
Subject: [PATCH v2 10/11] wilc1000: use correct write command sequence in
 wilc_spi_sync_ext()
Thread-Topic: [PATCH v2 10/11] wilc1000: use correct write command sequence in
 wilc_spi_sync_ext()
Thread-Index: AQHXqxrMFSlUj9OZo0avjyjHukVIrg==
Date:   Thu, 16 Sep 2021 16:49:22 +0000
Message-ID: <20210916164902.74629-11-ajay.kathat@microchip.com>
References: <20210916164902.74629-1-ajay.kathat@microchip.com>
In-Reply-To: <20210916164902.74629-1-ajay.kathat@microchip.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?iso-8859-1?Q?aMpBubWmYroO9c1uhIdHnd2f2fTP+G7V/kv6a23o8kvc/aiKhv2E1ZB3Tp?=
 =?iso-8859-1?Q?2GTNWQr+JUy8/sWPiBZVHvt7BcaAAk+IHA7rdfWL4AcbuouFauJ9vy4dF7?=
 =?iso-8859-1?Q?2TEh0VU/xP2Phg6NsqRecSPi+du3Bv3IhVQc0AmCd5saiIXUq0CMaqLsr9?=
 =?iso-8859-1?Q?Ci7O/+5IT+lQ2IK34iqqdSMgFWU6RZQCqZb3UFagb1AEg9bX3eoHmc8U3s?=
 =?iso-8859-1?Q?bay4mW0HWbvj/osjmOmTfmUlbMKTeVAU6OvmNvvt/UqwQnso/Yk0bkyTZ7?=
 =?iso-8859-1?Q?B+VauiTTmqKJQ4lbjCO1kx4W9A/dVEuZ/ynH7PXbFFf+IQqJCZBfJyVoHe?=
 =?iso-8859-1?Q?USudD8+FJ5X+N8hMdVBIIBmwsXPnI/20av0oDpzRkaao3b0dgRUyLGug4l?=
 =?iso-8859-1?Q?53dJtqdaVCFt9nZBLliTKBVNEe8DChVD+iLrWM/IH/dVyI/AFd9zVoJcG3?=
 =?iso-8859-1?Q?uhX0/5KkrDzv3c6NwaEMAXNEpsdD+HldQUxfL4to39cRcC+b7JycdnrLEv?=
 =?iso-8859-1?Q?9eYgXi5e46edZKzdLzeR2dveCisx60SR5oQM2GDmH7JQFuboUTf9KbOXDj?=
 =?iso-8859-1?Q?CVCfzQwvRew4NvKmS/21j13opYIyH1RkLrF/LyWIXrpxn90UOi8H8Etx5d?=
 =?iso-8859-1?Q?U4tSv2JWjwtBKZGFsxGlhmY3j0Ef0gXHYSXMS3XDpMbCiZGh5p8hEKQsK9?=
 =?iso-8859-1?Q?MF27WCpuzUQi/rhzjZfW8DbB61v7cMPzIVgir7vh291zFbwC7oNozoJvOL?=
 =?iso-8859-1?Q?cvXftwXtrNXgXYGiD5U7VDCPiqbEkFAQXgvus0La+3oU/fDev7Nx5Hmido?=
 =?iso-8859-1?Q?Wkd/j4neTbqTg4vcDOcDO3d2FpenEwXVGbQZcdrZ0BF9OatJa+gG1j1m6y?=
 =?iso-8859-1?Q?IPh937vKApga1eU7kimDuDP+dNcGViyn4xLJtDDeKtO5BW5zZUYxPyCRss?=
 =?iso-8859-1?Q?DzFl9yyI/rDarzhrAkppWwfL15TAZl4w0udIEwzeoZ1DnNQ8SYnlaOOYQE?=
 =?iso-8859-1?Q?wksK75IlrMPPJx2NbOCBBEpG8mCx8Z8HDPxdi3ge62sWQ4s3A2v3yR+ffJ?=
 =?iso-8859-1?Q?J5SpuC7QJmo9xbU12om4kS80OnpLtVKxyr+eS9wIeL0MfvaibAV8y7cP7n?=
 =?iso-8859-1?Q?GifWdU8raR9vGhwTBnCPoUNm0ym13ZXaDRCUPSJ4293zzjaMTSZoeIlQFy?=
 =?iso-8859-1?Q?D5f7zHsyJzvgwETyDyiBcFpMmCVE7GWOXbtFwssDlhMt8FpVgoDTCzhkdC?=
 =?iso-8859-1?Q?33AjMRpHrTixTAAguMtqvD9zpBnqeohirufx4aFPhpBy7pJ8HmlbjxwSj4?=
 =?iso-8859-1?Q?xXWqsqChdOKSeBWYnlgPrd1v5iM8lKLCKV+jPcovMwkcDhiL5wUtXe+HOr?=
 =?iso-8859-1?Q?0VEEO0HLBh?=
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: SJ0PR11MB4943.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 48e4f2e8-bc41-418b-3655-08d97931f4bd
X-MS-Exchange-CrossTenant-originalarrivaltime: 16 Sep 2021 16:49:22.5173
 (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: nL+sGdJcfuoW1DDbC2V3T2fdej1Osr+2Xczbkrl0AU5VrEi1cBbyo9Imwr2MrIF3+KLHrRXRtBsdK+fcZR437JIgPkEbLQIy/04y+Da4Zyo=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY5PR11MB3990
Precedence: bulk
List-ID: <linux-wireless.vger.kernel.org>
X-Mailing-List: linux-wireless@vger.kernel.org

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

Instead of using double read for the same register, use the write register
command after the read command.
The correct sequence is to use the read value in write command instead of
reading the same register again.

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

diff --git a/drivers/net/wireless/microchip/wilc1000/spi.c b/drivers/net/wi=
reless/microchip/wilc1000/spi.c
index 4f0bc24d2a78..640850f989dd 100644
--- a/drivers/net/wireless/microchip/wilc1000/spi.c
+++ b/drivers/net/wireless/microchip/wilc1000/spi.c
@@ -1167,7 +1167,7 @@ static int wilc_spi_sync_ext(struct wilc *wilc, int n=
int)
 		for (i =3D 0; (i < 3) && (nint > 0); i++, nint--)
 			reg |=3D BIT(i);
=20
-		ret =3D wilc_spi_read_reg(wilc, WILC_INTR2_ENABLE, &reg);
+		ret =3D wilc_spi_write_reg(wilc, WILC_INTR2_ENABLE, reg);
 		if (ret) {
 			dev_err(&spi->dev, "Failed write reg (%08x)...\n",
 				WILC_INTR2_ENABLE);
--=20
2.25.1