linux-kernel - Re: [PATCH][V2] iio: ad5755: fix spelling mistake "to" -> "too" and grammar

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Sun, 2 Feb 2020 15:53:04 +0000
From:   Jonathan Cameron <jic23@...nel.org>
To:     "Ardelean, Alexandru" <alexandru.Ardelean@...log.com>
Cc:     "zzzzPopa, zzzzStefan Serban" <StefanSerban.Popa@...log.com>,
        "linux-iio@...r.kernel.org" <linux-iio@...r.kernel.org>,
        "colin.king@...onical.com" <colin.king@...onical.com>,
        "Hennerich, Michael" <Michael.Hennerich@...log.com>,
        "lars@...afoo.de" <lars@...afoo.de>,
        "knaack.h@....de" <knaack.h@....de>,
        "pmeerw@...erw.net" <pmeerw@...erw.net>,
        "kernel-janitors@...r.kernel.org" <kernel-janitors@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        Joe Perches <joe@...ches.com>
Subject: Re: [PATCH][V2] iio: ad5755: fix spelling mistake "to" -> "too" and
 grammar

On Thu, 23 Jan 2020 12:16:03 +0000
"Ardelean, Alexandru" <alexandru.Ardelean@...log.com> wrote:

> On Thu, 2020-01-23 at 09:19 +0000, Colin King wrote:
> > From: Colin Ian King <colin.king@...onical.com>
> > 
> > There is a spelling mistake and grammar mistake in a dev_err
> > message. Fix it.
> >   
> 
> Reviewed-by: Alexandru Ardelean <alexandru.ardelean@...log.com>
> 
> > Signed-off-by: Colin Ian King <colin.king@...onical.com>

This crossed with Joe observing there are quite a few other nice little bits
of janitorial work to be done in this driver. 

I merged the two together and fiddled the patch description to cover
the result.  Hope no one minds.  Seemed like it would save everyone
some time rather than asking for a v3.

Shout out if I've messed it up or you would rather I change the
various tags.

Applied to the togreg branch of iio.git and pushed out as testing for
the autobuilders to play with it.

Thanks all,

Jonathan

> > ---
> > V2: fix grammar too, thanks to Alexandru Ardelean for spotting this.
> > ---
> >  drivers/iio/dac/ad5755.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/drivers/iio/dac/ad5755.c b/drivers/iio/dac/ad5755.c
> > index b9175fb4c8ab..1359a1a92fdc 100644
> > --- a/drivers/iio/dac/ad5755.c
> > +++ b/drivers/iio/dac/ad5755.c
> > @@ -655,7 +655,7 @@ static struct ad5755_platform_data *ad5755_parse_dt(struct
> > device *dev)
> >  	for_each_child_of_node(np, pp) {
> >  		if (devnr >= AD5755_NUM_CHANNELS) {
> >  			dev_err(dev,
> > -				"There is to many channels defined in DT\n");
> > +				"There are too many channels defined in DT\n");
> >  			goto error_out;
> >  		}
> >