Re: [PATCH] isst: isst_if.h: drop a duplicated word

From: Andy Shevchenko
Date: Mon Jul 27 2020 - 06:27:51 EST


On Tue, Jul 21, 2020 at 6:10 PM Srinivas Pandruvada
<srinivas.pandruvada@xxxxxxxxxxxxxxx> wrote:
>
> On Sat, 2020-07-18 at 17:29 -0700, Randy Dunlap wrote:
> > Drop the repeated word "for" in a comment.
> >
> > Signed-off-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
> > Cc: Srinivas Pandruvada <srinivas.pandruvada@xxxxxxxxxxxxxxx>
> > Cc: platform-driver-x86@xxxxxxxxxxxxxxx
> > Cc: Darren Hart <dvhart@xxxxxxxxxxxxx>
> > Cc: Andy Shevchenko <andy@xxxxxxxxxxxxx>
> Acked-by: Srinivas Pandruvada <srinivas.pandruvada@xxxxxxxxxxxxxxx>
>

Pushed to my review and testing queue, thanks!

> > ---
> > This description could still use some better wording.
>
> I will fix the wording in a follow up patch.
>
> > include/uapi/linux/isst_if.h | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > --- linux-next-20200717.orig/include/uapi/linux/isst_if.h
> > +++ linux-next-20200717/include/uapi/linux/isst_if.h
> > @@ -69,7 +69,7 @@ struct isst_if_cpu_maps {
> > * @logical_cpu: Logical CPU number to get target PCI device.
> > * @reg: PUNIT register offset
> > * @value: For write operation value to write and for
> > - * for read placeholder read value
> > + * read placeholder read value
> > *
> > * Structure to specify read/write data to PUNIT registers.
> > */
>


--
With Best Regards,
Andy Shevchenko