From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759274AbaELRM7 (ORCPT ); Mon, 12 May 2014 13:12:59 -0400 Received: from merlin.infradead.org ([205.233.59.134]:56465 "EHLO merlin.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759044AbaELRM5 (ORCPT ); Mon, 12 May 2014 13:12:57 -0400 Message-ID: <53710114.8080405@infradead.org> Date: Mon, 12 May 2014 10:12:52 -0700 From: Randy Dunlap User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.3.0 MIME-Version: 1.0 To: Bjorn Helgaas , =?ISO-8859-1?Q?Emilio_L=F3pez?= CC: "linux-doc@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH] Documentation/DMA-API-HOWTO.txt: fix duplicated word References: <1399839324-21773-1-git-send-email-emilio@elopez.com.ar> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 05/12/2014 06:14 AM, Bjorn Helgaas wrote: > On Sun, May 11, 2014 at 2:15 PM, Emilio López wrote: >> "coherent" is written twice when it should be just once >> >> Signed-off-by: Emilio López >> --- >> >> 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 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" [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