lkml.org 
[lkml]   [2015]   [May]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [V2 PATCH 2/5] arm64 : Introduce support for ACPI _CCA object
On 5/5/2015 10:44 AM, Arnd Bergmann wrote:
> On Tuesday 05 May 2015 10:12:06 Suravee Suthikulpanit wrote:
>> +struct dma_map_ops dummy_dma_ops = {
>> + .alloc = __dummy_alloc,
>> + .free = __dummy_free,
>> + .mmap = __dummy_mmap,
>> + .map_page = __dummy_map_page,
>> + .unmap_page = __dummy_unmap_page,
>> + .map_sg = __dummy_map_sg,
>> + .unmap_sg = __dummy_unmap_sg,
>> + .sync_single_for_cpu = __dummy_sync_single_for_cpu,
>> + .sync_single_for_device = __dummy_sync_single_for_device,
>> + .sync_sg_for_cpu = __dummy_sync_sg_for_cpu,
>> + .sync_sg_for_device = __dummy_sync_sg_for_device,
>> + .mapping_error = __dummy_mapping_error,
>> + .dma_supported = __dummy_dma_supported,
>> +};
>> +EXPORT_SYMBOL(dummy_dma_ops);
>> +
>>
>
> This will clearly work, but I think it's easier to just leave
> the dma_mask pointer as NULL when creating the platform device,
> which should let the normal dma ops fail all the callbacks.
>
> Arnd
>

However, codes in several places are making use of dma_map_ops without
checking if the ops are NULL (i.e.
include/asm-generic/dma-mapping-common.h and in arch-specific
implementation). If setting it to NULL is what we are planning to
support, we would need to scrub the current code to put NULL check.
Also, would you consider if that is safe to do going forward?

Thanks,
Suravee



\
 
 \ /
  Last update: 2015-05-05 19:41    [W:0.088 / U:0.352 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site