Quentin LAMBERT
Author's posts
Aug 07 2014
brcmfmac: fix memory leakage in msgbuf
Author: Arend van Spriel <arend@broadcom.com> The kbuild robot came up with the following warning: tree: …/kernel/git/linville/wireless-next.git master head: dc6be9f54a4ecb0a09765d1f515ed947d86b7528 commit: 9a1bb60250d2b6b546a62e5b73f55c4f1d22016b [5/13] brcmfmac: Adding msgbuf protocol. coccinelle warnings: drivers/net/wireless/brcm80211/brcmfmac/msgbuf.c:1309:1-28: alloc with no test, possible model on line 1318 Looking into the issue, it turned out that the referred allocation buffer was not being released in …
Aug 07 2014
kernel/printk/printk.c: fix bool assignements
Author: Neil Zhang <zhangwm@marvell.com> Fix coccinelle warnings. Signed-off-by: Neil Zhang Signed-off-by: Andrew Morton Signed-off-by: Linus Torvalds — kernel/printk/printk.c | 6 +++— 1 file changed, 3 insertions(+), 3 deletions(-) diff –git a/kernel/printk/printk.c b/kernel/printk/printk.c index df202fe..de1a6bb 100644 — a/kernel/printk/printk.c +++ b/kernel/printk/printk.c @@ -919,7 +919,7 @@ static bool __read_mostly ignore_loglevel; static int __init ignore_loglevel_setup(char *str) { …
Aug 07 2014
drivers: video: fbdev: atmel_lcdfb.c: fix error return code
Author: Julia Lawall <Julia.Lawall@lip6.fr> Convert a zero return value on error to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // ( if@p1 (\(ret < 0\|ret != 0\)) { ... return ret; } | ret@p1 = 0 ) ... …
Aug 07 2014
mfd: twl6040: Fix error return code
Author: Julia Lawall <Julia.Lawall@lip6.fr> Convert a zero return value on error to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // ( if@p1 (\(ret < 0\|ret != 0\)) { ... return ret; } | ret@p1 = 0 ) ... …
Aug 06 2014
pcmcia/vrc4171: Remove typedefs for enums and struct
Author: Himangi Saraogi <himangi774@gmail.com> The Linux kernel coding style guidelines suggest not using typedefs for structure and enum types. This patch gets rid of the typedefs for vrc4171_slot_t, vrc4171_slotb_t and vrc4171_socket_t. Also, the names of the enums and the struct are changed to drop the _t, to make the name look less typedef-like. The following …
Aug 06 2014
pcmcia: Remove typedef in structs and emum
Author: Himangi Saraogi <himangi774@gmail.com> The Linux kernel coding style guidelines suggest not using typedefs for structure and enum types. This patch gets rid of the typedefs for cirrus_state_t, vg46x_state_t and pcic_id. Also, the names of the structs are changed to drop the _t, to make the name look less typedef-like. The following Coccinelle semantic patch …
Aug 06 2014
sound: oss: Remove typedefs wanc_info and wavnc_port_info
Author: Himangi Saraogi <himangi774@gmail.com> The Linux kernel coding style guidelines suggest not using typedefs for structure types. This patch gets rid of the typedefs for wanc_info and wavnc_port_info. A simplified version of the Coccinelle semantic patch that finds the case is: @tn@ identifier i; type td; @@ -typedef struct i { … } -td ; …
Aug 06 2014
sound: oss: uart401: Remove typedef uart401_devc
Author: Himangi Saraogi <himangi774@gmail.com> The Linux kernel coding style guidelines suggest not using typedefs for structure types. This patch gets rid of the typedef for uart401_devc. The following Coccinelle semantic patch detects the case. @tn@ identifier i; type td; @@ -typedef struct i { … } -td ; @@ type tn.td; identifier tn.i; @@ -td …
Aug 06 2014
pcmcia: Remove typedef tuple_flags
Author: Himangi Saraogi <himangi774@gmail.com> The Linux kernel coding style guidelines suggest not using typedefs for structure types. This patch gets rid of the typedef for tuple_flags. The following Coccinelle semantic patch makes the transformation. @tn@ identifier i; type td; @@ -typedef struct i { … } -td ; @@ type tn.td; identifier tn.i; @@ -td …
Aug 06 2014
drm: Reduce the amount of dev->vblank[crtc] in the code
Author: Ville Syrjälä <ville.syrjala@linux.intel.com> Declare a local struct drm_vblank_crtc * and use that instead of having to do dig it out via ‘dev->vblank[crtc]’ everywhere. Performed with the following coccinelle incantation, and a few manual whitespace cleanups: @@ identifier func,member; expression num_crtcs; struct drm_device *dev; unsigned int crtc; @@ func (…) { + struct drm_vblank_crtc *vblank; …