Re: [PATCH] ARM: DMA: Add missing semicolon for the define of dma_alloc_area

[Axel Lin]

2011/7/1 Sergei Shtylyov <sshtylyov@xxxxxxxxxx>:
> Hello.
>
> On 01-07-2011 12:37, Axel Lin wrote:
>
>> I got below build error by: make at91x40_defconfig;make
>> Fix it by adding a missing semicolon for the define of dma_alloc_area.
>
>>   CC      arch/arm/mm/dma-mapping.o
>> arch/arm/mm/dma-mapping.c:309: warning: 'struct dma_coherent_area'
>> declared inside parameter list
>> arch/arm/mm/dma-mapping.c:309: warning: its scope is only this definition
>> or declaration, which is probably not what you want
>> arch/arm/mm/dma-mapping.c: In function '__dma_alloc':
>> arch/arm/mm/dma-mapping.c:324: error: expected ';' before '}' token
>> arch/arm/mm/dma-mapping.c: At top level:
>> arch/arm/mm/dma-mapping.c:334: warning: 'struct dma_coherent_area'
>> declared inside parameter list
>> arch/arm/mm/dma-mapping.c:374: warning: 'struct dma_coherent_area'
>> declared inside parameter list
>> make[1]: *** [arch/arm/mm/dma-mapping.o] Error 1
>> make: *** [arch/arm/mm] Error 2
>
>   You didn't fix the warning though, so not clear why you cited them....
Because it's a copy-paste from the output of make.
I just think I should not manually edit the log.

Well, a patch to fix the warning is on the way.

Regards,
Axel
--
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/