Ezequiel Garcia <ezequiel.gar...@free-electrons.com> writes:

>> +static irqreturn_t pxa3xx_nand_irq_thread(int irq, void *data)
>> +{
>> +    struct pxa3xx_nand_info *info = data;
>> +
>> +    handle_data_pio(info);
>> +
>> +    info->state = STATE_CMD_DONE;
>
> Are you sure you need to set the state here?
Euh no, I'm not. I made it to be symmetric with dma_complete_func(). If it's not
needed, why is it needed in dma_complete_func() ?

Cheers.

-- 
Robert
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to