From patchwork Tue Nov 24 10:48:55 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Milan Lakhani X-Patchwork-Id: 11927907 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=-13.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED 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 C598AC56202 for ; Tue, 24 Nov 2020 10:49:16 +0000 (UTC) Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id E9566206F9 for ; Tue, 24 Nov 2020 10:49:15 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=lists.elisa.tech header.i=@lists.elisa.tech header.b="O9vh4hDF" Authentication-Results: mail.kernel.org; dmarc=permerror header.from=codethink.co.uk Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=bounce+72012+199+5278000+9232812@lists.elisa.tech X-Received: by 127.0.0.2 with SMTP id BagmYY5279335xvwxfnrVPoY; Tue, 24 Nov 2020 02:49:15 -0800 X-Received: from ubuntu.localdomain (ubuntu.localdomain [2.127.6.98]) by mx.groups.io with SMTP id smtpd.web08.50159.1606214953679323105 for ; Tue, 24 Nov 2020 02:49:14 -0800 X-Received: by ubuntu.localdomain (Postfix, from userid 1000) id B15EF160BAE; Tue, 24 Nov 2020 10:49:11 +0000 (GMT) From: "Milan Lakhani" To: gregkh@linuxfoundation.org, lukas.bulwahn@gmail.com, linux-kernel@vger.kernel.org, linux-safety@lists.elisa.tech, forest@alittletooquiet.net Cc: Milan Lakhani Subject: [linux-safety] [PATCH v2 1/2] staging: vt6655: Fix alignment in device_main.c Date: Tue, 24 Nov 2020 10:48:55 +0000 Message-Id: <1606214936-35192-1-git-send-email-milan.lakhani@codethink.co.uk> Precedence: Bulk List-Unsubscribe: Sender: linux-safety@lists.elisa.tech List-Id: Mailing-List: list linux-safety@lists.elisa.tech; contact linux-safety+owner@lists.elisa.tech List-Post: X-Gm-Message-State: DHZZ8QgvUBXTwtU20vUo8nDqx5278000AA= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=lists.elisa.tech; q=dns/txt; s=20140610; t=1606214955; bh=pSnHC+VtpWbXi3roAjqdtD+ZhXPndmeozhjxnhimdY8=; h=Cc:Date:From:Subject:To; b=O9vh4hDFUGdTqkMshF52UUiyf6z8wpkbacwAmKsRsaew8KrUNBs+nHZerpaCFn/piKm Ur8YGws9TKadfl5GpGfBQqHqJAqcqj3aKvX2k13207e0AN8tnwgkGakraR7YcijgpKaEr ier2n64cnuh3KlnXBIlQKOUxF5w4FGMkGxg= Correct alignment to open parenthesis in device_main.c. Reported by checkpatch. Signed-off-by: Milan Lakhani --- drivers/staging/vt6655/device_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/vt6655/device_main.c b/drivers/staging/vt6655/device_main.c index 9c7d70e..2108ae6 100644 --- a/drivers/staging/vt6655/device_main.c +++ b/drivers/staging/vt6655/device_main.c @@ -1077,7 +1077,7 @@ static void vnt_interrupt_process(struct vnt_private *priv) PSbIsNextTBTTWakeUp((void *)priv); if ((priv->op_mode == NL80211_IFTYPE_AP || - priv->op_mode == NL80211_IFTYPE_ADHOC) && + priv->op_mode == NL80211_IFTYPE_ADHOC) && priv->vif->bss_conf.enable_beacon) MACvOneShotTimer1MicroSec(priv, (priv->vif->bss_conf.beacon_int - From patchwork Tue Nov 24 10:48:56 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Milan Lakhani X-Patchwork-Id: 11927905 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=-13.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED 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 C357FC56201 for ; Tue, 24 Nov 2020 10:49:16 +0000 (UTC) Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id EF25D2083E for ; Tue, 24 Nov 2020 10:49:15 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=lists.elisa.tech header.i=@lists.elisa.tech header.b="ZqBSfT7R" Authentication-Results: mail.kernel.org; dmarc=permerror header.from=codethink.co.uk Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=bounce+72012+198+5278000+9232812@lists.elisa.tech X-Received: by 127.0.0.2 with SMTP id GLasYY5279335xVQEACPTTap; Tue, 24 Nov 2020 02:49:15 -0800 X-Received: from ubuntu.localdomain (ubuntu.localdomain [2.127.6.98]) by mx.groups.io with SMTP id smtpd.web11.50602.1606214953675252629 for ; Tue, 24 Nov 2020 02:49:14 -0800 X-Received: by ubuntu.localdomain (Postfix, from userid 1000) id B53C4160BAD; Tue, 24 Nov 2020 10:49:11 +0000 (GMT) From: "Milan Lakhani" To: gregkh@linuxfoundation.org, lukas.bulwahn@gmail.com, linux-kernel@vger.kernel.org, linux-safety@lists.elisa.tech, forest@alittletooquiet.net Cc: Milan Lakhani Subject: [linux-safety] [PATCH v2 2/2] staging: vt6655: Use macro in rxtx.c Date: Tue, 24 Nov 2020 10:48:56 +0000 Message-Id: <1606214936-35192-2-git-send-email-milan.lakhani@codethink.co.uk> In-Reply-To: <1606214936-35192-1-git-send-email-milan.lakhani@codethink.co.uk> References: <1606214936-35192-1-git-send-email-milan.lakhani@codethink.co.uk> Precedence: Bulk List-Unsubscribe: Sender: linux-safety@lists.elisa.tech List-Id: Mailing-List: list linux-safety@lists.elisa.tech; contact linux-safety+owner@lists.elisa.tech List-Post: X-Gm-Message-State: 9AWYtvPJrQ54aoaffiPazHVSx5278000AA= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=lists.elisa.tech; q=dns/txt; s=20140610; t=1606214955; bh=AdeZ890CWy37I6JfHJAW32L3oaZ6Gsuv+3q1iAv5ceE=; h=Cc:Date:From:Subject:To; b=ZqBSfT7RDpq+3pxuLUNhTfc+hUl0OZNN1Wdo1+L8TcRDAaS96SjVDfrOjO41r2IYKJk W6+zOAPv/vYh5sayWuqwGV97m9PCrL0lH7E0W5f4/Nruozp8JocBiGVrE+yGwg9XhCfcS GuAchd10/+NOONczQSoMpY8ANzQZSU4yQCo= Use macro for repeated function to get the data duration and timestamp, correcting line length and alignment warnings in rxtx.c. Reported by checkpatch. Signed-off-by: Milan Lakhani --- drivers/staging/vt6655/rxtx.c | 51 ++++++++++++++++--------------------------- 1 file changed, 19 insertions(+), 32 deletions(-) diff --git a/drivers/staging/vt6655/rxtx.c b/drivers/staging/vt6655/rxtx.c index 508e1bd..c5dfe3c 100644 --- a/drivers/staging/vt6655/rxtx.c +++ b/drivers/staging/vt6655/rxtx.c @@ -423,6 +423,10 @@ s_uGetRTSCTSDuration( return cpu_to_le16((u16)uDurTime); } +#define getDurationTimeStamp(byDurType, byPktType, wRate) \ + (cpu_to_le16((u16)s_uGetDataDuration(pDevice, (byDurType), cbFrameLength, (byPktType), \ + (wRate), bNeedAck, uFragIdx, cbLastFragmentSize, uMACfragNum, byFBOption))) + static __le16 s_uFillDataHead( @@ -467,15 +471,10 @@ s_uFillDataHead( } else { /* Get Duration and TimeStamp */ buf->duration_a = - cpu_to_le16((u16)s_uGetDataDuration(pDevice, DATADUR_A, cbFrameLength, - byPktType, wCurrentRate, bNeedAck, uFragIdx, - cbLastFragmentSize, uMACfragNum, - byFBOption)); + getDurationTimeStamp(DATADUR_A, byPktType, wCurrentRate); buf->duration_b = - cpu_to_le16((u16)s_uGetDataDuration(pDevice, DATADUR_B, cbFrameLength, - PK_TYPE_11B, pDevice->byTopCCKBasicRate, - bNeedAck, uFragIdx, cbLastFragmentSize, - uMACfragNum, byFBOption)); + getDurationTimeStamp(DATADUR_B, PK_TYPE_11B, + pDevice->byTopCCKBasicRate); } buf->time_stamp_off_a = vnt_time_stamp_off(pDevice, wCurrentRate); @@ -492,14 +491,11 @@ s_uFillDataHead( pDevice->byTopCCKBasicRate, PK_TYPE_11B, &buf->b); /* Get Duration and TimeStamp */ - buf->duration_a = cpu_to_le16((u16)s_uGetDataDuration(pDevice, DATADUR_A, cbFrameLength, byPktType, - wCurrentRate, bNeedAck, uFragIdx, cbLastFragmentSize, uMACfragNum, byFBOption)); - buf->duration_b = cpu_to_le16((u16)s_uGetDataDuration(pDevice, DATADUR_B, cbFrameLength, PK_TYPE_11B, - pDevice->byTopCCKBasicRate, bNeedAck, uFragIdx, cbLastFragmentSize, uMACfragNum, byFBOption)); - buf->duration_a_f0 = cpu_to_le16((u16)s_uGetDataDuration(pDevice, DATADUR_A_F0, cbFrameLength, byPktType, - wCurrentRate, bNeedAck, uFragIdx, cbLastFragmentSize, uMACfragNum, byFBOption)); - buf->duration_a_f1 = cpu_to_le16((u16)s_uGetDataDuration(pDevice, DATADUR_A_F1, cbFrameLength, byPktType, - wCurrentRate, bNeedAck, uFragIdx, cbLastFragmentSize, uMACfragNum, byFBOption)); + buf->duration_a = getDurationTimeStamp(DATADUR_A, byPktType, wCurrentRate); + buf->duration_b = getDurationTimeStamp(DATADUR_B, PK_TYPE_11B, + pDevice->byTopCCKBasicRate); + buf->duration_a_f0 = getDurationTimeStamp(DATADUR_A_F0, byPktType, wCurrentRate); + buf->duration_a_f1 = getDurationTimeStamp(DATADUR_A_F1, byPktType, wCurrentRate); buf->time_stamp_off_a = vnt_time_stamp_off(pDevice, wCurrentRate); buf->time_stamp_off_b = vnt_time_stamp_off(pDevice, pDevice->byTopCCKBasicRate); @@ -517,12 +513,11 @@ s_uFillDataHead( byPktType, &buf->a); /* Get Duration and TimeStampOff */ - buf->duration = cpu_to_le16((u16)s_uGetDataDuration(pDevice, DATADUR_A, cbFrameLength, byPktType, - wCurrentRate, bNeedAck, uFragIdx, cbLastFragmentSize, uMACfragNum, byFBOption)); - buf->duration_f0 = cpu_to_le16((u16)s_uGetDataDuration(pDevice, DATADUR_A_F0, cbFrameLength, byPktType, - wCurrentRate, bNeedAck, uFragIdx, cbLastFragmentSize, uMACfragNum, byFBOption)); - buf->duration_f1 = cpu_to_le16((u16)s_uGetDataDuration(pDevice, DATADUR_A_F1, cbFrameLength, byPktType, - wCurrentRate, bNeedAck, uFragIdx, cbLastFragmentSize, uMACfragNum, byFBOption)); + buf->duration = getDurationTimeStamp(DATADUR_A, byPktType, wCurrentRate); + buf->duration_f0 = getDurationTimeStamp(DATADUR_A_F0, byPktType, + wCurrentRate); + buf->duration_f1 = getDurationTimeStamp(DATADUR_A_F1, byPktType, + wCurrentRate); buf->time_stamp_off = vnt_time_stamp_off(pDevice, wCurrentRate); return buf->duration; } @@ -537,11 +532,7 @@ s_uFillDataHead( buf->duration = dur; } else { /* Get Duration and TimeStampOff */ - buf->duration = - cpu_to_le16((u16)s_uGetDataDuration(pDevice, DATADUR_A, cbFrameLength, byPktType, - wCurrentRate, bNeedAck, uFragIdx, - cbLastFragmentSize, uMACfragNum, - byFBOption)); + buf->duration = getDurationTimeStamp(DATADUR_A, byPktType, wCurrentRate); } buf->time_stamp_off = vnt_time_stamp_off(pDevice, wCurrentRate); @@ -558,11 +549,7 @@ s_uFillDataHead( buf->duration = dur; } else { /* Get Duration and TimeStampOff */ - buf->duration = - cpu_to_le16((u16)s_uGetDataDuration(pDevice, DATADUR_B, cbFrameLength, byPktType, - wCurrentRate, bNeedAck, uFragIdx, - cbLastFragmentSize, uMACfragNum, - byFBOption)); + buf->duration = getDurationTimeStamp(DATADUR_B, byPktType, wCurrentRate); } buf->time_stamp_off = vnt_time_stamp_off(pDevice, wCurrentRate);