Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752938AbcC1NlW (ORCPT ); Mon, 28 Mar 2016 09:41:22 -0400 Received: from mail-pa0-f41.google.com ([209.85.220.41]:33911 "EHLO mail-pa0-f41.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751160AbcC1NlU (ORCPT ); Mon, 28 Mar 2016 09:41:20 -0400 Date: Mon, 28 Mar 2016 06:41:16 -0700 From: Eduardo Valentin To: "Champ, Andy" Cc: Javi Merino , "corbet@lwn.net" , "durgadoss.r@intel.com" , "leo.yan@linaro.org" , "kapileshwar.singh@arm.com" , "wni@nvidia.com" , "linux-doc@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH] Syntactic and factual errors in the API document Message-ID: <20160328134115.GB7624@localhost.localdomain> References: <1458650245-24303-1-git-send-email-andycham@amazon.com> <20160322143529.GB3087@e104805> <20160322160030.GA4079@localhost.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20160322160030.GA4079@localhost.localdomain> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1044 Lines: 27 On Tue, Mar 22, 2016 at 09:00:32AM -0700, Eduardo Valentin wrote: > Hello Andy, > > On Tue, Mar 22, 2016 at 12:37:25PM +0000, Andy Champ wrote: > > There are several places where the English in the document is syntactically > > invalid, or unclear. There are also one or two factual errors. BTW, you are supposed to sign your patches too. I am adding your signed-off-by here, but next time, remember to sign your patches (have a look at the Documentation/SubmittingPatches file. > > > > --- > > Documentation/thermal/sysfs-api.txt | 44 ++++++++++++++++++------------------- > > 1 file changed, 22 insertions(+), 22 deletions(-) > > Thanks for your patch improving the documentation. I have no comments in > the text correction. The only minor is that the patch title should have > 'thermal:' prefix (have a look in the git log > Documentation/thermal/sysfs-api.txt). > > Also, I will get this patch only after the merge window is finished and > my current pending patches queue has been sent and merged. > > BR, > > Eduardo Valentin