From patchwork Wed Apr 15 16:11:47 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: 216004 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-9.7 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, MSGID_FROM_MTA_HEADER, SIGNED_OFF_BY, SPF_HELO_NONE, SPF_PASS, URIBL_BLOCKED, USER_AGENT_GIT autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 060F6C3815B for ; Wed, 15 Apr 2020 16:13:35 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id D611E2076A for ; Wed, 15 Apr 2020 16:13:34 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=silabs.onmicrosoft.com header.i=@silabs.onmicrosoft.com header.b="fCxQBa+h" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1415323AbgDOQNd (ORCPT ); Wed, 15 Apr 2020 12:13:33 -0400 Received: from mail-bn8nam12on2054.outbound.protection.outlook.com ([40.107.237.54]:6138 "EHLO NAM12-BN8-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1415293AbgDOQNT (ORCPT ); Wed, 15 Apr 2020 12:13:19 -0400 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=jpXGtCFGELEqSPYIPzJfzget8T7ZEk1z1+RCOYaM+rd9kK8+aNfgN5rRHhuKjTVqOY+r9pykab2jwpPFMWBfGo4JLIlFO48EhM3kV78gh6CdrY3Ll6C3V7axBxUoJxjxy6kn466y7OBIbzDq7vJPMmiwtKiK4JAU6JPdJ2ft7RFOtSwTX0NTnlFYUw7+3XGoDs776TCkjMSdNbIC2OUCSvZCyJXlKxRocaWhXri+KvsSxWZosw5kZa4t6o6zY/mGySTMkHLs2Zedlcpm73HBUhxaqXK8VdxCAfqPNEZsejNF4Uj+KLCz17gisiuHO+/tJHnzFNv0Pm8JIBRRZ1NQMQ== 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=xMMpG8gnISmIROGGpUscDAlczvuyFeJmquYoEVpkyCc=; b=LzMRUz1SYcRd2U+L/akzoL7CzfaRb3XW2VbKaBloheFyRq7RQNvHIUmUWdj2LzZMy8JnDM2hk8gQDybqMPqBYhptHWWdIHZVgdcGTIk1DBw1rwv076QGV8NvVjo/gnI/d7LvB7sMzi9pw+9tGkFWOtDBqOZwWd3yPak2TuDXkGgz9cyJ/R1SRCM/dc2yWm8JODGkzi+Opqk79BICLUpl/ExgxqxHJSmxUczlHatmF5MXx+CTKT2JIwAy9Q63SuWd+y+zFlBZhXnkAWUJmK2BNP1TqZn7txEC7P9jJtH+qi0fnqpp6Tb/GJZZQI88xvLljHpWvsHU/PGprvjhQI82Zw== 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=xMMpG8gnISmIROGGpUscDAlczvuyFeJmquYoEVpkyCc=; b=fCxQBa+hQFwVK5FWAA73+Evh7H2VLdOy4+W6tyuUJARgu8LcJYdO1Y7xXTbzGLqKJ8+DS4PKtwq2/5iYvzX9PEN5B/3ppgxiE0C1BxmV204b4YadSvALQCtwMd8/JBQmF/+J3abhEcusuzbdFXIebRfcNaYsJ3Z0/yRATAS2t+k= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Jerome.Pouiller@silabs.com; Received: from MWHPR11MB1775.namprd11.prod.outlook.com (2603:10b6:300:10e::14) by MWHPR11MB1327.namprd11.prod.outlook.com (2603:10b6:300:2a::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2900.15; Wed, 15 Apr 2020 16:13:03 +0000 Received: from MWHPR11MB1775.namprd11.prod.outlook.com ([fe80::81d5:b62b:3770:ffbe]) by MWHPR11MB1775.namprd11.prod.outlook.com ([fe80::81d5:b62b:3770:ffbe%10]) with mapi id 15.20.2921.024; Wed, 15 Apr 2020 16:13:03 +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 20/20] staging: wfx: update TODO Date: Wed, 15 Apr 2020 18:11:47 +0200 Message-Id: <20200415161147.69738-21-Jerome.Pouiller@silabs.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20200415161147.69738-1-Jerome.Pouiller@silabs.com> References: <20200415161147.69738-1-Jerome.Pouiller@silabs.com> X-ClientProxiedBy: PR1PR01CA0027.eurprd01.prod.exchangelabs.com (2603:10a6:102::40) To MWHPR11MB1775.namprd11.prod.outlook.com (2603:10b6:300:10e::14) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from pc-42.home (2a01:e35:2435:66a0:544b:f17b:7ae8:fb7) by PR1PR01CA0027.eurprd01.prod.exchangelabs.com (2603:10a6:102::40) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2921.25 via Frontend Transport; Wed, 15 Apr 2020 16:13:01 +0000 X-Mailer: git-send-email 2.25.1 X-Originating-IP: [2a01:e35:2435:66a0:544b:f17b:7ae8:fb7] X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: a1a22fcc-e692-45d0-a3ad-08d7e157dfcf X-MS-TrafficTypeDiagnostic: MWHPR11MB1327: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:9508; X-Forefront-PRVS: 0374433C81 X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:MWHPR11MB1775.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFTY:; SFS:(10009020)(39850400004)(136003)(346002)(366004)(376002)(396003)(4326008)(966005)(81156014)(107886003)(52116002)(54906003)(8676002)(478600001)(8936002)(6506007)(5660300002)(6512007)(36756003)(8886007)(86362001)(316002)(6486002)(1076003)(2616005)(186003)(2906002)(66574012)(6666004)(66556008)(16526019)(66946007)(66476007)(15650500001); DIR:OUT; SFP:1101; Received-SPF: None (protection.outlook.com: silabs.com does not designate permitted sender hosts) X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: KiB6MGJ3LRxuqSR9ScBeyqxStb7RTb+RkengRBAsQ2e0oVag/7uoLTuK7Amle7WzDmniCXO47tFcuPUx02NcPAlvjgLv9PiNg0Bi3JfVLVOLybnzVBI1WUFMXGQFJLrpUoumWVXAjc2dmJJkShPUXlDgG56gAX51R2BxwBVFVCQ+Yljtq6HeKTvLzB4j3gezFbUpAQsFu16yDKlko6LBo6m+TEB5P1Lv9y/003X5eB34PAu3SxaqkzP141WvGjkQIQ1wDfOo16X35JyPDnD6yCwTVvIl8/AflzHmJAWz804/s+WK+zTziwvhn1V5VDp9NqcTc4/wHBr+vLWpDbQcd1/3Dvj6NEOMm4gGBD0UleDyjZaH17zX+eV7a8XUAjgEF6J2iOHvNnKghvlRSsTgHhhydm0oQeQ02vTgX8N0TMY0OvPGqqZiU09twAurTUFiw1tgMSm+WyQ5MREzDm3xucyzlfMAkiQUoif29bCy/hyB4epXdXev4vmEYWvHPLoNuy+qEw8C1BROXWh20YD+SQ== X-MS-Exchange-AntiSpam-MessageData: 3ImYzr2IDHDpw+QGVHWrxWCEaO++cRZ0WTIPpjcWED8fv+atShJRumVSzxW9z6tP3Do7oFvG2sOU0aJiFr4eKi0WRegwbLwv5m26wwQ/oAR4O25qrqT6lIp1twjMwtJx//GpccBmpDtRsUH04Y/qY7QERWUsK76fEqIesbRm3AGJ/fL/+A6bueTJ8mhNEWXA9nZyOT4eEcIpvzn8JrwsNQ== X-OriginatorOrg: silabs.com X-MS-Exchange-CrossTenant-Network-Message-Id: a1a22fcc-e692-45d0-a3ad-08d7e157dfcf X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Apr 2020 16:13:03.0635 (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: zqMko9GGvsZMwM7qX9CPHvKCC9ctloagB2NdmXMYs5kr1CgS1IskcfnuozwEIjMD0C1TtYIT22z9SLXRAa/Siw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR11MB1327 Sender: linux-wireless-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-wireless@vger.kernel.org From: Jérôme Pouiller Update the TODO list associated to the wfx driver with the last progresses and last remarks made by reviewers. The items about support for P2P and mesh have also been removed. Indeed, it seems that the device does not fully support them. Signed-off-by: Jérôme Pouiller --- drivers/staging/wfx/TODO | 40 ++++++++++++---------------------------- 1 file changed, 12 insertions(+), 28 deletions(-) diff --git a/drivers/staging/wfx/TODO b/drivers/staging/wfx/TODO index efcb7c6a5aa7..fca3332e42ce 100644 --- a/drivers/staging/wfx/TODO +++ b/drivers/staging/wfx/TODO @@ -1,26 +1,18 @@ This is a list of things that need to be done to get this driver out of the staging directory. - - All structures defined in hif_api_*.h are intended to sent/received to/from - hardware. All their members whould be declared __le32 or __le16. - See: + - The HIF API is not yet clean enough. + + - Fix support for big endian architectures. See: https://lore.kernel.org/lkml/20191111202852.GX26530@ZenIV.linux.org.uk - - Once previous item done, it will be possible to audit the driver with - `sparse'. It will probably find tons of problems with big endian - architectures. + - The pointers returned by allocation functions are always checked. - - hif_api_*.h whave been imported from firmware code. Some of the structures - are never used in driver. + - The code that check the corectness of received message (in rx_helper()) can + be improved. See: + https://lore.kernel.org/driverdev-devel/2302785.6C7ODC2LYm@pc-42/ - - Driver try to maintains power save status of the stations. However, this - work is already done by mac80211. sta_asleep_mask and pspoll_mask should be - dropped. - - - wfx_tx_queues_get() should be reworked. It currently try compute itself the - QoS policy. However, firmware already do the job. Firmware would prefer to - have a few packets in each queue and be able to choose itself which queue to - use. + - Support for SDIO with external IRQ is broken. - As suggested by Felix, rate control could be improved following this idea: https://lore.kernel.org/lkml/3099559.gv3Q75KnN1@pc-42/ @@ -28,17 +20,8 @@ staging directory. - When driver is about to loose BSS, it forge its own Null Func request (see wfx_cqm_bssloss_sm()). It should use mechanism provided by mac80211. - - AP is actually is setup after a call to wfx_bss_info_changed(). Yet, - ieee80211_ops provide callback start_ap(). - - - The current process for joining a network is incredibly complex. Should be - reworked. - - Monitoring mode is not implemented despite being mandatory by mac80211. - - "compatible" value are not correct. They should be "vendor,chip". See: - https://lore.kernel.org/driverdev-devel/5226570.CMH5hVlZcI@pc-42 - - The "state" field from wfx_vif should be replaced by "vif->type". - It seems that wfx_upload_keys() is useless. @@ -49,12 +32,13 @@ staging directory. - Feature called "secure link" should be either developed (using kernel crypto API) or dropped. + - The device allows to filter multicast traffic. The code to support these + filters exists in the driver but it is disabled because it has never been + tested. + - In wfx_cmd_send(), "async" allow to send command without waiting the reply. It may help in some situation, but it is not yet used. In add, it may cause some trouble: https://lore.kernel.org/driverdev-devel/alpine.DEB.2.21.1910041317381.2992@hadrien/ So, fix it (by replacing the mutex with a semaphore) or drop it. - - Chip support P2P, but driver does not implement it. - - - Chip support kind of Mesh, but driver does not implement it.