From: Juston Li <juston.h.li@gmail.com>
To: sudipm.mukherjee@gmail.com, teddy.wang@siliconmotion.com,
gregkh@linuxfoundation.org, devel@driverdev.osuosl.org,
linux-fbdev@vger.kernel.org, linux-kernel@vger.kernel.org
Cc: Juston Li <juston.h.li@gmail.com>
Subject: [PATCH v5 09/19] staging: sm750fb: add space after struct definition
Date: Wed, 15 Jul 2015 04:14:38 +0000 [thread overview]
Message-ID: <1436933688-5408-9-git-send-email-juston.h.li@gmail.com> (raw)
In-Reply-To: <CAFow2F9-0W4VRjLHmXZuv2vCwyGYRDatbCOrmg5o-AY0-nb-dw@mail.gmail.com>
Fixes checkpatch.pl warning:
WARNING: missing space after struct definition
Signed-off-by: Juston Li <juston.h.li@gmail.com>
---
drivers/staging/sm750fb/sm750.h | 12 ++++++------
drivers/staging/sm750fb/sm750_hw.h | 6 +++---
2 files changed, 9 insertions(+), 9 deletions(-)
diff --git a/drivers/staging/sm750fb/sm750.h b/drivers/staging/sm750fb/sm750.h
index 8ac5410..9989ff6 100644
--- a/drivers/staging/sm750fb/sm750.h
+++ b/drivers/staging/sm750fb/sm750.h
@@ -14,7 +14,7 @@
extern int smi_indent;
-struct lynx_accel{
+struct lynx_accel {
/* base virtual address of DPR registers */
volatile unsigned char __iomem * dprBase;
/* base virtual address of de data port */
@@ -41,7 +41,7 @@ struct lynx_accel{
/* lynx_share stands for a presentation of two frame buffer
that use one smi adaptor , it is similar to a basic class of C++
*/
-struct lynx_share{
+struct lynx_share {
/* common members */
u16 devid;
u8 revid;
@@ -68,7 +68,7 @@ struct lynx_share{
void (*resume)(struct lynx_share*);
};
-struct lynx_cursor{
+struct lynx_cursor {
/* cursor width ,height and size */
int w;
int h;
@@ -92,7 +92,7 @@ struct lynx_cursor{
void (*setData)(struct lynx_cursor *, u16, const u8*, const u8*);
};
-struct lynxfb_crtc{
+struct lynxfb_crtc {
unsigned char __iomem *vCursor; /* virtual address of cursor */
unsigned char __iomem *vScreen; /* virtual address of on_screen */
int oCursor; /* cursor address offset in vidmem */
@@ -123,7 +123,7 @@ struct lynxfb_crtc{
struct lynx_cursor cursor;
};
-struct lynxfb_output{
+struct lynxfb_output {
int dpms;
int paths;
/* which paths(s) this output stands for,for sm750:
@@ -149,7 +149,7 @@ struct lynxfb_output{
void (*clear)(struct lynxfb_output*);
};
-struct lynxfb_par{
+struct lynxfb_par {
/* either 0 or 1 for dual head adaptor,0 is the older one registered */
int index;
unsigned int pseudo_palette[256];
diff --git a/drivers/staging/sm750fb/sm750_hw.h b/drivers/staging/sm750fb/sm750_hw.h
index af2db7c..ef0a16f 100644
--- a/drivers/staging/sm750fb/sm750_hw.h
+++ b/drivers/staging/sm750fb/sm750_hw.h
@@ -42,7 +42,7 @@ enum sm750_path {
sm750_pnc = 3,/* panel and crt */
};
-struct init_status{
+struct init_status {
ushort powerMode;
/* below three clocks are in unit of MHZ*/
ushort chip_clk;
@@ -52,7 +52,7 @@ struct init_status{
ushort resetMemory;
};
-struct sm750_state{
+struct sm750_state {
struct init_status initParm;
enum sm750_pnltype pnltype;
enum sm750_dataflow dataflow;
@@ -66,7 +66,7 @@ struct sm750_state{
in C++
*/
-struct sm750_share{
+struct sm750_share {
/* it's better to put lynx_share struct to the first place of sm750_share */
struct lynx_share share;
struct sm750_state state;
--
2.4.5
next prev parent reply other threads:[~2015-07-15 4:14 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-07-15 4:10 [PATCH v5 00/19] staging: sm750fb: checkpatch.pl fixes Juston Li
2015-07-15 4:14 ` [PATCH v5 01/19] staging: sm750fb: use tabs for indentation Juston Li
2015-07-15 4:14 ` [PATCH v5 02/19] staging: sm750fb: remove spacing after open parenthesis Juston Li
2015-07-15 4:14 ` [PATCH v5 03/19] staging: sm750fb: remove space before close parenthesis Juston Li
2015-07-15 4:14 ` [PATCH v5 04/19] staging: sm750fb: add space before open parenthesis Juston Li
2015-07-15 4:14 ` [PATCH v5 05/19] staging: sm750fb: remove space between function name and parenthesis Juston Li
2015-07-15 4:14 ` [PATCH v5 06/19] staging: sm750fb: add space before open brace Juston Li
2015-07-15 4:14 ` [PATCH v5 07/19] staging: sm750fb: add space after close brace Juston Li
2015-07-15 4:14 ` [PATCH v5 08/19] staging: sm750fb: add space after enum definition Juston Li
2015-07-15 4:14 ` Juston Li [this message]
2015-07-15 4:14 ` [PATCH v5 10/19] staging: sm750fb: add space after return type Juston Li
2015-07-15 4:14 ` [PATCH v5 11/19] staging: sm750fb: consistent spacing around operators Juston Li
2015-07-15 4:14 ` [PATCH v5 12/19] staging: sm750fb: add spaces " Juston Li
2015-07-15 4:14 ` [PATCH v5 13/19] staging: sm750fb: add space after semicolon Juston Li
2015-07-15 4:14 ` [PATCH v5 14/19] staging: sm750fb: remove trailing whitespace Juston Li
2015-07-15 4:14 ` [PATCH v5 15/19] staging: sm750fb: remove unnecessary whitespace Juston Li
2015-07-15 4:14 ` [PATCH v5 16/19] staging: sm750fb: fix brace placement Juston Li
2015-07-15 4:14 ` [PATCH v5 17/19] staging: sm750fb: move while to follow do close brace Juston Li
2015-07-15 4:14 ` [PATCH v5 18/19] staging: sm750fb: remove unnecessary braces Juston Li
2015-07-15 4:14 ` [PATCH v5 19/19] staging: sm750fb: add missing blank line after declarations Juston Li
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=1436933688-5408-9-git-send-email-juston.h.li@gmail.com \
--to=juston.h.li@gmail.com \
--cc=devel@driverdev.osuosl.org \
--cc=gregkh@linuxfoundation.org \
--cc=linux-fbdev@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=sudipm.mukherjee@gmail.com \
--cc=teddy.wang@siliconmotion.com \
/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;
as well as URLs for NNTP newsgroup(s).