Re: [PATCH v2] Spelling comments/text

2022-01-11 Thread Thomas Monjalon
11/01/2022 15:00, Kevin Traynor: > On 11/01/2022 10:51, Thomas Monjalon wrote: > > 06/01/2022 17:46, Thomas Monjalon: > >> 29/11/2021 17:08, Josh Soref: > > Applied, thanks. > > > > The git log became: > > > > fix spelling in comments and strings > > > > The tool comes from https:

Re: [PATCH v2] Spelling comments/text

2022-01-11 Thread Kevin Traynor
On 11/01/2022 10:51, Thomas Monjalon wrote: 06/01/2022 17:46, Thomas Monjalon: 29/11/2021 17:08, Josh Soref: -* external phy devices registred through kerenl apis +* external phy devices registred through kernel apis another spelling to fix: "registred" and while at it

Re: [PATCH v2] Spelling comments/text

2022-01-11 Thread Thomas Monjalon
06/01/2022 17:46, Thomas Monjalon: > 29/11/2021 17:08, Josh Soref: > > -* external phy devices registred through kerenl apis > > +* external phy devices registred through kernel apis > > another spelling to fix: > "registred" > and while at it, "apis" should be "APIs" > > -

Re: [PATCH v2] Spelling comments/text

2022-01-07 Thread Bruce Richardson
On Fri, Jan 07, 2022 at 05:27:55PM +0100, Morten Brørup wrote: > > From: Bruce Richardson [mailto:bruce.richard...@intel.com] > > Sent: Friday, 7 January 2022 16.29 > > > > On Fri, Jan 07, 2022 at 03:58:27PM +0100, Thomas Monjalon wrote: > > > 07/01/2022 12:23, Bruce Richardson: > > > > On Thu, Ja

RE: [PATCH v2] Spelling comments/text

2022-01-07 Thread Morten Brørup
> From: Bruce Richardson [mailto:bruce.richard...@intel.com] > Sent: Friday, 7 January 2022 16.29 > > On Fri, Jan 07, 2022 at 03:58:27PM +0100, Thomas Monjalon wrote: > > 07/01/2022 12:23, Bruce Richardson: > > > On Thu, Jan 06, 2022 at 05:52:49PM +0100, Thomas Monjalon wrote: > > > > 29/11/2021 1

Re: [PATCH v2] Spelling comments/text

2022-01-07 Thread Bruce Richardson
On Fri, Jan 07, 2022 at 03:58:27PM +0100, Thomas Monjalon wrote: > 07/01/2022 12:23, Bruce Richardson: > > On Thu, Jan 06, 2022 at 05:52:49PM +0100, Thomas Monjalon wrote: > > > 29/11/2021 17:08, Josh Soref: > > > > -* dequeueing once we've filled up the queue, we have to > > > > benchmark

Re: [PATCH v2] Spelling comments/text

2022-01-07 Thread Josh Soref
I only make changes like this when both flavors are used in a corpus: https://github.com/DPDK/dpdk/search?q=queuing&type=code https://github.com/DPDK/dpdk/search?q=dequeuing&type=code I don't have a preference beyond consistency On Fri, Jan 7, 2022, 9:58 AM Thomas Monjalon wrote: > 07/01/2022

Re: [PATCH v2] Spelling comments/text

2022-01-07 Thread Thomas Monjalon
07/01/2022 12:23, Bruce Richardson: > On Thu, Jan 06, 2022 at 05:52:49PM +0100, Thomas Monjalon wrote: > > 29/11/2021 17:08, Josh Soref: > > > -* dequeueing once we've filled up the queue, we have to > > > benchmark it > > > +* dequeuing once we've filled up the queue, we have to b

Re: [PATCH v2] Spelling comments/text

2022-01-07 Thread Bruce Richardson
On Thu, Jan 06, 2022 at 05:52:49PM +0100, Thomas Monjalon wrote: > 29/11/2021 17:08, Josh Soref: > > -* dequeueing once we've filled up the queue, we have to benchmark > > it > > +* dequeuing once we've filled up the queue, we have to benchmark it > > I think "dequeueing" is corre

Re: [PATCH v2] Spelling comments/text

2022-01-06 Thread Thomas Monjalon
29/11/2021 17:08, Josh Soref: > -* dequeueing once we've filled up the queue, we have to benchmark it > +* dequeuing once we've filled up the queue, we have to benchmark it I think "dequeueing" is correct.

Re: [PATCH v2] Spelling comments/text

2022-01-06 Thread Thomas Monjalon
29/11/2021 17:08, Josh Soref: > -* external phy devices registred through kerenl apis > +* external phy devices registred through kernel apis another spelling to fix: "registred" and while at it, "apis" should be "APIs" - * When set to zero, simple forwaring path is eanbl