Re: [PATCH v7 0/7] link bonding

["Doherty, Declan" <declan.doherty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>]

Hey, sorry about that, I submitted the patchset without the typo fix. I'll resubmit the patchset now.
What version of checkpatch are you using, I don't get any warning for the missing spaces for the
version of the script with kernel 3.15 on Fedora

Thanks
Declan


> -----Original Message-----
> From: Thomas Monjalon [mailto:thomas.monjalon-pdR9zngts4EAvxtiuMwx3w@public.gmane.org]
> Sent: Monday, November 24, 2014 3:36 PM
> To: Doherty, Declan
> Cc: dev-VfR2kkLFssw@public.gmane.org
> Subject: Re: [PATCH v7 0/7] link bonding
> 
> Hi Declan,
> 
> There are still 2 kind of errors reported by checkpatch:
> - 14 occurrences of this typo:
> WARNING:TYPO_SPELLING: 'interupt' may be misspelled - perhaps 'interrupt'?
> - 6 occurences of missing space:
> WARNING:MISSING_SPACE: break quoted strings at a space character
> 
> Example:
> #2016: FILE: lib/librte_pmd_bond/rte_eth_bond_pmd.c:1420:
> +                               "LSC polling interval can be specified only once for bonded"
> +                               "device %s\n", name);
> You should put a space between "bonded" and "device".
> 
> --
> Thomas