lkml.org 
[lkml]   [2024]   [Mar]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH v2 01/15] dmaengine: bcm2835: Fix several spellos
On 09:38 Wed 13 Mar     , Florian Fainelli wrote:
> On 3/13/24 08:26, Andrea della Porta wrote:
> > On 08:00 Wed 13 Mar , Florian Fainelli wrote:
> > >
> > >
> > > On 3/13/2024 7:08 AM, Andrea della Porta wrote:
> > > > Fixed Codespell reported warnings about spelling and coding convention
> > > > violations, among which there are also a couple potential operator
> > > > precedence issue in macroes.
>
> and s/macroes/macros/

right, sorry about that...

>
> > > >
> > > > Signed-off-by: Andrea della Porta <andrea.porta@suse.com>
> > >
> > > There are no spelling errors being fixed in this commit, this is purely
> > > stylistic and conforming to the Linux coding style guidelines.
> >
> > - /* detect a size missmatch */
> > - if (buf_len && (d->size != buf_len))
> > + /* detect a size mismatch */
> > + if (buf_len && d->size != buf_len)
> >
> > Isn't 'missmatch' a spelling error? Maybe I can drop the word 'several', since it's
> > indeed only one...
>
> Can we agree this was easy to miss when 99% of the changes are stylistic?
> The summary is that the commit subject and the message are not describing
> what this patch is about.
> --
> Florian
>

I agree with you that I could've added '...and coding style fixes' to the subject
(while also dropping 'several'). The comment seems fine enough though. Maybe
you're suggesting to separate the patches into two, one dealing with spelling
errors and the other with coding conventions?

Many thanks,
Andrea

\
 
 \ /
  Last update: 2024-03-13 18:58    [W:0.056 / U:0.108 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site