Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
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 v2 1/4] wilc1000: use correct sequence of RESET for chip
 Power-UP/Down
Thread-Topic: [PATCH v2 1/4] wilc1000: use correct sequence of RESET for chip
 Power-UP/Down
Thread-Index: AQHYbq6p2rJ8Qd5opkWdRD9tXlXgEA==
Date:   Mon, 23 May 2022 14:09:06 +0000
Message-ID: <20220523140812.164028-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?fEDqh0K1yVF68OpH2iFX2j4FMN54Mz0AGqtXfsHvmvNUeOjalIm9+rzu1i?=
 =?iso-8859-1?Q?EvlGq7LEWMzPMWK4sPIfe+hVXJPFPCodJcZ9ZBCNVqSB7Pua+R9DbS/N5O?=
 =?iso-8859-1?Q?2QfCYwI4hZ4ST7BDaWeBJec1f8hM855PXUreM/OnT4xiDs5kP1jqtQGpDh?=
 =?iso-8859-1?Q?ceT0YP3syZTQiXRnYMpGRQ5XmB7CBfSKgg0GeXFDLA2gnS0V1iOi9XxRNs?=
 =?iso-8859-1?Q?fJWo5+gRu2+7Xp6ZLCC6+8VQ3q9LnhKTnSRhHxoELrvo1a7J0vDbbVahRD?=
 =?iso-8859-1?Q?cA3hM/QAUm+ROuLmhPXHHEnLQaHI1uddIk5ffO/aC1tm4ZkZ0kq4qbLlFV?=
 =?iso-8859-1?Q?ve7UQeXwRMzlwrR55ZOs0+mXn0JskYC2FFNnMVBJBtZeKaKDN88LHPRzuL?=
 =?iso-8859-1?Q?KH8zpkdWuEfX0i3ukrxSeqdXzOhWmH1/0rZXMX2Z4U/BS9epndtgfB6IFb?=
 =?iso-8859-1?Q?uqgM8J/IT+HvZ1u2VCuHjCe+mLtJ1/THETrfyA61krbd+qSLcUaGXRjMRA?=
 =?iso-8859-1?Q?nVdvonDL+SGXwKwpD58hwc6RfS05BhQV5EFfSu9o1njDqZSMezzsicvGay?=
 =?iso-8859-1?Q?bfqogag7ZqfVmI3idSeSv6fhfTHsr8VyS703gEZ2ssv54x23WMf9SFjWMW?=
 =?iso-8859-1?Q?4ATSJ7P7xrsRbR44CPOeaLgCvqfJAGV+uQVw/4Zb3gofEYHe2KuvdnIkPl?=
 =?iso-8859-1?Q?MV7+kVqB/nOKhNFsYAIeFGtSr4xkOePXqGq9hOBRF9z4kIJv2sTSiNGpPS?=
 =?iso-8859-1?Q?NblNHDltcw5tV9m9DRkKjgwLEViTyJzLOy6AbB1NDNkwBDDkO6JgSOrnli?=
 =?iso-8859-1?Q?5b7ZksUdE7xbUjaIkG9cj4QhVv9aOiIiWnvVWkBKFAceSPLLpuJTRyLRVo?=
 =?iso-8859-1?Q?xkjlc4E1c+Gd1rwRcjsA2iVAEiT3zZwRDigmEJmTGUUmdFZm7RqCeiZtrv?=
 =?iso-8859-1?Q?46JaYTGKzBnYNYz4Y7qA8yF3EO5fTfSaJ5QYTm4yJds0EHvyBeyDeMJQza?=
 =?iso-8859-1?Q?cfNTNgxiaYFDjCK6MCycrI/+/ZhpBHgovqz5QY7gyWSl8XRYi/+JL2ehKj?=
 =?iso-8859-1?Q?E2wdN5lgHHWHuNCKVDAedn9nGyOKJjlly/vnmsbEPA5DDO4hKNYeJD2r+B?=
 =?iso-8859-1?Q?SZE9/KWLbxlvUh8MnEL+41RZxMN2cMnIAydjCnWM3MEq1oFQDIv8RAIW3H?=
 =?iso-8859-1?Q?YluYjPFxCVmtLFzlC8okYoAh08EjBKIRXdPIpgpkbt719NatO7H9bM89E+?=
 =?iso-8859-1?Q?qS4qNLAn4TqgbOpPTZsFvc60B8IFwxzzzRuzleVFUwQRCu4oYape41OxTt?=
 =?iso-8859-1?Q?J/eQWNvGH1DvDUaMt9ttKDTxXAx+Hny4vcRNMU4YnrZ6BZvt7+HqJAPe8R?=
 =?iso-8859-1?Q?AXjRWbzVVV1gqKeyGTkfc7veZpDppLAxeR5wQInzPS19i/yvEdhSQi+6a4?=
 =?iso-8859-1?Q?74+3dz1C53z72qH1zV3zHxQI3/TVBnTJWuwcxj5jCmKw7O26qzkL34PzDK?=
 =?iso-8859-1?Q?5JcqOk0O0ccO7BpnC3dLIgxVu3cRL3pETa63bAeAsx/iAZLaD9GZJa5XwQ?=
 =?iso-8859-1?Q?CO60wMQNsRQJ7Ku81tGmsu75MP64XXj2vb5om9kBiFwSjWPPrH2Qroby75?=
 =?iso-8859-1?Q?bTYwWHDQsUiNTQnloGBLM9POVGNrc+TxddrjDLh0UVLPiVtp2dwJT5SsPb?=
 =?iso-8859-1?Q?VFT8JkY/7ONKfdiQyte9oBngsDqRvL1KCkt+LlNjEsrytfRmYCpPeu+pkZ?=
 =?iso-8859-1?Q?el7UPjJWqO1f3WsFdogXD5eTgcYl60fGCixH3qegN+e06t1nLQuIvWcfUF?=
 =?iso-8859-1?Q?J0Zop3pvWpGc2hnP+YAVg+hJLX+hqFACe8cASjTG22YE7CZuic8P?=
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: 34c7ce09-53c3-4b29-616c-08da3cc5cc6b
X-MS-Exchange-CrossTenant-originalarrivaltime: 23 May 2022 14:09:06.2003
 (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: qOxJAcP0HDF2jOwo59HjFgmf4SArU9LLtsmTTY+/IHoGG5fhIgoaYAeP1/34jTlp6rT79R/kxsKHQ2P0dOYt/8/6AS7WZt1brV08Nx9wa0Q=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR11MB5600
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED,
	DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,RDNS_NONE,
	SPF_HELO_NONE,T_SCC_BODY_TEXT_LINE autolearn=no 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.

Signed-off-by: Ajay Singh <ajay.kathat@microchip.com>
---
 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