Hello Mark A. Greer, The patch 1961843ceeca: "NFC: trf7970a: Handle timeout values of zero" from Sep 2, 2014, leads to the following static checker warning: drivers/nfc/trf7970a.c:897 trf7970a_irq() warn: passing zero to 'ERR_PTR' drivers/nfc/trf7970a.c 892 trf7970a_cmd(trf, TRF7970A_CMD_FIFO_RESET); 893 894 if (!trf->timeout) { 895 trf->ignore_timeout = !cancel_delayed_work( 896 &trf->timeout_work); 897 trf->rx_skb = ERR_PTR(0); ^^^^^^^^^^ Why not just say "NULL"? 898 trf7970a_send_upstream(trf); 899 break; 900 } regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html