From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alex Courbot Subject: Re: [PATCH v2] Documentation: gpiolib: document new interface Date: Mon, 18 Nov 2013 19:33:18 +0900 Message-ID: <5289ECEE.4070505@nvidia.com> References: <1384605261-8682-1-git-send-email-acourbot@nvidia.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from hqemgate16.nvidia.com ([216.228.121.65]:5858 "EHLO hqemgate16.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750754Ab3KRKdX (ORCPT ); Mon, 18 Nov 2013 05:33:23 -0500 In-Reply-To: Sender: linux-gpio-owner@vger.kernel.org List-Id: linux-gpio@vger.kernel.org To: Linus Walleij Cc: Rob Landley , "linux-gpio@vger.kernel.org" , "linux-doc@vger.kernel.org" , "linux-kernel@vger.kernel.org" , Alexandre Courbot On 11/18/2013 06:34 PM, Linus Walleij wrote: > On Sat, Nov 16, 2013 at 1:34 PM, Alexandre Courbot wrote: > >> The first version received zero feedback, hopefully this one will get more >> attention. :) Not much changes, just some more proofreading and the fixes >> and improvements that came from it. It looks ok as far as I am concerned. > > Sorry I was swamped with other stuff... > >> Linus, I hope this can be merged during the -rc cycle of 3.13, since the >> gpiod_ interface is going to be introduced there. It would not make much >> sense for it to come without its documentation. > > You're right of course. I'll read through it and apply fixes on top > (or squash into your patch.) Great, thanks! > Formal stuff: > Don't we need an 00-INDEX file? > (Maybe Rob can tell whether this is desirable.) Good idea. gpio.txt somehow fulfills that role, but it might be better if we split it. Would you like me to submit a new revision? > I don't like double spaces after periods. I will just go and remove > all of them from the new documentation unless you convince me > otherwise. Reference: > https://en.wikipedia.org/wiki/Sentence_spacing Please do - actually I started doing it (they come from the old documentation, not the bits I have written myself) but have obviously missed at least a few of them. :) Thanks, Alex.