On Sat, Jan 08, 2022 at 09:33:40PM +1100, Jonathan Gray wrote:
> On Sat, Jan 08, 2022 at 09:59:10AM +0000, Jason McIntyre wrote:
> > On Sat, Jan 08, 2022 at 02:38:05PM +1100, Jonathan Gray wrote:
> > > dwc2, drm and microcode purposefully excluded
> > > 
> > 
> > morning.
> > 
> > no need for breakfast after reading this one!
> > 
> > the spelling checks out. i have a few tweaks though:
> > 
> > acpireg.h:
> > rsdp_signaturee is a change to the actual #define. not sure if that
> > changes anything.
> 
> there are no other references to that define
> 

that's fine, just wanted to check

> > 
> > if_wi_ieee.h:
> > "weirdity" should probably just be "oddity", but whatever
> > 
> > wdcvar.h:
> > channels-specific data -> channel-specific data
> > 
> > pcppi.c:
> > normally it's be to quick -> normally it's too quick
> 
> tb@ suggested this as well
> 
> > 
> > if_iwmreg.h:
> > allows Tx with narrower BW then requested -> ... than requested
> > 
> > pciide.c:
> > timings setups -> timing setups
> > 
> > cgtwelve.c/tvtwo.c/zx.c:
> > advertize -> advertise: although i dislike the "z" spelling myself,
> > it is not incorrect. generally -ise -> -ize seems to be preferred in US 
> > english.
> 
> I think this is an exception?
> 

you're right. i checked my dictionaries. i get so used to the -ise->-ize
change that i expect it everywhere.

> > 
> > zx.c:
> > a 8-bit plane -> an 8-bit plane
> > 
> > if_atu.c:
> > "requeres" line has a double full stop,
> > as does if_atureg.h "psuedo" line
> 
> The .. in this file seems to be for indicating a range of values
> 

ok then

> thanks, adding the below, pcppi part already included
> 

so all ok as far as i can see.

jmc

Reply via email to