Date: Thu, 31 May 2012 10:53:28 -0700 From: Sean Bruno <seanbru@yahoo-inc.com> To: Warren Block <wblock@wonkity.com> Cc: "freebsd-net@freebsd.org" <freebsd-net@freebsd.org> Subject: Re: bce(4) man page updates Message-ID: <1338486809.2985.10.camel@powernoodle-l7.corp.yahoo.com> In-Reply-To: <1338484537.2985.8.camel@powernoodle-l7.corp.yahoo.com> References: <1338424909.9051.12.camel@powernoodle-l7.corp.yahoo.com> <alpine.BSF.2.00.1205310628200.81499@wonkity.com> <1338484537.2985.8.camel@powernoodle-l7.corp.yahoo.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, 2012-05-31 at 10:15 -0700, Sean Bruno wrote: > On Thu, 2012-05-31 at 05:46 -0700, Warren Block wrote: > > On Wed, 30 May 2012, Sean Bruno wrote: > > > > > Was trolling around inside of bce(4) and the Broadcom docs today and > > > made the following update to the man page. Thoughts? > > > > > > Sean > > > > > > http://people.freebsd.org/~sbruno/bce_man.txt > > > > Nice! > > > > Just minor suggestions. > > > > The two "whether or not" sentences ought to be just "Enable/Disable ..." > > like the rest. > > > > A couple of "Cannot be assigned the value 0" sentences can be simpler as > > "Cannot be set to 0" if that doesn't change the meaning. > > > > "Valid values are in the range from 0-256." can be simpler as > > "Values from 0-256 are valid." > > > > Many "The default value is" which could be simplified to "The default > > is", or added to the end of a previous sentence as an aside: > > "Values from 0-100 are valid (default 18)." > > > Indeed. Much more gooder wordingness. Updated! > > http://people.freebsd.org/~sbruno/bce_man.txt > > Sean > Updated again, looks like one of the tuneables has changed in -head: hw.bce.loose_rx_mtu --> hw.bce.strict_rx_mtu Sean
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1338486809.2985.10.camel>