All of lore.kernel.org
 help / color / mirror / Atom feed
From: Joe Hershberger <joe.hershberger@ni.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 5/5] net: mii: Clean up legacy glue that is not used
Date: Mon, 8 Aug 2016 11:28:41 -0500	[thread overview]
Message-ID: <1470673721-20590-6-git-send-email-joe.hershberger@ni.com> (raw)
In-Reply-To: <1470673721-20590-1-git-send-email-joe.hershberger@ni.com>

The cleanup of the legacy mii registration API that's no longer used now
that the drivers have been converted to use the (more) modern API.

Signed-off-by: Joe Hershberger <joe.hershberger@ni.com>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>

---

Changes in v2: None

 common/miiphyutil.c | 73 -----------------------------------------------------
 include/miiphy.h    | 13 ----------
 2 files changed, 86 deletions(-)

diff --git a/common/miiphyutil.c b/common/miiphyutil.c
index 7e41957..08aa854 100644
--- a/common/miiphyutil.c
+++ b/common/miiphyutil.c
@@ -65,79 +65,6 @@ void miiphy_init(void)
 	current_mii = NULL;
 }
 
-static int legacy_miiphy_read(struct mii_dev *bus, int addr, int devad, int reg)
-{
-	unsigned short val;
-	int ret;
-	struct legacy_mii_dev *ldev = bus->priv;
-
-	ret = ldev->read(bus->name, addr, reg, &val);
-
-	return ret ? -1 : (int)val;
-}
-
-static int legacy_miiphy_write(struct mii_dev *bus, int addr, int devad,
-				int reg, u16 val)
-{
-	struct legacy_mii_dev *ldev = bus->priv;
-
-	return ldev->write(bus->name, addr, reg, val);
-}
-
-/*****************************************************************************
- *
- * Register read and write MII access routines for the device <name>.
- * This API is now deprecated. Please use mdio_alloc and mdio_register, instead.
- */
-void miiphy_register(const char *name,
-		      int (*read)(const char *devname, unsigned char addr,
-				   unsigned char reg, unsigned short *value),
-		      int (*write)(const char *devname, unsigned char addr,
-				    unsigned char reg, unsigned short value))
-{
-	struct mii_dev *new_dev;
-	struct legacy_mii_dev *ldev;
-
-	BUG_ON(strlen(name) >= MDIO_NAME_LEN);
-
-	/* check if we have unique name */
-	new_dev = miiphy_get_dev_by_name(name);
-	if (new_dev) {
-		printf("miiphy_register: non unique device name '%s'\n", name);
-		return;
-	}
-
-	/* allocate memory */
-	new_dev = mdio_alloc();
-	ldev = malloc(sizeof(*ldev));
-
-	if (new_dev == NULL || ldev == NULL) {
-		printf("miiphy_register: cannot allocate memory for '%s'\n",
-			name);
-		free(ldev);
-		mdio_free(new_dev);
-		return;
-	}
-
-	/* initalize mii_dev struct fields */
-	new_dev->read = legacy_miiphy_read;
-	new_dev->write = legacy_miiphy_write;
-	strncpy(new_dev->name, name, MDIO_NAME_LEN);
-	new_dev->name[MDIO_NAME_LEN - 1] = 0;
-	ldev->read = read;
-	ldev->write = write;
-	new_dev->priv = ldev;
-
-	debug("miiphy_register: added '%s', read=0x%08lx, write=0x%08lx\n",
-	       new_dev->name, ldev->read, ldev->write);
-
-	/* add it to the list */
-	list_add_tail(&new_dev->link, &mii_devs);
-
-	if (!current_mii)
-		current_mii = new_dev;
-}
-
 struct mii_dev *mdio_alloc(void)
 {
 	struct mii_dev *bus;
diff --git a/include/miiphy.h b/include/miiphy.h
index 8a55a13..83141b4 100644
--- a/include/miiphy.h
+++ b/include/miiphy.h
@@ -21,13 +21,6 @@
 #include <net.h>
 #include <phy.h>
 
-struct legacy_mii_dev {
-	int (*read)(const char *devname, unsigned char addr,
-		     unsigned char reg, unsigned short *value);
-	int (*write)(const char *devname, unsigned char addr,
-		      unsigned char reg, unsigned short value);
-};
-
 int miiphy_read(const char *devname, unsigned char addr, unsigned char reg,
 		 unsigned short *value);
 int miiphy_write(const char *devname, unsigned char addr, unsigned char reg,
@@ -44,12 +37,6 @@ int miiphy_link(const char *devname, unsigned char addr);
 
 void miiphy_init(void);
 
-void miiphy_register(const char *devname,
-		      int (*read)(const char *devname, unsigned char addr,
-				   unsigned char reg, unsigned short *value),
-		      int (*write)(const char *devname, unsigned char addr,
-				    unsigned char reg, unsigned short value));
-
 int miiphy_set_current_dev(const char *devname);
 const char *miiphy_get_current_dev(void);
 struct mii_dev *mdio_get_current_dev(void);
-- 
1.7.11.5

  parent reply	other threads:[~2016-08-08 16:28 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-04  9:07 [U-Boot] [PATCH 0/6] Remove calls from net drivers to legacy miiphy_register() entry-point Joe Hershberger
2016-04-04  9:07 ` [U-Boot] [PATCH 1/6] common: Always include errno.h in common.h Joe Hershberger
2016-04-11  4:23   ` Bin Meng
2016-07-12 17:28   ` [U-Boot] " Joe Hershberger
2016-04-04  9:07 ` [U-Boot] [PATCH 2/6] scripts: Add a cocci patch for miiphy_register Joe Hershberger
2016-04-04  9:07 ` [U-Boot] [PATCH 3/6] net: mii: Use spatch to update miiphy_register Joe Hershberger
2016-04-11  4:23   ` Bin Meng
2016-04-04  9:07 ` [U-Boot] [PATCH 4/6] net: mii: Changes not made by spatch Joe Hershberger
2016-04-05  6:13   ` Stefan Roese
2016-04-11  4:23   ` Bin Meng
2016-04-04  9:07 ` [U-Boot] [PATCH 5/6] net: mii: Fix changes " Joe Hershberger
2016-04-11  4:23   ` Bin Meng
2016-04-04  9:07 ` [U-Boot] [PATCH 6/6] net: mii: Clean up legacy glue that is not used Joe Hershberger
2016-04-11  4:23   ` Bin Meng
2016-04-09 18:35 ` [U-Boot] [PATCH 0/6] Remove calls from net drivers to legacy miiphy_register() entry-point Simon Glass
2016-08-08 16:28 ` [U-Boot] [PATCH v2 0/5] " Joe Hershberger
2016-08-08 16:28   ` [U-Boot] [PATCH v2 1/5] scripts: Add a cocci patch for miiphy_register Joe Hershberger
2016-08-15 20:33     ` [U-Boot] " Joe Hershberger
2016-08-08 16:28   ` [U-Boot] [PATCH v2 2/5] net: mii: Use spatch to update miiphy_register Joe Hershberger
2016-08-15 20:33     ` [U-Boot] " Joe Hershberger
2016-08-08 16:28   ` [U-Boot] [PATCH v2 3/5] net: mii: Changes not made by spatch Joe Hershberger
2016-08-15 20:33     ` [U-Boot] " Joe Hershberger
2016-08-08 16:28   ` [U-Boot] [PATCH v2 4/5] net: mii: Fix changes " Joe Hershberger
2016-08-15 20:33     ` [U-Boot] " Joe Hershberger
2016-08-08 16:28   ` Joe Hershberger [this message]
2016-08-15 20:33     ` [U-Boot] net: mii: Clean up legacy glue that is not used Joe Hershberger
2016-08-19  8:14   ` [U-Boot] [PATCH v2 0/5] Remove calls from net drivers to legacy miiphy_register() entry-point Karl Beldan
2016-08-19 18:48     ` Joe Hershberger
2016-08-20  9:08       ` Karl Beldan

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=1470673721-20590-6-git-send-email-joe.hershberger@ni.com \
    --to=joe.hershberger@ni.com \
    --cc=u-boot@lists.denx.de \
    /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.