From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alexandre Belloni Subject: Re: [PATCH 3/5] docs: i2c: convert to ReST and add to driver-api bookset Date: Sat, 29 Jun 2019 00:10:10 +0200 Message-ID: <20190628221010.GV3692@piout.net> References: <3997b54a2e73887b96ec665573f08ded78b71421.1561756511.git.mchehab+samsung@kernel.org> <20190628214138.GU3692@piout.net> <20190628185445.71eb824b@coco.lan> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Linux Doc Mailing List , Mauro Carvalho Chehab , linux-kernel@vger.kernel.org, Jonathan Corbet , Peter Rosin , Rob Herring , Mark Rutland , Jean Delvare , Guenter Roeck , Andreas Werner , Wolfram Sang , Rudolf Marek , Seth Heasley , Neil Horman , Vadim Pasternak , Michael Shych , Ajay Gupta , Peter Korsgaard , Andrew Lunn , Jim Cromie , Mark Bro To: Mauro Carvalho Chehab Return-path: Content-Disposition: inline In-Reply-To: <20190628185445.71eb824b@coco.lan> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-spi.vger.kernel.org On 28/06/2019 18:54:45-0300, Mauro Carvalho Chehab wrote: > Em Fri, 28 Jun 2019 23:41:38 +0200 > Alexandre Belloni escreveu: > > > On 28/06/2019 18:23:14-0300, Mauro Carvalho Chehab wrote: > > > diff --git a/drivers/rtc/rtc-ds1374.c b/drivers/rtc/rtc-ds1374.c > > > index 225a8df1d4e9..1803f3cab39f 100644 > > > --- a/drivers/rtc/rtc-ds1374.c > > > +++ b/drivers/rtc/rtc-ds1374.c > > > @@ -14,7 +14,7 @@ > > > */ > > > /* > > > * It would be more efficient to use i2c msgs/i2c_transfer directly but, as > > > - * recommened in .../Documentation/i2c/writing-clients section > > > + * recommened in .../Documentation/i2c/writing-clients.rst section > > > * "Sending and receiving", using SMBus level communication is preferred. > > > */ > > > > > > > Honestly, the whole comment could be removed. The current trend is to > > move everything to regmap anyway. > > > > However, I'm fine with that change if you want to keep it that way (and > > probably scripted). > > While the conversion was manually made, the renames were scripted, > and checked with: > > ./scripts/documentation-file-ref-check > > Otherwise I would very likely fix the typo: > > recommened -> recommended > > :-) > > I can certainly add new patch at this (before or after patch 3/5 - as you > prefer) in order to get rid of the comment, but I would avoid doing a > somewhat unrelated changes at the same documentation patch. > I'm okay with that. -- Alexandre Belloni, Bootlin Embedded Linux and Kernel engineering https://bootlin.com