Le 21/07/2014 06:13, David Miller a écrit :
> From: Cyrille Pitchen <cyrille.pitc...@atmel.com>
> Date: Fri, 18 Jul 2014 16:21:14 +0200
> 
>> +            if (tx_skb->mapped_as_page) {
>> +                    dma_unmap_page(&bp->pdev->dev, tx_skb->mapping,
>> +                                   tx_skb->size, DMA_TO_DEVICE);
>> +            } else {
>> +                    dma_unmap_single(&bp->pdev->dev, tx_skb->mapping,
>> +                                     tx_skb->size, DMA_TO_DEVICE);
>> +            }
> 
> Please do not use curly braces for basic block composed of a single statement.
> 
Hi David,

thanks for your review, I'll fix it in the next series of patch.

Regards,

Cyrille
--
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