Thanks for the feedback, corrected in 94466c432ed611a49cacae7ff99b224b37c906bb.

--Gordon

On Sun, Sep 04, 2022 at 10:36:47PM +0100, Jessica Clarke wrote:
> On 4 Sept 2022, at 16:40, Gordon Bergling <[email protected]> wrote:
> > 
> > The branch main has been updated by gbe (doc committer):
> > 
> > URL: 
> > https://cgit.FreeBSD.org/src/commit/?id=8ca67bf1273a5168f8a3787183159c477632e582
> > 
> > commit 8ca67bf1273a5168f8a3787183159c477632e582
> > Author:     Gordon Bergling <[email protected]>
> > AuthorDate: 2022-09-04 15:40:25 +0000
> > Commit:     Gordon Bergling <[email protected]>
> > CommitDate: 2022-09-04 15:40:25 +0000
> > 
> >    usb: Remove a double words in a source code comments
> > 
> >    - s/that that/that/
> > 
> >    MFC after:      3 days
> > ---
> > sys/dev/usb/net/if_muge.c | 2 +-
> > sys/dev/usb/net/if_smsc.c | 2 +-
> > sys/dev/usb/usb_busdma.c  | 2 +-
> > 3 files changed, 3 insertions(+), 3 deletions(-)
> > 
> > diff --git a/sys/dev/usb/net/if_muge.c b/sys/dev/usb/net/if_muge.c
> > index 3087eb40b40a..c705fcd56eed 100644
> > --- a/sys/dev/usb/net/if_muge.c
> > +++ b/sys/dev/usb/net/if_muge.c
> > @@ -1556,7 +1556,7 @@ muge_set_leds(struct usb_ether *ue)
> >  *  @ue: the USB ethernet device
> >  *
> >  *  This is where the chip is intialised for the first time.  This is
> > - * different from the muge_init() function in that that one is designed to
> > + * different from the muge_init() function in that one is designed to
> 
> The original comment was correct; changing it loses a key part.
> 
> >  *  setup the H/W to match the UE settings and can be called after a reset.
> >  *
> >  */
> > diff --git a/sys/dev/usb/net/if_smsc.c b/sys/dev/usb/net/if_smsc.c
> > index e3b4ae9a5cbd..c34cc85269cc 100644
> > --- a/sys/dev/usb/net/if_smsc.c
> > +++ b/sys/dev/usb/net/if_smsc.c
> > @@ -1545,7 +1545,7 @@ smsc_ioctl(struct ifnet *ifp, u_long cmd, caddr_t 
> > data)
> >  *  @ue: the USB ethernet device
> >  *
> >  *  This is where the chip is intialised for the first time.  This is 
> > different
> > - * from the smsc_init() function in that that one is designed to setup the
> > + * from the smsc_init() function in that one is designed to setup the
> 
> Ditto.
> 
> Jess
> 
> >  *  H/W to match the UE settings and can be called after a reset.
> >  *
> >  *
> > diff --git a/sys/dev/usb/usb_busdma.c b/sys/dev/usb/usb_busdma.c
> > index 62e22805b39c..69d9ea292523 100644
> > --- a/sys/dev/usb/usb_busdma.c
> > +++ b/sys/dev/usb/usb_busdma.c
> > @@ -546,7 +546,7 @@ usb_pc_alloc_mem(struct usb_page_cache *pc, struct 
> > usb_page *pg,
> >             /*
> >              * XXX BUS-DMA workaround - FIXME later:
> >              *
> > -            * We assume that that the alignment at this point of
> > +            * We assume that the alignment at this point of
> >              * the code is greater than or equal to the size and
> >              * less than two times the size, so that if we double
> >              * the size, the size will be greater than the
> 

-- 

Reply via email to