From patchwork Thu Aug 20 15:58:58 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?b?SsOpcsO0bWUgUG91aWxsZXI=?= X-Patchwork-Id: 1348469 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming-netdev@ozlabs.org Delivered-To: patchwork-incoming-netdev@ozlabs.org Authentication-Results: ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=vger.kernel.org (client-ip=23.128.96.18; helo=vger.kernel.org; envelope-from=netdev-owner@vger.kernel.org; receiver=) Authentication-Results: ozlabs.org; dmarc=none (p=none dis=none) header.from=silabs.com Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=silabs.onmicrosoft.com header.i=@silabs.onmicrosoft.com header.a=rsa-sha256 header.s=selector2-silabs-onmicrosoft-com header.b=K01bQjyv; dkim-atps=neutral Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by ozlabs.org (Postfix) with ESMTP id 4BXTt46n4cz9sR4 for ; Fri, 21 Aug 2020 02:01:48 +1000 (AEST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729413AbgHTQBk (ORCPT ); Thu, 20 Aug 2020 12:01:40 -0400 Received: from mail-bn7nam10on2087.outbound.protection.outlook.com ([40.107.92.87]:47585 "EHLO NAM10-BN7-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1728603AbgHTQAp (ORCPT ); Thu, 20 Aug 2020 12:00:45 -0400 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=gGSruB/KuPLCWnDMq728ZojBRHX+vtBQglzQMtUUTlXJ5k1s7iWbXQl8GKotdaLZ2UA9tPyBopdK61BiEk9SnIwP/3R98f3KGp9+MVkAxd5qsb1M8jioaLlb8FsdfSqGUr520E7vmeaf0QtG/gZ77UI6cI+YhnNtBomAHypIVczBTl8+t4x5vdrf62NrAv/d7cr7ovZOGG+39ZDXBkJ8lR9d85dhVeW9DSwaB/m3w1Fb6FwOjqJaSteY/RWkzQewX93cFmpbxYFvSStEoblh/cKi2dMxdfec5S326VgRpzawAMtSqhBCqaMHoHD4WodJvICPTdGIRJ/G26rMdrbhEg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=vnx1LDN0uuuMoQM1TsWcaSizN1CAv69/PathJt1c3xk=; b=EKYPjxHqIWNHdS9xDYujBZ7d7+NAvHtlKYTK9RDpkBUXNLrFw4rnGNV3jrEv+on++ujhQ3n+RwJ6vPQbF1KkD4RlqxCYXEQ7pgqn3E02Pj+4Da6FqU3Z5/P7BSsxakOtG9sM56vOJg6qBblEDnTukjEIDh1WtknNVn4aFDMTaiABVfbwahFXbPLNr05Q6UkL0bxC76WkGOJzsbKKeM4Bgh/1QKki+kelbu/2nTAaBGQPbBUWw+BgY0AvvocZPVZpjPYgdocrI/3pd2r/hs5SNTgR5uDtq4kuBf/+fQMsbi814Y6x+Ko4/KZFgy/uVw3+Ok+u2FR2tWQYfVoiLiUhcg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=silabs.com; dmarc=pass action=none header.from=silabs.com; dkim=pass header.d=silabs.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=silabs.onmicrosoft.com; s=selector2-silabs-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=vnx1LDN0uuuMoQM1TsWcaSizN1CAv69/PathJt1c3xk=; b=K01bQjyvj1PJ7XAcaRVBIiFGz6xSlQc+7AcM4FxmsAnu+08X5IOkUzNH44IwIlqY2UOixPqdRwt38xU6/21b9g+mbm2QG89Ng7KEEBfradv3PkSjq14wpJ/RSEPIMDoSJWViHF0HkTm5glK28OKHoS7HMWQcp3VRmzVRd0RIZXM= Authentication-Results: driverdev.osuosl.org; dkim=none (message not signed) header.d=none;driverdev.osuosl.org; dmarc=none action=none header.from=silabs.com; Received: from SN6PR11MB2718.namprd11.prod.outlook.com (2603:10b6:805:63::18) by SA0PR11MB4541.namprd11.prod.outlook.com (2603:10b6:806:94::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3305.25; Thu, 20 Aug 2020 15:59:42 +0000 Received: from SN6PR11MB2718.namprd11.prod.outlook.com ([fe80::85c9:1aa9:aeab:3fa6]) by SN6PR11MB2718.namprd11.prod.outlook.com ([fe80::85c9:1aa9:aeab:3fa6%4]) with mapi id 15.20.3305.026; Thu, 20 Aug 2020 15:59:42 +0000 From: Jerome Pouiller To: devel@driverdev.osuosl.org, linux-wireless@vger.kernel.org Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Greg Kroah-Hartman , Kalle Valo , "David S . Miller" , =?utf-8?b?SsOpcsO0bWUgUG91aWxsZXI=?= Subject: [PATCH 12/12] staging: wfx: add workaround for 'timeout while wake up chip' Date: Thu, 20 Aug 2020 17:58:58 +0200 Message-Id: <20200820155858.351292-12-Jerome.Pouiller@silabs.com> X-Mailer: git-send-email 2.28.0 In-Reply-To: <20200820155858.351292-1-Jerome.Pouiller@silabs.com> References: <20200820155858.351292-1-Jerome.Pouiller@silabs.com> X-ClientProxiedBy: PR0P264CA0122.FRAP264.PROD.OUTLOOK.COM (2603:10a6:100:1a::14) To SN6PR11MB2718.namprd11.prod.outlook.com (2603:10b6:805:63::18) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from 255.255.255.255 (255.255.255.255) by PR0P264CA0122.FRAP264.PROD.OUTLOOK.COM (2603:10a6:100:1a::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3305.24 via Frontend Transport; Thu, 20 Aug 2020 15:59:40 +0000 X-Mailer: git-send-email 2.28.0 X-Originating-IP: [37.71.187.125] X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 8a1a4a50-e580-4d53-bed7-08d845220cbb X-MS-TrafficTypeDiagnostic: SA0PR11MB4541: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:9508; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: pZtDQQBqpgNhl+ybQ6PX4Yt95mVhW9LRQqJT/O2OqrABC1DkmsC916Wi+0kW/W6sub+btkj4NRuQsSc8cV/Fcowz5YD14UOHfh1EiK/1QqPFdJJPjuidzEVnhLrns/8wUKoZ79gyOXIx3O0fbsV4fJ1zCR+RFfuypaeWXKb17MvekFeTVbK81WnwArQBQ0zm24YxoJGF0lKl8wruUdj5zxXEikXUyPeMNw+dxJN7MCXolxgm5jb3aBAmIxePdJHwZbPcyfbyZP2DrShtpK4rhBUwMk/26K15Qbc0DHAu2cImnKF/US4VDMGHzP8QCzKI1KXhzW7Y05fMxzgKF1eE5K5vVyLY+FfNk4JRNYKxRAzE6jK+PDAHO5fcG148IJyY X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:SN6PR11MB2718.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(376002)(396003)(39850400004)(366004)(346002)(136003)(186003)(6666004)(2906002)(8676002)(316002)(16576012)(54906003)(956004)(26005)(66574015)(8936002)(2616005)(110011004)(52116002)(83380400001)(107886003)(1076003)(5660300002)(478600001)(36756003)(4326008)(66476007)(66556008)(66946007)(86362001)(6486002);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData: E9gMyB7a8DkMDl5KQCMlHWRffYV8WrWv7wmqm1VTvEDTRInKKsWw97yRsRv5MxR38Pas4oPAmIMRAHFP7JJ7/Of5KgSU8chQzEKrj7m4redUhAx9y4j+Kvu0Fl/vGO4qikZFOvEI+XetJDYFOAAkTG15EulB+KcPe7DopELLq+OCmJbRlKLQztER5DwB4T9q/Uds36LU/iz/bi9D8V+QOgz1RyG68sEBxucSDAHAVwFnFg2flTn+ZdKh4kYxsNzpgsGYxafvJ+tQXUj0OO1t2tUiBFYQHqMI5fVpR36u7dTA4smzNgHpwKIx5lj3Qvk9Hb6bEptziN5bFSGi1BF1hWDHBcrC9QlKtlrQfGjY8fRuJlftL3b270fKlyOwzpRS4N9YqTUD+FU15Ic9fLrfOw69wXW8AkS0RDKndrTDkdsN8HLUPSvJe59ziYUZe8YbqBsHKK9QvOs9O33VWZdJiEn4d/+PJ2ato6vQYf4fkAJm5SWTug1W+5mJhcIrTKkwLchPrFqoyIn7X3/KDHdoVXpBJFVXMF4A+imu9UFVT5VVbQAdvLUsMjObPmBNmyKpsRGrVkJarjG2pWW1665IzmzYQ9urVaFuNxTnf8euv9OgYed820FXvAaZyTpY4aZgtASjenhapfTIEv9yC7n30w== X-OriginatorOrg: silabs.com X-MS-Exchange-CrossTenant-Network-Message-Id: 8a1a4a50-e580-4d53-bed7-08d845220cbb X-MS-Exchange-CrossTenant-AuthSource: SN6PR11MB2718.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Aug 2020 15:59:41.9309 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 54dbd822-5231-4b20-944d-6f4abcd541fb X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: ckeZJ/7WWX2HFIlaYLiPQo8SqiNyqCTUilpsniWVQAKDc1nG58MV/0os3XSjUk/9Pts6Ncv+U0HHh0Suyy9xSA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA0PR11MB4541 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org From: Jérôme Pouiller The host and the device can be connected with a called Wake-Up GPIO. When the host fall down this GPIO, it allows the device to enter in deep sleep and no communication with the device is no more possible (the device wakes up automatically on DTIM and fetch data if necessary). So, before to communicate with the device, the driver have to raise the Wake-up GPIO and then wait for an IRQ from the device. Unfortunately, old firmwares have a race in sleep/wake-up process and the device may never wake up. In this case, the IRQ is not sent and driver complains with "timeout while wake up chip". Then, the driver tries anyway to access the bus and an other error is raised by the bus. Fortunately, when the bug occurs, it is possible to fall down the IRQ and the device will eventually finish the sleep process. Then the driver can wake it up normally. The patch implements that workaround and add a retry limit in case something goes very wrong. Signed-off-by: Jérôme Pouiller --- drivers/staging/wfx/bh.c | 23 +++++++++++++++++++---- 1 file changed, 19 insertions(+), 4 deletions(-) diff --git a/drivers/staging/wfx/bh.c b/drivers/staging/wfx/bh.c index 07304a80c29b..f07bcee50e3f 100644 --- a/drivers/staging/wfx/bh.c +++ b/drivers/staging/wfx/bh.c @@ -18,25 +18,40 @@ static void device_wakeup(struct wfx_dev *wdev) { + int max_retry = 3; + if (!wdev->pdata.gpio_wakeup) return; if (gpiod_get_value_cansleep(wdev->pdata.gpio_wakeup)) return; - gpiod_set_value_cansleep(wdev->pdata.gpio_wakeup, 1); if (wfx_api_older_than(wdev, 1, 4)) { + gpiod_set_value_cansleep(wdev->pdata.gpio_wakeup, 1); if (!completion_done(&wdev->hif.ctrl_ready)) usleep_range(2000, 2500); - } else { + return; + } + for (;;) { + gpiod_set_value_cansleep(wdev->pdata.gpio_wakeup, 1); // completion.h does not provide any function to wait // completion without consume it (a kind of // wait_for_completion_done_timeout()). So we have to emulate // it. if (wait_for_completion_timeout(&wdev->hif.ctrl_ready, - msecs_to_jiffies(2))) + msecs_to_jiffies(2))) { complete(&wdev->hif.ctrl_ready); - else + return; + } else if (max_retry-- > 0) { + // Older firmwares have a race in sleep/wake-up process. + // Redo the process is sufficient to unfreeze the + // chip. dev_err(wdev->dev, "timeout while wake up chip\n"); + gpiod_set_value_cansleep(wdev->pdata.gpio_wakeup, 0); + usleep_range(2000, 2500); + } else { + dev_err(wdev->dev, "max wake-up retries reached\n"); + return; + } } }