Re: [PATCH 07/10] x86: add checks for alloc/free_coherent code

From: Ingo Molnar
Date: Fri Nov 21 2008 - 11:58:15 EST



please use this nice and clean structure init style:

> + entry->type = DMA_DEBUG_COHERENT;
> + entry->dev = dev;
> + entry->cpu_addr = virt;
> + entry->size = size;
> + entry->dev_addr = dma_addr;
> + entry->direction = DMA_BIDIRECTIONAL;

here too:

> + struct dma_debug_entry ref = {
> + .type = DMA_DEBUG_COHERENT,
> + .dev = dev,
> + .cpu_addr = virt,
> + .dev_addr = addr,
> + .size = size,
> + .direction = DMA_BIDIRECTIONAL,
> + };

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