Re: [PATCH] tc: fix warning and coding style

2014-09-30 Thread Thibaut Robert
Le mardi 30 septembre 2014 à 16:08 +0200, Geert Uytterhoeven a écrit : > I guess that's acceptable, as tc is probably limited to 32-bit anyway. Maciej? > > Note that there's also %pa, for phys_addr_t/resource_size_t. Yes, but unfortunately that needs a reference so we can't use it. Thibaut -- To

Re: [PATCH] tc: fix warning and coding style

2014-09-30 Thread Thibaut Robert
Hi Geert, Le mardi 30 septembre 2014 à 14:28 +0200, Geert Uytterhoeven a écrit : > Hi Thibaut, > > On Tue, Sep 30, 2014 at 2:16 PM, Thibaut Robert > wrote: > > Fix gcc warning: > > warning: format ‘%d’ expects argument of type ‘int’, but argument 4 has > > ty

[PATCH] tc: fix warning and coding style

2014-09-30 Thread Thibaut Robert
alues Signed-off-by: Thibaut Robert --- drivers/tc/tc.c | 28 1 file changed, 12 insertions(+), 16 deletions(-) diff --git a/drivers/tc/tc.c b/drivers/tc/tc.c index 9465623..9b1f831 100644 --- a/drivers/tc/tc.c +++ b/drivers/tc/tc.c @@ -12,6 +12,7 @@ #include

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/gdm

[PATCH] staging: gdm72xx: fix typo and coding style

2014-09-30 Thread Thibaut Robert
Fix a typo in error message. Add missing curling braces to conform to coding style. Improve readibility. Signed-off-by: Thibaut Robert --- drivers/staging/gdm72xx/netlink_k.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/gdm72xx/netlink_k.c

[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

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 fo

[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