Re: [PATCH] wifi: cfg80211: wext: Correct spelling in iw_handler.h

[Simon Horman <horms@kernel.org>]

On Thu, Aug 29, 2024 at 06:07:00PM +0200, Johannes Berg wrote:
> On Thu, 2024-08-29 at 17:03 +0100, Simon Horman wrote:
> > Correct spelling in iw_handler.h.
> > As reported by codespell.
> > 
> > Signed-off-by: Simon Horman <horms@kernel.org>
> > ---
> >  include/net/iw_handler.h | 12 ++++++------
> >  1 file changed, 6 insertions(+), 6 deletions(-)
> > 
> > diff --git a/include/net/iw_handler.h b/include/net/iw_handler.h
> > index b2cf243ebe44..f7f4c2a79b9e 100644
> > --- a/include/net/iw_handler.h
> > +++ b/include/net/iw_handler.h
> > @@ -23,7 +23,7 @@
> >   * to handle wireless statistics.
> >   *
> >   * The initial APIs served us well and has proven a reasonably good design.
> > - * However, there is a few shortcommings :
> > + * However, there is a few shortcomings :
> 
> If we're going to touch it, maybe that should also say "there are a few
> shortcomings"? :)

Yes, indeed :)

> But is it worth it at all? This stuff is totally on the way out, so all
> the thing about "proven a reasonably good design", well, didn't really
> pan out in practice...

I think if the text is there it may as well be correct.
But if you prefer to leave it as is, then we can drop this topic.