Re: [PATCH] Documentation/DMA-API-HOWTO.txt: fix duplicated word

[Randy Dunlap <rdunlap@infradead.org>]

On 05/12/2014 06:14 AM, Bjorn Helgaas wrote:
> On Sun, May 11, 2014 at 2:15 PM, Emilio López <emilio@elopez.com.ar> wrote:
>> "coherent" is written twice when it should be just once
>>
>> Signed-off-by: Emilio López <emilio@elopez.com.ar>
>> ---
>>
>> The sentence still doesn't sound entirely correct in my head, maybe
>> someone can propose a better rewording.
> 
> I added this patch to my series.  I agree the wording is awkward, but
> I don't have a better proposal right now.

Thanks.

I have reviewed your patch series documentation changes.  The only other
change that I would make (can be done at some other time) is to refer to
CPU instead of cpu, so your patches 1 and 2 are also
Acked-by: Randy Dunlap <rdunlap@infradead.org>


You can send the merge request for all of these to Linus
if you want to...

Would you also please pick up these 2 new patches from today?
(or comment on them/correct them)

From: "gioh.kim" <gioh.kim@lge.com>
[PATCH] Documentation/DMA-API-HOWTO.txt: update API descriptions
[PATCH] Documentation/dma-buf-sharing.txt: update API descriptions





> 
>>  Documentation/DMA-API-HOWTO.txt | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/Documentation/DMA-API-HOWTO.txt b/Documentation/DMA-API-HOWTO.txt
>> index 5e98303..4ede25a 100644
>> --- a/Documentation/DMA-API-HOWTO.txt
>> +++ b/Documentation/DMA-API-HOWTO.txt
>> @@ -192,7 +192,7 @@ the case would look like this:
>>                 goto ignore_this_device;
>>         }
>>
>> -The coherent coherent mask will always be able to set the same or a
>> +The coherent mask will always be able to set the same or a
>>  smaller mask as the streaming mask. However for the rare case that a
>>  device driver only uses consistent allocations, one would have to
>>  check the return value from dma_set_coherent_mask().
>> --


-- 
~Randy