Re: [PATCH] Staging: gdm72xx: fix coding style

2014-09-30 Thread Thibaut Robert
Le Tuesday 30 Sep 2014 à 12:23:12 (+0200), Thibaut Robert a écrit : > Remove unnecessary else after return. > Follow coding style for curly braces around single statements. > Fix a typo in error message. > > Signed-off-by: Thibaut Robert > --- > drivers/staging/gdm72xx/netlink_k.c | 24 +

[PATCH] Staging: gdm72xx: fix coding style

2014-09-30 Thread Thibaut Robert
Remove unnecessary else after return. Follow coding style for curly braces around single statements. Fix a typo in error message. Signed-off-by: Thibaut Robert --- drivers/staging/gdm72xx/netlink_k.c | 24 +++- 1 file changed, 11 insertions(+), 13 deletions(-) diff --git a/d

Re: [PATCH] Staging: gdm72xx: fix coding style

2014-09-30 Thread Thibaut Robert
Le Monday 29 Sep 2014 à 11:31:24 (-0700), Joe Perches a écrit : > On Mon, 2014-09-29 at 20:15 +0200, Thibaut Robert wrote: > > This patch remove a checkstyle warning in netlink_k.c. > > Note: This is my homework of the day. > > The "Note: " line isn't useful for a commit message. > Neither is the

Re: [PATCH] Staging: gdm72xx: fix coding style

2014-09-29 Thread Joe Perches
On Mon, 2014-09-29 at 20:15 +0200, Thibaut Robert wrote: > This patch remove a checkstyle warning in netlink_k.c. > Note: This is my homework of the day. The "Note: " line isn't useful for a commit message. Neither is the checkpatch warning bit. Maybe the commit message could be something like: "

[PATCH] Staging: gdm72xx: fix coding style

2014-09-29 Thread Thibaut Robert
This patch remove a checkstyle warning in netlink_k.c. Note: This is my homework of the day. Signed-off-by: Thibaut Robert --- drivers/staging/gdm72xx/netlink_k.c | 13 ++--- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/drivers/staging/gdm72xx/netlink_k.c b/drivers/stag