Author: Jonathan Cameron <jic23@cam.ac.uk> Coccinelle found this one. I’m not certain what the intent is, but this fix makes no functional changes. Signed-off-by: Jonathan Cameron Signed-off-by: Greg Kroah-Hartman — drivers/staging/iio/addac/adt7316.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff –git a/drivers/staging/iio/addac/adt7316.c b/drivers/staging/iio/addac/adt7316.c index 637316f..c828b6f 100644 — a/drivers/staging/iio/addac/adt7316.c +++ b/drivers/staging/iio/addac/adt7316.c @@ -203,7 +203,7 …
Aug 08 2011
staging: brcm80211: fix for ‘remove unnecessary braces’ checkpatch warning
Author: Henry Ptasinski <henryp@broadcom.com> Removed unnecessary braces in single-statement blocks. Used the tools ‘uncrustify’ and ‘coccinelle’ to accomplish this. Signed-off-by: Henry Ptasinski Reviewed-by: Arend van Spriel Reviewed-by: Roland Vossen Signed-off-by: Arend van Spriel Signed-off-by: Greg Kroah-Hartman — drivers/staging/brcm80211/brcmfmac/bcmsdh_sdmmc.c | 20 +- drivers/staging/brcm80211/brcmfmac/dhd_common.c | 45 +- drivers/staging/brcm80211/brcmfmac/dhd_linux.c | 13 +- drivers/staging/brcm80211/brcmfmac/dhd_sdio.c | 64 +- drivers/staging/brcm80211/brcmfmac/wl_cfg80211.c | …
Aug 08 2011
drivers/net/wireless/wl1251: add missing kfree
Author: Julia Lawall <julia@diku.dk> In each case, the kfree already at the end of the function is also needed in the error case. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @exists@ local idexpression x; statement S,S1; expression E; identifier fl; expression *ptr != NULL; @@ x …
Aug 08 2011
arch/powerpc/sysdev/ehv_pic.c: add missing kfree
Author: Julia Lawall <julia@diku.dk> At this point, ehv_pic has been allocated but not stored anywhere, so it should be freed before leaving the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @exists@ local idexpression x; statement S,S1; expression E; identifier fl; expression *ptr != NULL; @@ …
Aug 08 2011
drivers/staging/bcm/nvm.c: add missing kfree
Author: Julia Lawall <julia@diku.dk> Buff is only used as a temporary buffer within the function, so it should be freed before leaving the function in an error case. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @exists@ local idexpression x; statement S,S1; expression E; identifier fl; expression …
Aug 08 2011
drivers/ata/sata_dwc_460ex.c: add missing kfree
Author: Julia Lawall <julia@diku.dk> Currently, error handling code in this function calls the function sata_dwc_port_stop, but this function has essentially no effect if hsdevp has not been stored in ap, which is the case throughout this function. The only effect is to print a debugging message including ap->print_id. The code is rewritten to not call …
Aug 08 2011
pseries/iommu: Add missing kfree
Author: Julia Lawall <julia@diku.dk> At this point, window has not been stored anywhere, so it has to be freed before leaving the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @exists@ local idexpression x; statement S,S1; expression E; identifier fl; expression *ptr != NULL; @@ x …
Aug 06 2011
staging: brcm80211: use kzalloc()
Author: Thomas Meyer <thomas@m3y3r.de> Use kzalloc rather than kmalloc followed by memset with 0 This considers some simple cases that are common and easy to validate Note in particular that there are no …s in the rule, so all of the matched code has to be contiguous The semantic patch that makes this output is …
Aug 06 2011
ALSA: asihpi – use kzalloc()
Author: Thomas Meyer <thomas@m3y3r.de> Use kzalloc rather than kmalloc followed by memset with 0 This considers some simple cases that are common and easy to validate Note in particular that there are no …s in the rule, so all of the matched code has to be contiguous The semantic patch that makes this output is …
Aug 06 2011
ARM: davinci: edma: use kzalloc()
Author: Thomas Meyer <thomas@m3y3r.de> Use kzalloc rather than kmalloc followed by memset with 0 This considers some simple cases that are common and easy to validate Note in particular that there are no …s in the rule, so all of the matched code has to be contiguous The semantic patch that makes this output is …
