Hi Rabin,
On ven., avril 08 2016, Rabin Vincent <rabin@xxxxxx> wrote:
> On Thu, Mar 31, 2016 at 11:09:39AM +0200, Gregory CLEMENT wrote:
>> diff --git a/arch/arm/mm/dma-mapping.c b/arch/arm/mm/dma-mapping.c
>> index deac58d5f1f7..1538bb293e90 100644
>> --- a/arch/arm/mm/dma-mapping.c
>> +++ b/arch/arm/mm/dma-mapping.c
>> @@ -61,7 +61,7 @@ struct arm_dma_free_args {
>>
>> struct arm_dma_allocator {
>> void *(*alloc)(struct arm_dma_alloc_args *args,
>> - struct page **ret_page);
>> + struct page **ret_page, bool is_coherent);
>> void (*free)(struct arm_dma_free_args *args);
>> };
>
> Could you please add this parameter to the arm_dma_alloc_args structure
> instead? As you see from the code, that's where all the other various
> parameters needed by the allocators are grouped.
I will do it inthe 8th version, it indeed simplify a little the patch.
Thanks,
Gregory
--
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html