Quentin LAMBERT
Author's posts
Jan 21 2013
IB/mlx4: Adjust duplicate test
Author: Julia Lawall <Julia.Lawall@lip6.fr> Delete successive tests to the same location. The code tested the result of a previous allocation, that itself was already tested. It is changed to test the result of the most recent allocation. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @s exists@ …
Jan 21 2013
drivers/ipack/devices/ipoctal.c: adjust duplicate test
Author: Julia Lawall <Julia.Lawall@lip6.fr> Delete successive tests to the same location. The code tested the result of a previous allocation, that itself was already tested. It is changed to test the result of the most recent allocation. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @s exists@ …
Jan 21 2013
arch/powerpc/platforms/85xx/p1022_ds.c: adjust duplicate test
Author: Julia Lawall <Julia.Lawall@lip6.fr> Delete successive tests to the same location. The code tested the result of a previous call, that itself was already tested. It is changed to test the result of the most recent call. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @s exists@ …
Jan 21 2013
crypto: atmel-aes – adjust duplicate test
Author: Julia Lawall <Julia.Lawall@lip6.fr> Delete successive tests to the same location. The code tested the result of a previous allocation, that itself was already tested. It is changed to test the result of the most recent allocation. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @s exists@ …
Jan 21 2013
drivers/net/wireless/iwlegacy/4965-mac.c: adjust duplicate test
Author: Julia Lawall <Julia.Lawall@lip6.fr> Delete successive tests to the same location. This looks like simple code duplication. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @s exists@ local idexpression y; expression x,e; @@ *if ( \(x == NULL\|IS_ERR(x)\|y != 0\) ) { … when forall return …; …
Jan 21 2013
[SCSI] bnx2fc: adjust duplicate test
Author: Julia Lawall <Julia.Lawall@lip6.fr> Delete successive tests to the same location. The code tested the result of a previous allocation, that itself was already tested. It is changed to test the result of the most recent allocation. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @s exists@ …
Jan 21 2013
staging: omap-thermal/omap-bandgap.c: adjust duplicate test
Author: Julia Lawall <Julia.Lawall@lip6.fr> Delete successive tests to the same location. There was a previous test on ret and it has not been updated since then. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @s exists@ local idexpression y; expression x,e; @@ *if ( \(x == NULL\|IS_ERR(x)\|y …
Jan 21 2013
drivers/usb/chipidea/core.c: adjust duplicate test
Author: Julia Lawall <Julia.Lawall@lip6.fr> Delete successive tests to the same location. In this case res has already been tested for being NULL, and calling devm_request_and_ioremap will not make it NULL. On the other hand, devm_request_and_ioremap can return NULL on failure. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) …
Jan 21 2013
drivers/net/ethernet/sfc/ptp.c: adjust duplicate test
Author: Julia Lawall <Julia.Lawall@lip6.fr> Delete successive tests to the same location. rc was previously tested and not subsequently updated. efx_phc_adjtime can return an error code, so the call is updated so that is tested instead. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @s exists@ local idexpression …
Jan 09 2013
spi: Add helper functions for setting up transfers
Author: Lars-Peter Clausen <lars@metafoo.de> Quite often the pattern used for setting up and transferring a synchronous SPI transaction looks very much like the following: struct spi_message msg; struct spi_transfer xfers[] = { … }; spi_message_init(&msg); spi_message_add_tail(&xfers[0], &msg); … spi_message_add_tail(&xfers[ARRAY_SIZE(xfers) – 1], &msg); ret = spi_sync(&msg); This patch adds two new helper functions for handling this …