From: Ben Hutchings <bhutchings@solarflare.com>
To: Mark Einon <mark.einon@gmail.com>
Cc: netdev@vger.kernel.org, davem@davemloft.net
Subject: Re: [PATCH] mii: Remove references to DP83840 PHY in mii.h
Date: Wed, 31 Aug 2011 18:34:30 +0100 [thread overview]
Message-ID: <1314812070.3274.19.camel@bwh-desktop> (raw)
In-Reply-To: <1314804928-7353-1-git-send-email-mark.einon@gmail.com>
On Wed, 2011-08-31 at 16:35 +0100, Mark Einon wrote:
> There are references to this PHY chip in the generic mii.h header, so removing them.
>
> Signed-off-by: Mark Einon <mark.einon@gmail.com>
> ---
> include/linux/mii.h | 12 ++++++------
> 1 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/include/linux/mii.h b/include/linux/mii.h
> index 103113a..4c3cfb5 100644
> --- a/include/linux/mii.h
> +++ b/include/linux/mii.h
> @@ -40,12 +40,12 @@
> #define BMCR_CTST 0x0080 /* Collision test */
> #define BMCR_FULLDPLX 0x0100 /* Full duplex */
> #define BMCR_ANRESTART 0x0200 /* Auto negotiation restart */
> -#define BMCR_ISOLATE 0x0400 /* Disconnect DP83840 from MII */
> -#define BMCR_PDOWN 0x0800 /* Powerdown the DP83840 */
> +#define BMCR_ISOLATE 0x0400 /* Disconnect PHY from MII */
Since you're trying to improve these comments, I think this could do
with further improvement. When this bit is set, the PHY's data paths
are isolated from the MII (or other interface to the MAC). The control
path is still connected to the management interface (MDIO), which is
important when we want to clear this bit! So it would be better to say
something like 'Isolate data paths fromn MII'.
> +#define BMCR_PDOWN 0x0800 /* Powerdown */
This selects a low-power state (if implemented). It doesn't entirely
turn the PHY off, and at least the management interface must stil be
functional. So it would be better to say 'Request low-power state'.
> #define BMCR_ANENABLE 0x1000 /* Enable auto negotiation */
> #define BMCR_SPEED100 0x2000 /* Select 100Mbps */
> #define BMCR_LOOPBACK 0x4000 /* TXD loopback bits */
> -#define BMCR_RESET 0x8000 /* Reset the DP83840 */
> +#define BMCR_RESET 0x8000 /* Reset */
>
> /* Basic mode status register. */
> #define BMSR_ERCAP 0x0001 /* Ext-reg capability */
> @@ -55,9 +55,9 @@
> #define BMSR_RFAULT 0x0010 /* Remote fault detected */
> #define BMSR_ANEGCOMPLETE 0x0020 /* Auto-negotiation complete */
> #define BMSR_RESV 0x00c0 /* Unused... */
> -#define BMSR_ESTATEN 0x0100 /* Extended Status in R15 */
> -#define BMSR_100HALF2 0x0200 /* Can do 100BASE-T2 HDX */
> -#define BMSR_100FULL2 0x0400 /* Can do 100BASE-T2 FDX */
> +#define BMSR_ESTATEN 0x0100 /* Extended Status in R15 */
> +#define BMSR_100HALF2 0x0200 /* Can do 100BASE-T2 HDX */
> +#define BMSR_100FULL2 0x0400 /* Can do 100BASE-T2 FDX */
This formatting change is unrelated. If you're going to fix formatting
then please convert spaces to tabs after each name and value.
Ben.
> #define BMSR_10HALF 0x0800 /* Can do 10mbps, half-duplex */
> #define BMSR_10FULL 0x1000 /* Can do 10mbps, full-duplex */
> #define BMSR_100HALF 0x2000 /* Can do 100mbps, half-duplex */
--
Ben Hutchings, Staff Engineer, Solarflare
Not speaking for my employer; that's the marketing department's job.
They asked us to note that Solarflare product names are trademarked.
next prev parent reply other threads:[~2011-08-31 17:34 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-08-31 15:35 [PATCH] mii: Remove references to DP83840 PHY in mii.h Mark Einon
2011-08-31 17:34 ` Ben Hutchings [this message]
2011-09-01 9:22 ` Mark Einon
2011-09-01 9:22 ` [PATCH 1/2] [TRIVIAL] mii: Convert spaces to tabs " Mark Einon
2011-09-01 9:22 ` [PATCH 2/2] mii: Remove references to DP83840 PHY " Mark Einon
2011-09-01 18:15 ` Ben Hutchings (bhutchings)
2011-09-15 19:45 ` [PATCH 1/2] [TRIVIAL] mii: Convert spaces to tabs " David Miller
2011-09-15 19:44 ` [PATCH] mii: Remove references to DP83840 PHY " David Miller
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1314812070.3274.19.camel@bwh-desktop \
--to=bhutchings@solarflare.com \
--cc=davem@davemloft.net \
--cc=mark.einon@gmail.com \
--cc=netdev@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.