Re: [PATCH v2] nvme/host: fixup some typos

From: Wilfred Mallawa

Date: Wed Jan 07 2026 - 05:33:47 EST


On Wed, 2026-01-07 at 11:09 +0100, Christoph Hellwig wrote:
> Bit: the subject prefix is we usually use is just "nvme: ".
>
> >   /*
> >   * Handle ioctls that apply to the controller instead of
> > the namespace
> > - * separately and drop the ns SRCU reference early.  This
> > avoids a
> > + * separately and drop the ns SRCU reference early. This
> > avoids a
>
> I don't see a typo here, just a reformatting to a different comment
> style not very commonly used in the nvme code.

It's just the extra white space before the "This avoids". Is that not a
typo?

>
> > -/* Define the socket priority to use for connections were it is
> > desirable
> > +/* Define the socket priority to use for connections where it is
> > desirable
>
> The change looks good, but if you touch this, you might as well fix
> up
> the comment style to match the normal kernel style:
>
> /*
>  * Define the socket priority to use for connections where it is
> desirable

Ah no worries, will fixup!

Wilfred