* [PATCH v2 0/2] ATM: HE: Cleanup coding style
@ 2014-12-16 0:26 Jonathan Jin
2014-12-16 0:26 ` [PATCH v2 1/2] ATM: HE: Cleanup coding style (whitespace) Jonathan Jin
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Jonathan Jin @ 2014-12-16 0:26 UTC (permalink / raw)
To: kernelnewbies
At Greg KH's advice, I've revised my previous formatting patch by splicing it
into two separate patches in the same sequence. If I could get additional
feedback, I'd very much appreciate it.
This patch sequence cleans up the following coding style issues that are
detected by checkpatch:
ERROR: trailing whitespace
ERROR: open brace '{' following struct go on the same line
Jonathan Jin (2):
ATM: HE: Cleanup coding style (whitespace)
ATM: HE: Cleanup coding style (struct braces)
drivers/atm/he.h | 25 +++++++++++--------------
1 file changed, 11 insertions(+), 14 deletions(-)
--
2.1.3
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2 1/2] ATM: HE: Cleanup coding style (whitespace)
2014-12-16 0:26 [PATCH v2 0/2] ATM: HE: Cleanup coding style Jonathan Jin
@ 2014-12-16 0:26 ` Jonathan Jin
2014-12-16 0:26 ` [PATCH v2 2/2] ATM: HE: Cleanup coding style (struct braces) Jonathan Jin
2014-12-16 0:40 ` [PATCH v2 0/2] ATM: HE: Cleanup coding style Greg KH
2 siblings, 0 replies; 6+ messages in thread
From: Jonathan Jin @ 2014-12-16 0:26 UTC (permalink / raw)
To: kernelnewbies
Address checkpatch.pl complaints regarding trailing whitespace
Signed-off-by: Jonathan Jin <jjin082693@gmail.com>
---
drivers/atm/he.h | 16 ++++++++--------
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/drivers/atm/he.h b/drivers/atm/he.h
index 110a27d..160a315 100644
--- a/drivers/atm/he.h
+++ b/drivers/atm/he.h
@@ -191,9 +191,9 @@ struct he_tpdrq {
struct he_hsp {
struct he_hsp_entry {
- volatile u32 tbrq_tail;
+ volatile u32 tbrq_tail;
volatile u32 reserved1[15];
- volatile u32 rbrq_tail;
+ volatile u32 rbrq_tail;
volatile u32 reserved2[15];
} group[HE_NUM_GROUPS];
};
@@ -457,7 +457,7 @@ struct he_vcc
#define G0_RBPS_BS 0x8040c
#define G0_RBPL_S 0x80410
#define G0_RBPL_T 0x80414
-#define G0_RBPL_QI 0x80418
+#define G0_RBPL_QI 0x80418
#define G0_RBPL_BS 0x8041c
#define G1_RBPS_S 0x80420
@@ -616,11 +616,11 @@ struct he_vcc
#define TM_RW_WAIT(x) (x<<2)
#define TM_SRAM_TYPE(x) (x)
-#define TSRB_BA 0x80744
-#define TSRC_BA 0x80748
-#define TMABR_BA 0x8074c
-#define TPD_BA 0x80750
-#define TSRD_BA 0x80758
+#define TSRB_BA 0x80744
+#define TSRC_BA 0x80748
+#define TMABR_BA 0x8074c
+#define TPD_BA 0x80750
+#define TSRD_BA 0x80758
#define TX_CONFIG 0x80760
#define DRF_THRESH(x) (x<<22)
--
2.1.3
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH v2 2/2] ATM: HE: Cleanup coding style (struct braces)
2014-12-16 0:26 [PATCH v2 0/2] ATM: HE: Cleanup coding style Jonathan Jin
2014-12-16 0:26 ` [PATCH v2 1/2] ATM: HE: Cleanup coding style (whitespace) Jonathan Jin
@ 2014-12-16 0:26 ` Jonathan Jin
2014-12-16 0:39 ` Greg KH
2014-12-16 0:40 ` [PATCH v2 0/2] ATM: HE: Cleanup coding style Greg KH
2 siblings, 1 reply; 6+ messages in thread
From: Jonathan Jin @ 2014-12-16 0:26 UTC (permalink / raw)
To: kernelnewbies
Address checkpatch.pl complaints regarding proper struct brace placement
Signed-off-by: Jonathan Jin <jjin082693@gmail.com>
---
drivers/atm/he.h | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/drivers/atm/he.h b/drivers/atm/he.h
index 160a315..a9750c6 100644
--- a/drivers/atm/he.h
+++ b/drivers/atm/he.h
@@ -236,13 +236,11 @@ struct he_group {
#define HE_LOOKUP_VCC(dev, cid) ((dev)->he_vcc_table[(cid)].vcc)
-struct he_vcc_table
-{
+struct he_vcc_table {
struct atm_vcc *vcc;
};
-struct he_cs_stper
-{
+struct he_cs_stper {
long pcr;
int inuse;
};
@@ -316,8 +314,7 @@ struct he_dev {
#define HE_MAXIOV 20
-struct he_vcc
-{
+struct he_vcc {
struct list_head buffers;
int pdu_len;
int rc_index;
--
2.1.3
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH v2 2/2] ATM: HE: Cleanup coding style (struct braces)
2014-12-16 0:26 ` [PATCH v2 2/2] ATM: HE: Cleanup coding style (struct braces) Jonathan Jin
@ 2014-12-16 0:39 ` Greg KH
0 siblings, 0 replies; 6+ messages in thread
From: Greg KH @ 2014-12-16 0:39 UTC (permalink / raw)
To: kernelnewbies
On Mon, Dec 15, 2014 at 06:26:41PM -0600, Jonathan Jin wrote:
> Address checkpatch.pl complaints regarding proper struct brace placement
Minor nit, why the extra spaces at the front of the line?
And properly wrap your lines at 72 columns, like git should have told
you to in the editor.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2 0/2] ATM: HE: Cleanup coding style
2014-12-16 0:26 [PATCH v2 0/2] ATM: HE: Cleanup coding style Jonathan Jin
2014-12-16 0:26 ` [PATCH v2 1/2] ATM: HE: Cleanup coding style (whitespace) Jonathan Jin
2014-12-16 0:26 ` [PATCH v2 2/2] ATM: HE: Cleanup coding style (struct braces) Jonathan Jin
@ 2014-12-16 0:40 ` Greg KH
2014-12-16 1:08 ` Jonathan Jin
2 siblings, 1 reply; 6+ messages in thread
From: Greg KH @ 2014-12-16 0:40 UTC (permalink / raw)
To: kernelnewbies
On Mon, Dec 15, 2014 at 06:26:39PM -0600, Jonathan Jin wrote:
> At Greg KH's advice, I've revised my previous formatting patch by splicing it
> into two separate patches in the same sequence. If I could get additional
> feedback, I'd very much appreciate it.
>
> This patch sequence cleans up the following coding style issues that are
> detected by checkpatch:
> ERROR: trailing whitespace
> ERROR: open brace '{' following struct go on the same line
>
> Jonathan Jin (2):
> ATM: HE: Cleanup coding style (whitespace)
> ATM: HE: Cleanup coding style (struct braces)
>
> drivers/atm/he.h | 25 +++++++++++--------------
> 1 file changed, 11 insertions(+), 14 deletions(-)
Looks good, just the one nit about the leading spaces in the changelog
entry.
But note, some maintainers don't care about patches like this, so if
they are ignored, be prepared. If they don't get picked up in a month
or so, try a subsystem that wants these types of patches, like the
staging tree.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2 0/2] ATM: HE: Cleanup coding style
2014-12-16 0:40 ` [PATCH v2 0/2] ATM: HE: Cleanup coding style Greg KH
@ 2014-12-16 1:08 ` Jonathan Jin
0 siblings, 0 replies; 6+ messages in thread
From: Jonathan Jin @ 2014-12-16 1:08 UTC (permalink / raw)
To: kernelnewbies
On Mon, Dec 15, 2014 at 04:40:19PM -0800, Greg KH wrote:
> But note, some maintainers don't care about patches like this, so if
> they are ignored, be prepared. If they don't get picked up in a month
> or so, try a subsystem that wants these types of patches, like the
> staging tree.
Thanks for the patience and advice, Greg. You're really doing a huge ton
to help an aspiring kernel dev out, and I really appreciate it.
I figured that I'd at least get a patch out onto this list and get it
torn apart first before then moving on to making an "actual" patch to
the staging tree.
I'll be looking into that soon; stay tuned.
And thanks again.
--
Jonathan Jin
jjin.me
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-12-16 1:08 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-12-16 0:26 [PATCH v2 0/2] ATM: HE: Cleanup coding style Jonathan Jin
2014-12-16 0:26 ` [PATCH v2 1/2] ATM: HE: Cleanup coding style (whitespace) Jonathan Jin
2014-12-16 0:26 ` [PATCH v2 2/2] ATM: HE: Cleanup coding style (struct braces) Jonathan Jin
2014-12-16 0:39 ` Greg KH
2014-12-16 0:40 ` [PATCH v2 0/2] ATM: HE: Cleanup coding style Greg KH
2014-12-16 1:08 ` Jonathan Jin
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.