[2.6 patch] export genphy_restart_aneg

View: New views
4 Messages — Rating Filter:   Alert me  

[2.6 patch] export genphy_restart_aneg

by Adrian Bunk-3 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

This patch fixes the following build error caused by
commit ed94493fb38a665cebcf750dfabe8a6dd13e136f
(mv643xx_eth: convert to phylib):

<--  snip  -->

...
  Building modules, stage 2.
  MODPOST 1280 modules
ERROR: "genphy_restart_aneg" [drivers/net/mv643xx_eth.ko] undefined!
...
make[2]: *** [__modpost] Error 1

<--  snip  -->


Signed-off-by: Adrian Bunk <bunk@...>

---

diff --git a/drivers/net/phy/phy_device.c b/drivers/net/phy/phy_device.c
index 1716274..f11e900 100644
--- a/drivers/net/phy/phy_device.c
+++ b/drivers/net/phy/phy_device.c
@@ -557,6 +557,7 @@ int genphy_restart_aneg(struct phy_device *phydev)
 
  return ctl;
 }
+EXPORT_SYMBOL(genphy_restart_aneg);
 
 
 /**

_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@...
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Re: [2.6 patch] export genphy_restart_aneg

by Andy Fleming :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message


On Oct 13, 2008, at 12:19, Adrian Bunk wrote:

> This patch fixes the following build error caused by
> commit ed94493fb38a665cebcf750dfabe8a6dd13e136f
> (mv643xx_eth: convert to phylib):
>
> <--  snip  -->
>
> ...
>  Building modules, stage 2.
>  MODPOST 1280 modules
> ERROR: "genphy_restart_aneg" [drivers/net/mv643xx_eth.ko] undefined!
> ...
> make[2]: *** [__modpost] Error 1
>
> <--  snip  -->
>
>
> Signed-off-by: Adrian Bunk <bunk@...>


Could you get genphy_setup_forced, too?  It doesn't appear to be used,  
but it's currently in include/linux/phy.h, and not static.

Alternatively, we could change genphy_setup_forced to be static.  :)

Andy
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@...
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Re: [2.6 patch] export genphy_restart_aneg

by David Miller-13 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

From: Andy Fleming <afleming@...>
Date: Mon, 13 Oct 2008 13:04:30 -0500

>
> On Oct 13, 2008, at 12:19, Adrian Bunk wrote:
>
> > This patch fixes the following build error caused by
> > commit ed94493fb38a665cebcf750dfabe8a6dd13e136f
> > (mv643xx_eth: convert to phylib):
> >
> > <--  snip  -->
> >
> > ...
> >  Building modules, stage 2.
> >  MODPOST 1280 modules
> > ERROR: "genphy_restart_aneg" [drivers/net/mv643xx_eth.ko] undefined!
> > ...
> > make[2]: *** [__modpost] Error 1
> >
> > <--  snip  -->
> >
> >
> > Signed-off-by: Adrian Bunk <bunk@...>
>
>
> Could you get genphy_setup_forced, too?  It doesn't appear to be used, but it's currently in include/linux/phy.h, and not static.

When there is a use we can add the export.
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@...
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Re: [2.6 patch] export genphy_restart_aneg

by David Miller-13 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

From: Adrian Bunk <bunk@...>
Date: Mon, 13 Oct 2008 20:19:00 +0300

> This patch fixes the following build error caused by
> commit ed94493fb38a665cebcf750dfabe8a6dd13e136f
> (mv643xx_eth: convert to phylib):

Applied, th anks Adrian.
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@...
https://ozlabs.org/mailman/listinfo/linuxppc-dev
LightInTheBox - Buy quality products at wholesale price!