[U-Boot] [PATCH] miiphy: Note that miiphy_* API is deprecated

[U-Boot] [PATCH] miiphy: Note that miiphy_* API is deprecated

[Andy Fleming]

We want to move everything to phylib, and we definitely don't want
new drivers using the miiphy infrastructure.

Signed-off-by: Andy Fleming <afleming@freescale.com>
---
 common/miiphyutil.c              |   10 ++++++++++
 doc/feature-removal-schedule.txt |   11 +++++++++++
 2 files changed, 21 insertions(+), 0 deletions(-)

diff --git a/common/miiphyutil.c b/common/miiphyutil.c
index 35ad357..2cc23b4 100644
--- a/common/miiphyutil.c
+++ b/common/miiphyutil.c
@@ -102,6 +102,7 @@ static int legacy_miiphy_write(struct mii_dev *bus, int addr, int devad,
 /*****************************************************************************
  *
  * 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,
@@ -281,6 +282,8 @@ static struct mii_dev *miiphy_get_active_dev(const char *devname)
  * Read to variable <value> from the PHY attached to device <devname>,
  * use PHY address <addr> and register <reg>.
  *
+ * This API is deprecated. Use phy_read on a phy_device found via phy_connect
+ *
  * Returns:
  *   0 on success
  */
@@ -307,6 +310,8 @@ int miiphy_read(const char *devname, unsigned char addr, unsigned char reg,
  * Write <value> to the PHY attached to device <devname>,
  * use PHY address <addr> and register <reg>.
  *
+ * This API is deprecated. Use phy_write on a phy_device found by phy_connect
+ *
  * Returns:
  *   0 on success
  */
@@ -350,6 +355,8 @@ void miiphy_listdev(void)
  * Model:    6 bits (unsigned char)
  * Revision: 4 bits (unsigned char)
  *
+ * This API is deprecated.
+ *
  * Returns:
  *   0 on success
  */
@@ -389,6 +396,9 @@ int miiphy_info(const char *devname, unsigned char addr, unsigned int *oui,
 /*****************************************************************************
  *
  * Reset the PHY.
+ *
+ * This API is deprecated. Use PHYLIB.
+ *
  * Returns:
  *   0 on success
  */
diff --git a/doc/feature-removal-schedule.txt b/doc/feature-removal-schedule.txt
index 00d87e4..e04ba2d 100644
--- a/doc/feature-removal-schedule.txt
+++ b/doc/feature-removal-schedule.txt
@@ -7,6 +7,17 @@ file.
 
 ---------------------------
 
+What:	Users of the legacy miiphy_* code
+When:	undetermined
+
+Why:	We now have a PHY library, which allows everyone to share PHY
+	drivers. All new drivers should use this infrastructure, and
+	all old drivers should get converted to use it.
+
+Who:	Andy Fleming <afleming@freescale.com> and driver maintainers
+
+---------------------------
+
 What:	boards with xxx_config targets in top level Makefile
 When:	Release v2012.03
 
-- 
1.7.3.4

[U-Boot] [PATCH] miiphy: Note that miiphy_* API is deprecated

[Tabi Timur-B04825]

On Mon, Oct 31, 2011 at 9:46 AM, Andy Fleming <afleming@freescale.com> wrote:
> We want to move everything to phylib, and we definitely don't want
> new drivers using the miiphy infrastructure.

How about using gcc's deprecated function feature?  Or is that too aggressive?

http://gcc.gnu.org/onlinedocs/gcc/Function-Attributes.html

-- 
Timur Tabi
Linux kernel developer at Freescale

[U-Boot] [PATCH] miiphy: Note that miiphy_* API is deprecated

[Andy Fleming]

On Oct 31, 2011, at 10:08 AM, Tabi Timur-B04825 wrote:

> On Mon, Oct 31, 2011 at 9:46 AM, Andy Fleming <afleming@freescale.com> wrote:
>> We want to move everything to phylib, and we definitely don't want
>> new drivers using the miiphy infrastructure.
> 
> How about using gcc's deprecated function feature?  Or is that too aggressive?


I don't think we want every net driver prior to the last release to create a warning. We may switch to that after we get some momentum on switching drivers over. The first goal is just to provide information that a new driver-writer may see so that the old API doesn't expand.

Andy

[U-Boot] [PATCH] miiphy: Note that miiphy_* API is deprecated

[Mike Frysinger]

On Monday 31 October 2011 10:46:13 Andy Fleming wrote:
> +What:	Users of the legacy miiphy_* code
> +When:	undetermined
> +
> +Why:	We now have a PHY library, which allows everyone to share PHY
> +	drivers. All new drivers should use this infrastructure, and
> +	all old drivers should get converted to use it.

could you explicitly mention the defines which are considered "old" and the 
ones that are considered "new" here ?

and update the top level README as well (if the old defines are mentioned) ?
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20111031/b2d8e4c6/attachment.pgp 

[U-Boot] [PATCH] miiphy: Note that miiphy_* API is deprecated

[Mike Frysinger]

On Monday 31 October 2011 11:12:12 Andy Fleming wrote:
> On Oct 31, 2011, at 10:08 AM, Tabi Timur-B04825 wrote:
> > On Mon, Oct 31, 2011 at 9:46 AM, Andy Fleming wrote:
> >> We want to move everything to phylib, and we definitely don't want
> >> new drivers using the miiphy infrastructure.
> > 
> > How about using gcc's deprecated function feature?  Or is that too
> > aggressive?
> 
> I don't think we want every net driver prior to the last release to create
> a warning. We may switch to that after we get some momentum on switching
> drivers over. The first goal is just to provide information that a new
> driver-writer may see so that the old API doesn't expand.

right, this is why i didn't suggest a #warning or __deprecated before.  i'm 
pretty sure way more code is using the old phy layer than the new phy layer 
atm.

i also want to say that the new phy layer doesn't have all the support that 
the old one did ... if you look in include/miiphy.h, there are a few defines at 
the end there which are not in linux/mii.h.
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20111031/296b8b1e/attachment.pgp 

[U-Boot] [PATCH] miiphy: Note that miiphy_* API is deprecated

[Wolfgang Denk]

Dear Andy Fleming,

In message <1320072373-367-1-git-send-email-afleming@freescale.com> you wrote:
> We want to move everything to phylib, and we definitely don't want
> new drivers using the miiphy infrastructure.
> 
> Signed-off-by: Andy Fleming <afleming@freescale.com>
> ---
>  common/miiphyutil.c              |   10 ++++++++++
>  doc/feature-removal-schedule.txt |   11 +++++++++++
>  2 files changed, 21 insertions(+), 0 deletions(-)

Applied, thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Niklaus Wirth has lamented that, whereas Europeans pronounce his name
correctly (Ni-klows Virt), Americans invariably mangle it into (Nick-
les Worth). Which is to say that Europeans  call  him  by  name,  but
Americans call him by value.