From patchwork Mon Sep 13 13:01:59 2021 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: 510229 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=-16.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID, HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_CR_TRAILER, INCLUDES_PATCH, MAILING_LIST_MULTI, MSGID_FROM_MTA_HEADER, SPF_HELO_NONE, SPF_PASS, URIBL_BLOCKED, USER_AGENT_GIT autolearn=ham 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 B8EBDC433FE for ; Mon, 13 Sep 2021 13:06:03 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id A0E546103B for ; Mon, 13 Sep 2021 13:06:03 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S240391AbhIMNHM (ORCPT ); Mon, 13 Sep 2021 09:07:12 -0400 Received: from mail-dm6nam10on2082.outbound.protection.outlook.com ([40.107.93.82]:64256 "EHLO NAM10-DM6-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S240226AbhIMNF2 (ORCPT ); Mon, 13 Sep 2021 09:05:28 -0400 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=JOgB/WT6RiWKWRI/68IKKsLeachQcfJ+dpQ7QkBxudU1vbaZKTUVJ7R3md2pNGgRJ2O9jR2C6mHf6V97/EtTmr/BY/6MXHbtI36lgQlYQYc5+qqXtNLYjzbhXjr7h3kpsjJhazT5Vw/7wMrFyU6lG1ZSQiS6Q1owmSeqLlCudp4KpGIjn6rozznPPZHe1batLXZc9WD9gF8hJGHBiDLQK/+RtQTsuPsKWadoFSsohjHousuRszrswzoQL1DYHH/ImiOWpnyXDbhYuxeYOr392S62tl2wcxGzFl7ynF5eyzF+D3FQCOLACt+uTiQ0Xs2/qLdzRnm8jn5u6a8/y3aNqg== 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; bh=6exHReMQZVm3Rqfp26e85943fuLvP90YScIDvmR1Gxs=; b=n2GlFvfGxDrk6CPHlp8FppPPsjYY7fAN3cCk6unJAqRIZZ4l+JdG3yoj9aJC1NONLK8N8c8kC3SOmDLEoobGqUhuKJxPRb38GIAKf5SsJfwJnKsYa4ZJYI7Y468t+y7ZNlL0TfjGL93pm9mMy9crZ+2wUG0ZsNp5zExSt1iX+BGXGfK8xxLYUVgdzD7CTQPoUla87s1374O1QupGnxmx1UoZ7n4UmhkUcqhJnMeLkpbNV7/qb4CA6U6FeQH9CuUNLcZL97P7CwR3ydxsw3FNEzjFfIgT0iJ1stAZxDPcdtYBkGnaY/rEgRcX6yWrukDEKiG7ARHst0PA3RMnqRaqMA== 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=6exHReMQZVm3Rqfp26e85943fuLvP90YScIDvmR1Gxs=; b=nRe2B8LPakd5mz0pemlJkEb5ZvXVpbk7cNyHqeloOaThpEyKeNgjEtgUL7yQcQZTLKY8yqyN9CCDQlYUfi2L3v4HTJrq/klGN4BhzsBd0Y86Hm7twbp7h/WBRkafeC5tDBl/RoSkUKqtycfCCjwNq8x5Zq3xMtYf2CB8tJxV7Wo= 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 SN6PR11MB3502.namprd11.prod.outlook.com (2603:10b6:805:da::27) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4500.14; Mon, 13 Sep 2021 13:03:10 +0000 Received: from SN6PR11MB2718.namprd11.prod.outlook.com ([fe80::7050:a0a:415:2ccd]) by SN6PR11MB2718.namprd11.prod.outlook.com ([fe80::7050:a0a:415:2ccd%7]) with mapi id 15.20.4500.017; Mon, 13 Sep 2021 13:03:10 +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 v3 28/32] staging: wfx: fix comments styles Date: Mon, 13 Sep 2021 15:01:59 +0200 Message-Id: <20210913130203.1903622-29-Jerome.Pouiller@silabs.com> X-Mailer: git-send-email 2.33.0 In-Reply-To: <20210913130203.1903622-1-Jerome.Pouiller@silabs.com> References: <20210913130203.1903622-1-Jerome.Pouiller@silabs.com> X-ClientProxiedBy: PR2P264CA0021.FRAP264.PROD.OUTLOOK.COM (2603:10a6:101::33) To SN6PR11MB2718.namprd11.prod.outlook.com (2603:10b6:805:63::18) MIME-Version: 1.0 Received: from pc-42.silabs.com (37.71.187.125) by PR2P264CA0021.FRAP264.PROD.OUTLOOK.COM (2603:10a6:101::33) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4500.14 via Frontend Transport; Mon, 13 Sep 2021 13:03:08 +0000 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: dc298e08-ec54-486b-4fa8-08d976b6d652 X-MS-TrafficTypeDiagnostic: SN6PR11MB3502: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:7219; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: a+N2tt/OlauzLxD/yOF0ozI4MmHgdxEeSnjuld/U0ZocjGU3oeCN108dIGempSfNDuxMxBY45eBxSwP12BUKxXhiVFcLBMd2unfL2xMsk+lW54sAvEkBKMakatmgb2Cuiyp3MtKovUSiBXlUH8uIePgvC2awVANtZFrIVOtmUdel/iZgVp0EMzZX1wDNzKRQVge9SpUBRC5zbakaWiGziYfKMn4XcM9g+llY2TxGrOtYwbRVF8YSx006a0JSElIT1qoT/utI4EX6cGMsbwQMp5Lqpc2GWmkoHsAdypCBvjKKj3IKcQXucyksphHs9KHNfy/Hy/4S1uWXzHp71rN/hwJG9ZvKPD/yRF8Nmtk26l+Sd8oeqS36jVy7gXGTbYYmbLKGsuoab8/BT5s9wIVgch/KWvGUNJFIgiIhaGgiB3HY7TNyewsiG3V9qs6gMdHO1z34v/jlz5H+n/Id+8wsTu/qjCCATMGdE+0kOr9Rd1FaJTfgytH7ymfr/Kdn9vqiTpB1YrNH2qtGDmaR33duJEqEmgAkJ/tQ8CZrPn4V/7yWQAXkaiqp043Zq1Mt3W0wdSTPhDoEiZQcok6kymtVV9gaqwIpdILt3G9AWXR2uOFJTWIMzUGsJXhTHBPw8/CyzwPLK+2f/YzhhXvCkcZxB2I6yLwyKzZtFmfVURe46L3JMIoViFCBV5zsLAziPo8EzQ+6cFG/n8NkV5mOeJ7pHw== 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:(39850400004)(136003)(396003)(376002)(346002)(366004)(6666004)(316002)(66476007)(8676002)(186003)(7696005)(52116002)(66946007)(26005)(4326008)(66556008)(2906002)(54906003)(107886003)(6486002)(2616005)(478600001)(1076003)(36756003)(5660300002)(66574015)(956004)(8936002)(86362001)(38100700002)(38350700002)(83380400001); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?X9xXrD79dQwaDFqVX8eEOSLff?= =?utf-8?q?td+nmJCch1/hxo6e4A5TgtwriWVvLY7qCxNDj0oNw3KIpnfSAwxzedvI?= =?utf-8?q?daeT+D3PJSwyLtg+2EIQdhRwvrIlNBiTAsJF4OgdMluyT/LIXSFf6+dU?= =?utf-8?q?XurcI381H54CAO198Yx3x+dK0waAdMurM2SQxTwTZt9W8P7SzY+roPu6?= =?utf-8?q?vBGV3n9HqKtkihUFiN+7mBZMU+W72XJH27Wx1OccC7jNAFFGtcVK2lrU?= =?utf-8?q?NEOe34E/BMz1jtxkIGDoX/eKDCdjVLNlPAXVYxg9188a6VzxbLCdU7nm?= =?utf-8?q?6Hs+L6NQ2Is5CNzqtFni90PmQ+ckpF6nBpVqsL5J6VJiwHdb0lKwlanh?= =?utf-8?q?lJYcIB7xHkwSRIBrrmuVTTztjkaWFMlChl8jYelK1OOROuN1tSagPjKW?= =?utf-8?q?/IeEOmxK+Qkcl3CxL8gRolRGDrHeY/rkXPisoO/GUXhhrWw1IhQfw3fw?= =?utf-8?q?U4DFCLY5IJ5FImwKNDPoAVr1GAANsCNnaKLn9yr609n/RKd4bQwVvjqg?= =?utf-8?q?5iy9uqJlJkWNL580LfRqpKf7ergG284bJrIRbFxvaag5n5EZMgsV7+74?= =?utf-8?q?OEX/fzQJiysbU1vSvmxBC8Ae+bp5DwNIk1v7xeJYczmZ1jfTSPkDSB7t?= =?utf-8?q?qiM5JeeBEbdXTfLJZYBYUUWPu9YR5cBAeEizHyOAf/tUdYEXR7CZFDph?= =?utf-8?q?hEnw7hvTmzHJkWChAOlWNr3OUUH2OzRzWiX3FH2HPChC/W12VyZhfEX/?= =?utf-8?q?OB+PrgME0WTvmwNaXArb3UxH6UHNLPUOadNeecxA7gc+4lQAR5uhC/S5?= =?utf-8?q?wuCmWZmoa4YyYhFSz0t0gEqJlfexI3pjI+WcP9c46Nj24WsY3GpmGrb/?= =?utf-8?q?6Nymdhm65tnmLKvGwKjwuCLkztEaFrysxJxSBLu3cA7CquShnZYAwWOM?= =?utf-8?q?t9t2Ag0Bg4vdXFjVs5bH6lqpNscaIcGGXFx/0MGOhOAjlHRI8KEhypg8?= =?utf-8?q?EzPk8ZQgb+I1OL6oefixdfzAsSNK9mT7akCvb7VmHZeTFUGMkz6ttvCj?= =?utf-8?q?eoLovqyBuDATmDLMZDX8c1DqZjUH16P511BAOWevDEZYAkamQMAnOGul?= =?utf-8?q?dfJ4x5+Qg3zRSpNm37YYhu1WK8+C64i8myhMTCow4P6TMNMSJgGc0aUg?= =?utf-8?q?NjYKnr8EHrU+7nWAcE2xrmBxGagc2Ii1jEQx6+Kzy0q+bStC+dmkosRB?= =?utf-8?q?PqtStQnSlCtYhfLDwMdIbe6M9ksvTTj/kLhjb3t5wUY+wkXGkh7rzjXl?= =?utf-8?q?UmGaiFBp3Zt6VaGGj57b6RMjWeSCLIcHRJEm4E25fl2QQJBQPw+2CEaE?= =?utf-8?q?z43vou4Dy70FYos3iA/oB1FRGVvkqlZ?= X-OriginatorOrg: silabs.com X-MS-Exchange-CrossTenant-Network-Message-Id: dc298e08-ec54-486b-4fa8-08d976b6d652 X-MS-Exchange-CrossTenant-AuthSource: SN6PR11MB2718.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Sep 2021 13:03:10.2829 (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: UriL9I+oka6o2IzyqWKRFzsfpCYI3VAmpz0nFHT3aySlkS2lVOw2T7jH5nOOEZsmfw62hCA+W6Us2dguihNrbA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN6PR11MB3502 Precedence: bulk List-ID: X-Mailing-List: linux-wireless@vger.kernel.org From: Jérôme Pouiller Unify all comments of the wfx driver to use the same comment style. Signed-off-by: Jérôme Pouiller --- drivers/staging/wfx/bh.c | 11 +++-------- drivers/staging/wfx/bus_spi.c | 6 ++---- drivers/staging/wfx/fwio.c | 3 +-- drivers/staging/wfx/main.h | 3 +-- drivers/staging/wfx/scan.c | 3 +-- 5 files changed, 8 insertions(+), 18 deletions(-) diff --git a/drivers/staging/wfx/bh.c b/drivers/staging/wfx/bh.c index e956b5a5ccac..a0f9d1b53019 100644 --- a/drivers/staging/wfx/bh.c +++ b/drivers/staging/wfx/bh.c @@ -265,9 +265,7 @@ static void bh_work(struct work_struct *work) wdev->hif.tx_buffers_used, release_chip); } -/* - * An IRQ from chip did occur - */ +/* An IRQ from chip did occur */ void wfx_bh_request_rx(struct wfx_dev *wdev) { u32 cur, prev; @@ -285,16 +283,13 @@ void wfx_bh_request_rx(struct wfx_dev *wdev) prev, cur); } -/* - * Driver want to send data - */ +/* Driver want to send data */ void wfx_bh_request_tx(struct wfx_dev *wdev) { queue_work(system_highpri_wq, &wdev->hif.bh); } -/* - * If IRQ is not available, this function allow to manually poll the control +/* If IRQ is not available, this function allow to manually poll the control * register and simulate an IRQ ahen an event happened. * * Note that the device has a bug: If an IRQ raise while host read control diff --git a/drivers/staging/wfx/bus_spi.c b/drivers/staging/wfx/bus_spi.c index 61f73b3ebc80..55ffcd7c42e2 100644 --- a/drivers/staging/wfx/bus_spi.c +++ b/drivers/staging/wfx/bus_spi.c @@ -38,8 +38,7 @@ struct wfx_spi_priv { bool need_swab; }; -/* - * The chip reads 16bits of data at time and place them directly into (little +/* The chip reads 16bits of data at time and place them directly into (little * endian) CPU register. So, the chip expects bytes order to be "B1 B0 B3 B2" * (while LE is "B0 B1 B2 B3" and BE is "B3 B2 B1 B0") * @@ -241,8 +240,7 @@ static int wfx_spi_remove(struct spi_device *func) return 0; } -/* - * For dynamic driver binding, kernel does not use OF to match driver. It only +/* For dynamic driver binding, kernel does not use OF to match driver. It only * use modalias and modalias is a copy of 'compatible' DT node with vendor * stripped. */ diff --git a/drivers/staging/wfx/fwio.c b/drivers/staging/wfx/fwio.c index c5ba0a50b474..98a9391b2bee 100644 --- a/drivers/staging/wfx/fwio.c +++ b/drivers/staging/wfx/fwio.c @@ -69,8 +69,7 @@ static const char * const fwio_errors[] = { [ERR_MAC_KEY] = "MAC key not initialized", }; -/* - * request_firmware() allocate data using vmalloc(). It is not compatible with +/* request_firmware() allocate data using vmalloc(). It is not compatible with * underlying hardware that use DMA. Function below detect this case and * allocate a bounce buffer if necessary. * diff --git a/drivers/staging/wfx/main.h b/drivers/staging/wfx/main.h index a0db322383a3..115abd2d4378 100644 --- a/drivers/staging/wfx/main.h +++ b/drivers/staging/wfx/main.h @@ -23,8 +23,7 @@ struct wfx_platform_data { const char *file_fw; const char *file_pds; struct gpio_desc *gpio_wakeup; - /* - * if true HIF D_out is sampled on the rising edge of the clock + /* if true HIF D_out is sampled on the rising edge of the clock * (intended to be used in 50Mhz SDIO) */ bool use_rising_clk; diff --git a/drivers/staging/wfx/scan.c b/drivers/staging/wfx/scan.c index 9e2d08317c9e..668ef2c60837 100644 --- a/drivers/staging/wfx/scan.c +++ b/drivers/staging/wfx/scan.c @@ -85,8 +85,7 @@ static int send_scan_req(struct wfx_vif *wvif, return ret; } -/* - * It is not really necessary to run scan request asynchronously. However, +/* It is not really necessary to run scan request asynchronously. However, * there is a bug in "iw scan" when ieee80211_scan_completed() is called before * wfx_hw_scan() return */