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 +

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: "