From owner-freebsd-net@FreeBSD.ORG  Thu May 31 12:46:21 2012
Return-Path: <owner-freebsd-net@FreeBSD.ORG>
Delivered-To: freebsd-net@freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52])
	by hub.freebsd.org (Postfix) with ESMTP id 4E0311065677;
	Thu, 31 May 2012 12:46:21 +0000 (UTC)
	(envelope-from wblock@wonkity.com)
Received: from wonkity.com (wonkity.com [67.158.26.137])
	by mx1.freebsd.org (Postfix) with ESMTP id 0AE0B8FC0C;
	Thu, 31 May 2012 12:46:20 +0000 (UTC)
Received: from wonkity.com (localhost [127.0.0.1])
	by wonkity.com (8.14.5/8.14.5) with ESMTP id q4VCkKAQ082141;
	Thu, 31 May 2012 06:46:20 -0600 (MDT)
	(envelope-from wblock@wonkity.com)
Received: from localhost (wblock@localhost)
	by wonkity.com (8.14.5/8.14.5/Submit) with ESMTP id q4VCkKcM082138;
	Thu, 31 May 2012 06:46:20 -0600 (MDT)
	(envelope-from wblock@wonkity.com)
Date: Thu, 31 May 2012 06:46:20 -0600 (MDT)
From: Warren Block <wblock@wonkity.com>
To: sbruno@freebsd.org
In-Reply-To: <1338424909.9051.12.camel@powernoodle-l7.corp.yahoo.com>
Message-ID: <alpine.BSF.2.00.1205310628200.81499@wonkity.com>
References: <1338424909.9051.12.camel@powernoodle-l7.corp.yahoo.com>
User-Agent: Alpine 2.00 (BSF 1167 2008-08-23)
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed
X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.2.7
	(wonkity.com [127.0.0.1]); Thu, 31 May 2012 06:46:20 -0600 (MDT)
Cc: "freebsd-net@freebsd.org" <freebsd-net@freebsd.org>
Subject: Re: bce(4) man page updates
X-BeenThere: freebsd-net@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: Networking and TCP/IP with FreeBSD <freebsd-net.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-net>,
	<mailto:freebsd-net-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-net>
List-Post: <mailto:freebsd-net@freebsd.org>
List-Help: <mailto:freebsd-net-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-net>,
	<mailto:freebsd-net-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Thu, 31 May 2012 12:46:21 -0000

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)."