From: Julia Lawall Date: Mon, 21 Jan 2013 13:02:46 +0000 (+0100) Subject: drivers/usb/chipidea/core.c: adjust duplicate test X-Git-Tag: v3.9-rc1~126^2~54 X-Git-Url: http://git.openpandora.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=5c6e9bf011bca072648017477b0fe36402be1415;p=pandora-kernel.git drivers/usb/chipidea/core.c: adjust duplicate test 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/) // @s exists@ local idexpression y; expression x,e; @@ *if ( \(x == NULL\|IS_ERR(x)\|y != 0\) ) { ... when forall return ...; } ... when != \(y = e\|y += e\|y -= e\|y |= e\|y &= e\|y++\|y--\|&y\) when != \(XT_GETPAGE(...,y)\|WMI_CMD_BUF(...)\) *if ( \(x == NULL\|IS_ERR(x)\|y != 0\) ) { ... when forall return ...; } // Signed-off-by: Julia Lawall Cc: Alexander Shishkin Signed-off-by: Greg Kroah-Hartman --- Reading git-diff-tree failed