From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: Qiu Wenbo <qiuwenbo@phytium.com.cn>
Cc: Larry Finger <Larry.Finger@lwfinger.net>,
Florian Schilhabel <florian.c.schilhabel@googlemail.com>,
Linux Driver Project Developer List
<driverdev-devel@linuxdriverproject.org>,
Christophe JAILLET <christophe.jaillet@wanadoo.fr>,
Nishka Dasgupta <nishkadg.linux@gmail.com>,
Dan Carpenter <dan.carpenter@oracle.com>,
Kees Cook <keescook@chromium.org>,
Marco Villegas <git@marvil07.net>,
devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] staging: rtl8712: Fixes coding style in several headers
Date: Thu, 16 Jul 2020 09:04:16 +0200 [thread overview]
Message-ID: <20200716070416.GA962748@kroah.com> (raw)
In-Reply-To: <20200716064118.61243-1-qiuwenbo@phytium.com.cn>
On Thu, Jul 16, 2020 at 02:41:12PM +0800, Qiu Wenbo wrote:
> This patch fixes warnings in several headers found by the checkpatch.pl tool.
What warnings specifically?
Please only fix one type of warning per patch.
>
> Signed-off-by: Qiu Wenbo <qiuwenbo@phytium.com.cn>
> ---
> drivers/staging/rtl8712/rtl871x_event.h | 15 ++++++++++-----
> drivers/staging/rtl8712/rtl871x_io.h | 3 ++-
> drivers/staging/rtl8712/rtl871x_pwrctrl.h | 15 ++++++++++-----
> drivers/staging/rtl8712/rtl871x_xmit.h | 15 ++++++++++-----
> 4 files changed, 32 insertions(+), 16 deletions(-)
>
> diff --git a/drivers/staging/rtl8712/rtl871x_event.h b/drivers/staging/rtl8712/rtl871x_event.h
> index d9a5476d2426..0e59d0ee6aae 100644
> --- a/drivers/staging/rtl8712/rtl871x_event.h
> +++ b/drivers/staging/rtl8712/rtl871x_event.h
> @@ -78,13 +78,16 @@ struct event_node {
> unsigned char *node;
> unsigned char evt_code;
> unsigned short evt_sz;
> - /*volatile*/ int *caller_ff_tail;
> + /*volatile*/
> + int *caller_ff_tail;
Why did you change this? It's fine as is, and a hint something needs to
be fixed.
> int caller_ff_sz;
> };
>
> struct c2hevent_queue {
> - /*volatile*/ int head;
> - /*volatile*/ int tail;
> + /*volatile*/
> + int head;
> + /*volatile*/
> + int tail;
Same for these.
> struct event_node nodes[C2HEVENT_SZ];
> unsigned char seq;
> };
> @@ -92,8 +95,10 @@ struct c2hevent_queue {
> #define NETWORK_QUEUE_SZ 4
>
> struct network_queue {
> - /*volatile*/ int head;
> - /*volatile*/ int tail;
> + /*volatile*/
> + int head;
> + /*volatile*/
> + int tail;
> struct wlan_bssid_ex networks[NETWORK_QUEUE_SZ];
> };
>
> diff --git a/drivers/staging/rtl8712/rtl871x_io.h b/drivers/staging/rtl8712/rtl871x_io.h
> index c20dd5a6bbd1..2e269b71072c 100644
> --- a/drivers/staging/rtl8712/rtl871x_io.h
> +++ b/drivers/staging/rtl8712/rtl871x_io.h
> @@ -101,7 +101,8 @@ struct _io_ops {
> struct io_req {
> struct list_head list;
> u32 addr;
> - /*volatile*/ u32 val;
> + /*volatile*/
> + u32 val;
> u32 command;
> u32 status;
> u8 *pbuf;
> diff --git a/drivers/staging/rtl8712/rtl871x_pwrctrl.h b/drivers/staging/rtl8712/rtl871x_pwrctrl.h
> index dd5a79f90b1a..e6b740c09120 100644
> --- a/drivers/staging/rtl8712/rtl871x_pwrctrl.h
> +++ b/drivers/staging/rtl8712/rtl871x_pwrctrl.h
> @@ -77,14 +77,19 @@ struct reportpwrstate_parm {
>
> struct pwrctrl_priv {
> struct mutex mutex_lock;
> - /*volatile*/ u8 rpwm; /* requested power state for fw */
> + /*volatile*/
> + u8 rpwm; /* requested power state for fw */
> /* fw current power state. updated when 1. read from HCPWM or
> * 2. driver lowers power level
> */
> - /*volatile*/ u8 cpwm;
> - /*volatile*/ u8 tog; /* toggling */
> - /*volatile*/ u8 cpwm_tog; /* toggling */
> - /*volatile*/ u8 tgt_rpwm; /* wanted power state */
> + /*volatile*/
> + u8 cpwm;
> + /*volatile*/
> + u8 tog; /* toggling */
> + /*volatile*/
> + u8 cpwm_tog; /* toggling */
> + /*volatile*/
> + u8 tgt_rpwm; /* wanted power state */
> uint pwr_mode;
> uint smart_ps;
> uint alives;
> diff --git a/drivers/staging/rtl8712/rtl871x_xmit.h b/drivers/staging/rtl8712/rtl871x_xmit.h
> index f227828094bf..6d2e12b35670 100644
> --- a/drivers/staging/rtl8712/rtl871x_xmit.h
> +++ b/drivers/staging/rtl8712/rtl871x_xmit.h
> @@ -182,11 +182,16 @@ struct sta_xmit_priv {
> };
>
> struct hw_txqueue {
> - /*volatile*/ sint head;
> - /*volatile*/ sint tail;
> - /*volatile*/ sint free_sz; /*in units of 64 bytes*/
> - /*volatile*/ sint free_cmdsz;
> - /*volatile*/ sint txsz[8];
> + /*volatile*/
> + sint head;
> + /*volatile*/
> + sint tail;
> + /*volatile*/
> + sint free_sz; /*in units of 64 bytes*/
> + /*volatile*/
> + sint free_cmdsz;
> + /*volatile*/
> + sint txsz[8];
> uint ff_hwaddr;
> uint cmd_hwaddr;
> sint ac_tag;
> --
> 2.27.0
>
Actually same for all of these, the code is fine as-is.
thanks,
greg k-h
prev parent reply other threads:[~2020-07-16 7:04 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-07-16 6:41 [PATCH] staging: rtl8712: Fixes coding style in several headers Qiu Wenbo
2020-07-16 7:04 ` Greg Kroah-Hartman [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200716070416.GA962748@kroah.com \
--to=gregkh@linuxfoundation.org \
--cc=Larry.Finger@lwfinger.net \
--cc=christophe.jaillet@wanadoo.fr \
--cc=dan.carpenter@oracle.com \
--cc=devel@driverdev.osuosl.org \
--cc=driverdev-devel@linuxdriverproject.org \
--cc=florian.c.schilhabel@googlemail.com \
--cc=git@marvil07.net \
--cc=keescook@chromium.org \
--cc=linux-kernel@vger.kernel.org \
--cc=nishkadg.linux@gmail.com \
--cc=qiuwenbo@phytium.com.cn \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox