[PATCH V3 1/3] arm: dma mapping: Export dma ops functions
Gregory CLEMENT
gregory.clement at free-electrons.com
Wed Nov 21 03:17:26 EST 2012
On 11/21/2012 09:12 AM, Marek Szyprowski wrote:
> Hello,
>
> On 11/21/2012 9:05 AM, Gregory CLEMENT wrote:
>> On 11/21/2012 08:06 AM, Marek Szyprowski wrote:
>>> On 11/20/2012 10:56 PM, Gregory CLEMENT wrote:
>>>> Expose the DMA operations functions. Until now only the dma_ops
>>>> structs in a whole or some dma operation were exposed. This patch
>>>> exposes all the dma coherents operations. They can be reused when an
>>>> architecture or a driver need to create its own set of dma_operation.
>>>>
>>>> Signed-off-by: Gregory CLEMENT <gregory.clement at free-electrons.com>
>>>
>>> Besides arm_dma_set_mask() function I see no reason to export the other
>>> arm dma related functions.
>>
>> The idea was to let other people use the arm dma related functions,
>> for their own dma ops. But for the mvebu machines we only need
>> arm_dma_set_mask() indeed.
>>
>> So you prefer that I only expose arm_dma_set_mask() and let future
>> user expose other function if they need it, right?
>
> I would prefer to avoid exporting functions which are not used anywhere
> else. This improves readability of the code and simply forces others to
> think twice before they use some static function and check if their use
> case is really correct.
OK so I will send a new version in a few minutes.
Do you think these last changes will be enough for getting your Acked-by?
Thanks,
Greogry
More information about the linux-arm-kernel
mailing list