On Wed, Oct 14, 2015 at 07:38:22PM +0530, Sakshi Bansal wrote: > Fixed allignmnet issues and block comments usage >
Split it apart by type of fix. > @@ -617,7 +618,7 @@ static irqreturn_t nvec_interrupt(int irq, void *dev) > } else { > nvec->rx = nvec_msg_alloc(nvec, NVEC_MSG_RX); > /* Should not happen in a normal world */ > - if (unlikely(nvec->rx == NULL)) { > + if (!unlikely(nvec->rx)) { This isn't right. You intented to say: if (unlikely(!nvec->rx)) { But even better to just remove the unlikely entirely. if (!nvec->rx) { > nvec->state = 0; > break; > } regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html