Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   <Ajay.Kathat@microchip.com>
To:     <linux-wireless@vger.kernel.org>
CC:     <Claudiu.Beznea@microchip.com>, <Sripad.Balwadgi@microchip.com>,
        <Ajay.Kathat@microchip.com>
Subject: [PATCH v3 1/4] wilc1000: use correct sequence of RESET for chip
 Power-UP/Down
Thread-Topic: [PATCH v3 1/4] wilc1000: use correct sequence of RESET for chip
 Power-UP/Down
Thread-Index: AQHYb2atM/50sN/76EaQWN5Ft5lV2g==
Date:   Tue, 24 May 2022 12:06:19 +0000
Message-ID: <20220524120606.9675-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?cec2utwFbtCjUsoJSdWsZ+UWPZlPXTVKB45i0kkw8Lv57IlhXMRk7cU0Tl?=
 =?iso-8859-1?Q?GnyEJbp9qmaN9fi0JBK7TEzhyMi037XFtP5iXsbv9dHvbTsX5gK/fttmt8?=
 =?iso-8859-1?Q?jSLcnDlv6Hh4RG4E4bAFRM1nym8u42gGAJGdhshfLUUwrY+r2PjbHHQIz7?=
 =?iso-8859-1?Q?oFHF5Sg/PhPYG8/MJF7OczWtui3tTsOXR6pf5MDta4R3aBCoERGLBXcN+s?=
 =?iso-8859-1?Q?ytF+jWeRG7SFkylJVw6IYBh0bNQtd0zzunhHJ6YT8O9ONZUXE/wTgw6dxZ?=
 =?iso-8859-1?Q?SKehiKa2JRVtVYMmcwWFMGt7ZJ+/AMiPnETtzay08L53wXYVoFo34UAEtO?=
 =?iso-8859-1?Q?SDj5Bl9kE4qoAWCVMr/vlA4fGVo/2hDWruMkSXir/mI6P4mAEQK0gKstkT?=
 =?iso-8859-1?Q?ipjIiRHZSI0D6MZAXk+i/7NSWMLSmcaGxXKTrAegLO1am0cnhkYgCl+v/P?=
 =?iso-8859-1?Q?qCLYWaUjGBeGm5nj72ktBYRSe6nP88hvZIu5VssQFo/opeS84vPSq/frbu?=
 =?iso-8859-1?Q?EKtE7PzOjY35TffifMYof/tRL9wFTmzLbh2E3BWHCTWh7u3LZA4H8vJFn+?=
 =?iso-8859-1?Q?gkpof3h1fH8rnFBNna5ZfxBgNODvXiIKwO0KVD7xHHqL0UmVFLuDH/qkQH?=
 =?iso-8859-1?Q?tP/s6fAuhH/kB7JcrLwMfHm0rUOcB4mm3o3hdJ6pt4XmtpAMKT3j8L9g/C?=
 =?iso-8859-1?Q?UMmxpFIQNWyfYygxaRnGWlnlXDgjffjIMAsPRITHH2i2drFslEpmpkuDoG?=
 =?iso-8859-1?Q?uilbGl8oinveLr2jUHbfZ/85WHAwtLhiOadgpwIZzSlkVAETZDTRAZ5KB/?=
 =?iso-8859-1?Q?jZCsSf9iVygupvLcPD/yAUKrS0WeSe7bvRC1/qv1SFlYvh/TGKcJpYwsnQ?=
 =?iso-8859-1?Q?5zsHb5IpEkuFnrRNitIU1nqFp37IAgXOnizS5WbawDZI439/qr7yuKa/sF?=
 =?iso-8859-1?Q?yfeGbjOButmOtfHGQ9CY2sISRpBT6l7jHSnlwrnvHOdlBOp1NTMvY8TOf7?=
 =?iso-8859-1?Q?9Yf1O/c9lQke34Jp9Zd4Alhhbi/ogRFG/IJALo1E5B3rcB8e0Oe/4xFy5W?=
 =?iso-8859-1?Q?XanL+CT1V+S0xT+mRZjXEwpuLUXY9N5AA8hgfh76bnas/QLBujg+KihXPF?=
 =?iso-8859-1?Q?JtpINlnnNQPzEZhe4P6ScZi1UD4GQvdfeYM/rntZzJk9i1N2KARYv7Hdpr?=
 =?iso-8859-1?Q?VZ8VTYwJflmWRY5TFeCDGpa1sft7yfXbdegQsvb0cWVf9hbW8nlFMiU6n6?=
 =?iso-8859-1?Q?KVqQ9DduP0SbsD458YVip31CzlI+Kc1l1Zq3A7IOhHTGkzOvUu6rIHeiFP?=
 =?iso-8859-1?Q?pxyo+euYuHzxqjIg1H+sDfPx2Y7/g7m/dOAsdER0yZlDBYIFXs8iaVfYQH?=
 =?iso-8859-1?Q?nlDl9AmHIXmT5S3arC65RJm1a1E9U/sL0fyswNHYgk43ilNNcwGY6qou39?=
 =?iso-8859-1?Q?VruPbmHTwpAISQgFaKHpWCYuZHETa3E54xjyPmVJoCp9PoVRYwKVMquiUm?=
 =?iso-8859-1?Q?QIBiE+PwvCGF5n1iqENjjAVwpeuvhZFJi97SPeaGnQ3yM/+tAV+MPwvlFh?=
 =?iso-8859-1?Q?gLlAZDIk9EgDVTudeRQidBcnAMYueMatHZImzIkoFnjC34vYeoyn+0rIOZ?=
 =?iso-8859-1?Q?DDKuHtssTBDNe/CzdXVPMVfxz6rP15o+88tU7ek+YMKHf2o83lyLM28R4j?=
 =?iso-8859-1?Q?ydx+NBMgF+BdkwQMU9Q77GTnDvQRRVp4uorUPupggz1L+mY/TFgDFp/X5q?=
 =?iso-8859-1?Q?iFRtwo+yLC8CLwl+faFfXg2JwxRdGVo5RiAo5iG9/KEbdsrndX8eXECQYm?=
 =?iso-8859-1?Q?pO+td+Y4ZNHkLz9hFPHuVGOSc1ki3VX8HxV92y4k+DT5ghD96ctB?=
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: PH0PR11MB5176.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b8fd5642-7702-4399-17bd-08da3d7dd001
X-MS-Exchange-CrossTenant-originalarrivaltime: 24 May 2022 12:06:19.5972
 (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: u6eU3P5sM/OAJe5AU7wgi3l44KpMeTZnAqpgmdsyYWNFd6dWrjD3CbiXbrXE8Skv8nYxVUxov3aveB64D9rjXsyT0vai20pn1cvjC4kbXx0=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH0PR11MB5412
X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,SPF_HELO_PASS,
        SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no
        version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-wireless.vger.kernel.org>
X-Mailing-List: linux-wireless@vger.kernel.org

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

For power-up sequence, WILC expects RESET set to high 5ms after making
chip_en(enable) so corrected chip power-up sequence by making RESET high.
For Power-Down sequence, the correct sequence make RESET and CHIP_EN low
without any extra delay.

Fixes: ec031ac4792c ("wilc1000: Add reset/enable GPIO support to SPI driver=
")
Signed-off-by: Ajay Singh <ajay.kathat@microchip.com>
---

 v2: No changes

 v3: add "Fixes" tag in commit description
=20
 drivers/net/wireless/microchip/wilc1000/spi.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/net/wireless/microchip/wilc1000/spi.c b/drivers/net/wi=
reless/microchip/wilc1000/spi.c
index 18420e954402..2ae8dd3411ac 100644
--- a/drivers/net/wireless/microchip/wilc1000/spi.c
+++ b/drivers/net/wireless/microchip/wilc1000/spi.c
@@ -191,11 +191,11 @@ static void wilc_wlan_power(struct wilc *wilc, bool o=
n)
 		/* assert ENABLE: */
 		gpiod_set_value(gpios->enable, 1);
 		mdelay(5);
-		/* deassert RESET: */
-		gpiod_set_value(gpios->reset, 0);
-	} else {
 		/* assert RESET: */
 		gpiod_set_value(gpios->reset, 1);
+	} else {
+		/* deassert RESET: */
+		gpiod_set_value(gpios->reset, 0);
 		/* deassert ENABLE: */
 		gpiod_set_value(gpios->enable, 0);
 	}
--=20
2.25.1