From mboxrd@z Thu Jan 1 00:00:00 1970 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.6 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS 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 825A1C433E3 for ; Sun, 26 Jul 2020 11:00:00 +0000 (UTC) Received: from whitealder.osuosl.org (smtp1.osuosl.org [140.211.166.138]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 55A9A206E3 for ; Sun, 26 Jul 2020 11:00:00 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="t5J1N832" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 55A9A206E3 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=linuxfoundation.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=driverdev-devel-bounces@linuxdriverproject.org Received: from localhost (localhost [127.0.0.1]) by whitealder.osuosl.org (Postfix) with ESMTP id 2D7E587846; Sun, 26 Jul 2020 11:00:00 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from whitealder.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id Hyv27T68vZCu; Sun, 26 Jul 2020 10:59:58 +0000 (UTC) Received: from ash.osuosl.org (ash.osuosl.org [140.211.166.34]) by whitealder.osuosl.org (Postfix) with ESMTP id DF90887821; Sun, 26 Jul 2020 10:59:58 +0000 (UTC) Received: from silver.osuosl.org (smtp3.osuosl.org [140.211.166.136]) by ash.osuosl.org (Postfix) with ESMTP id B97291BF232 for ; Sun, 26 Jul 2020 10:59:57 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by silver.osuosl.org (Postfix) with ESMTP id 7F69321539 for ; Sun, 26 Jul 2020 10:59:57 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from silver.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id BLhXYaZmtDeX for ; Sun, 26 Jul 2020 10:59:56 +0000 (UTC) X-Greylist: from auto-whitelisted by SQLgrey-1.7.6 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by silver.osuosl.org (Postfix) with ESMTPS id 11C0921513 for ; Sun, 26 Jul 2020 10:59:56 +0000 (UTC) Received: from localhost (83-86-89-107.cable.dynamic.v4.ziggo.nl [83.86.89.107]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 45071206E3; Sun, 26 Jul 2020 10:59:55 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1595761195; bh=BlWNHRx81BkVjfiVPZH2lP4OPiWIUw2bTOwNTT+SBNw=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=t5J1N8320A2lXEGDUBMQ9QLS/rNDPxDuntxRrLJwbyME6byZ7HmDYpa1XgCR+vOIv olt8YyQKpYVhahn09WHv4arWGIMp8hqm430g0Ih6lbrEhvClU2IqBGJmScRSVz2qz0 +x8Qd/MkwYW9NCUeHu+uuxffb+CEpQlh5RkiMsSs= Date: Sun, 26 Jul 2020 12:59:52 +0200 From: Greg KH To: Aditya Jain Subject: Re: [PATCH v2] staging: rtl8723bs: include: Fix coding style issues Message-ID: <20200726105952.GA1280500@kroah.com> References: <20200726105302.52188-1-aditya.jainadityajain.jain@gmail.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20200726105302.52188-1-aditya.jainadityajain.jain@gmail.com> X-BeenThere: driverdev-devel@linuxdriverproject.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Linux Driver Project Developer List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devel@driverdev.osuosl.org, hdegoede@redhat.com, linux-kernel@vger.kernel.org, Larry.Finger@lwfinger.net Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: driverdev-devel-bounces@linuxdriverproject.org Sender: "devel" On Sun, Jul 26, 2020 at 04:23:02PM +0530, Aditya Jain wrote: > Cleaning messy function declartions and fixing code style > errors in hal_phy_cfg.h as reported by checkpatch.pl > > Signed-off-by: Aditya Jain > --- > .../staging/rtl8723bs/include/hal_phy_cfg.h | 118 +++++------------- > 1 file changed, 33 insertions(+), 85 deletions(-) > > diff --git a/drivers/staging/rtl8723bs/include/hal_phy_cfg.h b/drivers/staging/rtl8723bs/include/hal_phy_cfg.h > index 419ddb0733aa..2f123903279c 100644 > --- a/drivers/staging/rtl8723bs/include/hal_phy_cfg.h > +++ b/drivers/staging/rtl8723bs/include/hal_phy_cfg.h > @@ -8,54 +8,31 @@ > #define __INC_HAL8723BPHYCFG_H__ > > /*--------------------------Define Parameters-------------------------------*/ > -#define LOOP_LIMIT 5 > -#define MAX_STALL_TIME 50 /* us */ > +#define LOOP_LIMIT 5 > +#define MAX_STALL_TIME 50 /* us */ > #define AntennaDiversityValue 0x80 /* Adapter->bSoftwareAntennaDiversity ? 0x00:0x80) */ > #define MAX_TXPWR_IDX_NMODE_92S 63 > -#define Reset_Cnt_Limit 3 > - > -#define MAX_AGGR_NUM 0x07 > +#define Reset_Cnt_Limit 3 > > +#define MAX_AGGR_NUM 0x07 > > /*--------------------------Define Parameters End-------------------------------*/ > > - > /*------------------------------Define structure----------------------------*/ > > /*------------------------------Define structure End----------------------------*/ > > /*--------------------------Exported Function prototype---------------------*/ > -u32 > -PHY_QueryBBReg_8723B( > -struct adapter *Adapter, > -u32 RegAddr, > -u32 BitMask > - ); > - > -void > -PHY_SetBBReg_8723B( > -struct adapter *Adapter, > -u32 RegAddr, > -u32 BitMask, > -u32 Data > - ); > - > -u32 > -PHY_QueryRFReg_8723B( > -struct adapter * Adapter, > -u8 eRFPath, > -u32 RegAddr, > -u32 BitMask > - ); > - > -void > -PHY_SetRFReg_8723B( > -struct adapter * Adapter, > -u8 eRFPath, > -u32 RegAddr, > -u32 BitMask, > -u32 Data > - ); > +u32 PHY_QueryBBReg_8723B(struct adapter *Adapter, u32 RegAddr, u32 BitMask); > + > +void PHY_SetBBReg_8723B(struct adapter *Adapter, u32 RegAddr, > + u32 BitMask, u32 Data); > + > +u32 PHY_QueryRFReg_8723B(struct adapter *Adapter, u8 eRFPath, > + u32 RegAddr, u32 BitMask); > + > +void PHY_SetRFReg_8723B(struct adapter *Adapter, u8 eRFPath, > + u32 RegAddr, u32 BitMask, u32 Data); > > /* MAC/BB/RF HAL config */ > int PHY_BBConfig8723B(struct adapter *Adapter); > @@ -64,57 +41,28 @@ int PHY_RFConfig8723B(struct adapter *Adapter); > > s32 PHY_MACConfig8723B(struct adapter *padapter); > > -void > -PHY_SetTxPowerIndex( > -struct adapter * Adapter, > -u32 PowerIndex, > -u8 RFPath, > -u8 Rate > - ); > - > -u8 > -PHY_GetTxPowerIndex( > -struct adapter * padapter, > -u8 RFPath, > -u8 Rate, > -enum CHANNEL_WIDTH BandWidth, > -u8 Channel > - ); > - > -void > -PHY_GetTxPowerLevel8723B( > -struct adapter * Adapter, > - s32* powerlevel > - ); > - > -void > -PHY_SetTxPowerLevel8723B( > -struct adapter * Adapter, > -u8 channel > - ); > - > -void > -PHY_SetBWMode8723B( > -struct adapter * Adapter, > -enum CHANNEL_WIDTH Bandwidth, /* 20M or 40M */ > -unsigned char Offset /* Upper, Lower, or Don't care */ > -); > +void PHY_SetTxPowerIndex(struct adapter *Adapter, u32 PowerIndex, > + u8 RFPath, u8 Rate); > > -void > -PHY_SwChnl8723B(/* Call after initialization */ > -struct adapter *Adapter, > -u8 channel > - ); > - > -void > -PHY_SetSwChnlBWMode8723B( > -struct adapter * Adapter, > -u8 channel, > -enum CHANNEL_WIDTH Bandwidth, > -u8 Offset40, > -u8 Offset80 > +u8 PHY_GetTxPowerIndex(struct adapter *padapter, u8 RFPath, u8 Rate, > + enum CHANNEL_WIDTH BandWidth, u8 Channel); > + > +void PHY_GetTxPowerLevel8723B(struct adapter *Adapter, s32 *powerlevel); > + > +void PHY_SetTxPowerLevel8723B(struct adapter *Adapter, u8 channel); > + > +void PHY_SetBWMode8723B(struct adapter *Adapter, > + enum CHANNEL_WIDTH Bandwidth, /* 20M or 40M */ > + unsigned char Offset /* Upper, Lower, or Don't care */ > ); > > +/* Call after initialization */ > +void PHY_SwChnl8723B(struct adapter *Adapter, u8 channel); > + > +void PHY_SetSwChnlBWMode8723B(struct adapter *Adapter, u8 channel, > + enum CHANNEL_WIDTH Bandwidth, > + u8 Offset40, u8 Offset80); > + > /*--------------------------Exported Function prototype End---------------------*/ > > #endif > -- > 2.25.1 > > _______________________________________________ > devel mailing list > devel@linuxdriverproject.org > http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel Hi, This is the friendly patch-bot of Greg Kroah-Hartman. You have sent him a patch that has triggered this response. He used to manually respond to these common problems, but in order to save his sanity (he kept writing the same thing over and over, yet to different people), I was created. Hopefully you will not take offence and will fix the problem in your patch and resubmit it so that it can be accepted into the Linux kernel tree. You are receiving this message because of the following common error(s) as indicated below: - Your patch did many different things all at once, making it difficult to review. All Linux kernel patches need to only do one thing at a time. If you need to do multiple things (such as clean up all coding style issues in a file/driver), do it in a sequence of patches, each one doing only one thing. This will make it easier to review the patches to ensure that they are correct, and to help alleviate any merge issues that larger patches can cause. - You did not specify a description of why the patch is needed, or possibly, any description at all, in the email body. Please read the section entitled "The canonical patch format" in the kernel file, Documentation/SubmittingPatches for what is needed in order to properly describe the change. - You did not write a descriptive Subject: for the patch, allowing Greg, and everyone else, to know what this patch is all about. Please read the section entitled "The canonical patch format" in the kernel file, Documentation/SubmittingPatches for what a proper Subject: line should look like. If you wish to discuss this problem further, or you have questions about how to resolve this issue, please feel free to respond to this email and Greg will reply once he has dug out from the pending patches received from other developers. thanks, greg k-h's patch email bot _______________________________________________ devel mailing list devel@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel