tree: git://linuxtv.org/snawrocki/samsung.git for-v3.16 head: 13b46c7a03adbcc347b77a13ed27066bc92d515c commit: 192292403147877c7d5f737a3cc751ded397aef7 [45/81] [media] em28xx: add [2013:025f] PCTV tripleStick (292e) drivers/media/dvb-frontends/si2168.c:47 si2168_cmd_execute() warn: add some parenthesis here? drivers/media/dvb-frontends/si2168.c:47 si2168_cmd_execute() warn: maybe use && instead of & drivers/media/tuners/si2157.c:44 si2157_cmd_execute() warn: add some parenthesis here? drivers/media/tuners/si2157.c:44 si2157_cmd_execute() warn: maybe use && instead of & git remote add linuxtv-samsung git://linuxtv.org/snawrocki/samsung.git git remote update linuxtv-samsung git checkout 192292403147877c7d5f737a3cc751ded397aef7 vim +47 drivers/media/dvb-frontends/si2168.c 845f3505 Antti Palosaari 2014-04-10 31 goto err_mutex_unlock; 845f3505 Antti Palosaari 2014-04-10 32 } else if (ret != cmd->rlen) { 845f3505 Antti Palosaari 2014-04-10 33 ret = -EREMOTEIO; 845f3505 Antti Palosaari 2014-04-10 34 goto err_mutex_unlock; 845f3505 Antti Palosaari 2014-04-10 35 } 845f3505 Antti Palosaari 2014-04-10 36 845f3505 Antti Palosaari 2014-04-10 37 /* firmware ready? */ 845f3505 Antti Palosaari 2014-04-10 38 if ((cmd->args[0] >> 7) & 0x01) 845f3505 Antti Palosaari 2014-04-10 39 break; 845f3505 Antti Palosaari 2014-04-10 40 } 845f3505 Antti Palosaari 2014-04-10 41 845f3505 Antti Palosaari 2014-04-10 42 dev_dbg(&s->client->dev, "%s: cmd execution took %d ms\n", 845f3505 Antti Palosaari 2014-04-10 43 __func__, 845f3505 Antti Palosaari 2014-04-10 44 jiffies_to_msecs(jiffies) - 845f3505 Antti Palosaari 2014-04-10 45 (jiffies_to_msecs(timeout) - TIMEOUT)); 845f3505 Antti Palosaari 2014-04-10 46 845f3505 Antti Palosaari 2014-04-10 @47 if (!(cmd->args[0] >> 7) & 0x01) { This should be: if (!((md->args[0] >> 7) & 0x01)) { Otherwise it is a precedence error where it does the negate before the bitwise AND. 845f3505 Antti Palosaari 2014-04-10 48 ret = -ETIMEDOUT; 845f3505 Antti Palosaari 2014-04-10 49 goto err_mutex_unlock; 845f3505 Antti Palosaari 2014-04-10 50 } 845f3505 Antti Palosaari 2014-04-10 51 } 845f3505 Antti Palosaari 2014-04-10 52 845f3505 Antti Palosaari 2014-04-10 53 ret = 0; 845f3505 Antti Palosaari 2014-04-10 54 845f3505 Antti Palosaari 2014-04-10 55 err_mutex_unlock: --- 0-DAY kernel build testing backend Open Source Technology Center http://lists.01.org/mailman/listinfo/kbuild Intel Corporation -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html