public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Arnd Bergmann <arnd@arndb.de>
To: David Hollis <dhollis@davehollis.com>
Cc: dbrownell@users.sourceforge.net, linux-kernel@vger.kernel.org,
	linux-usb-devel@lists.sourceforge.net, support@moschip.com,
	Michael Helmling <supermihi@web.de>
Subject: [PATCH] mcs7830: clean up use of kernel constants
Date: Sun, 27 Aug 2006 22:41:04 +0200	[thread overview]
Message-ID: <200608272241.05026.arnd@arndb.de> (raw)
In-Reply-To: <200608202207.39709.arnd@arndb.de>

This use the MII register constants provided
by the kernel instead of hardcoding numerical
values in the driver.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---

Index: linux-cg/drivers/usb/net/mcs7830.c
===================================================================
--- linux-cg.orig/drivers/usb/net/mcs7830.c	2006-08-25 21:17:24.000000000 +0200
+++ linux-cg/drivers/usb/net/mcs7830.c	2006-08-25 21:23:35.000000000 +0200
@@ -35,8 +35,10 @@
 #include "usbnet.h"
 
 /* requests */
-#define MCS7830_RD_BMREQ	(USB_DIR_IN  | USB_TYPE_VENDOR | USB_RECIP_DEVICE)
-#define MCS7830_WR_BMREQ	(USB_DIR_OUT | USB_TYPE_VENDOR | USB_RECIP_DEVICE)
+#define MCS7830_RD_BMREQ	(USB_DIR_IN  | USB_TYPE_VENDOR | \
+				 USB_RECIP_DEVICE)
+#define MCS7830_WR_BMREQ	(USB_DIR_OUT | USB_TYPE_VENDOR | \
+				 USB_RECIP_DEVICE)
 #define MCS7830_RD_BREQ		0x0E
 #define MCS7830_WR_BREQ		0x0D
 
@@ -46,6 +48,10 @@
 #define MCS7830_VENDOR_ID	0x9710
 #define MCS7830_PRODUCT_ID	0x7830
 
+#define MCS7830_MII_ADVERTISE	(ADVERTISE_PAUSE_CAP | ADVERTISE_100FULL | \
+				 ADVERTISE_100HALF | ADVERTISE_10FULL | \
+				 ADVERTISE_10HALF | ADVERTISE_CSMA)
+
 /* HIF_REG_XX coressponding index value */
 enum {
 	HIF_REG_MULTICAST_HASH			= 0x00,
@@ -258,16 +264,18 @@
 {
 	int ret;
 	/* Enable all media types */
-	ret = mcs7830_write_phy(dev, MII_ADVERTISE, 0x05e1);
-	/* First Disable All */
+	ret = mcs7830_write_phy(dev, MII_ADVERTISE, MCS7830_MII_ADVERTISE);
+
+	/* First reset BMCR */
 	if (!ret)
 		ret = mcs7830_write_phy(dev, MII_BMCR, 0x0000);
 	/* Enable Auto Neg */
 	if (!ret)
-		ret = mcs7830_write_phy(dev, MII_BMCR, 0x1000);
+		ret = mcs7830_write_phy(dev, MII_BMCR, BMCR_ANENABLE);
 	/* Restart Auto Neg (Keep the Enable Auto Neg Bit Set) */
 	if (!ret)
-		ret = mcs7830_write_phy(dev, MII_BMCR, 0x1200);
+		ret = mcs7830_write_phy(dev, MII_BMCR,
+				BMCR_ANENABLE | BMCR_ANRESTART	);
 	return ret < 0 ? : 0;
 }
 


  parent reply	other threads:[~2006-08-27 23:58 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-08-07 13:00 [PATCH] please review mcs7830 (DeLOCK USB etherner) driver Arnd Bergmann
2006-08-07 14:54 ` David Hollis
2006-08-07 16:11   ` Arnd Bergmann
2006-08-20 20:07     ` [PATCH] driver for mcs7830 (aka DeLOCK) USB ethernet adapter Arnd Bergmann
2006-08-20 20:13       ` [PATCH] usbnet: add a mutex around phy register access Arnd Bergmann
2006-08-21 14:34       ` [PATCH] driver for mcs7830 (aka DeLOCK) USB ethernet adapter David Hollis
2006-08-27 20:41       ` Arnd Bergmann [this message]
2006-08-27 20:41         ` [PATCH] mcs7830: fix reception of 1514 byte frames Arnd Bergmann
2006-09-02 10:33           ` [linux-usb-devel] " David Brownell
2006-09-02 18:29             ` Arnd Bergmann
2006-08-28 19:09         ` [PATCH] mcs7830: clean up use of kernel constants David Hollis
2006-09-02 10:38       ` [linux-usb-devel] [PATCH] driver for mcs7830 (aka DeLOCK) USB ethernet adapter David Brownell
2006-09-02 17:51         ` Arnd Bergmann
2006-09-04  6:40           ` David Brownell
2006-09-07 19:37             ` David Hollis

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=200608272241.05026.arnd@arndb.de \
    --to=arnd@arndb.de \
    --cc=dbrownell@users.sourceforge.net \
    --cc=dhollis@davehollis.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb-devel@lists.sourceforge.net \
    --cc=supermihi@web.de \
    --cc=support@moschip.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox