Re: [U-Boot] [PATCH 2/2] RFC: Let linker create phy array

2012-02-12 Thread Albert ARIBAUD
Le 10/02/2012 22:41, Mike Frysinger a écrit : On Friday 10 February 2012 15:57:50 Albert ARIBAUD wrote: Le 10/02/2012 21:32, Mike Frysinger a écrit : On Friday 10 February 2012 14:39:12 Albert ARIBAUD wrote: Le 07/02/2012 16:20, Mike Frysinger a écrit : On Monday 06 February 2012 16:01:56 Alb

Re: [U-Boot] [PATCH 2/2] RFC: Let linker create phy array

2012-02-10 Thread Mike Frysinger
On Friday 10 February 2012 15:57:50 Albert ARIBAUD wrote: > Le 10/02/2012 21:32, Mike Frysinger a écrit : > > On Friday 10 February 2012 14:39:12 Albert ARIBAUD wrote: > >> Le 07/02/2012 16:20, Mike Frysinger a écrit : > >>> On Monday 06 February 2012 16:01:56 Albert ARIBAUD wrote: > Le 06/02/

Re: [U-Boot] [PATCH 2/2] RFC: Let linker create phy array

2012-02-10 Thread Albert ARIBAUD
Le 10/02/2012 21:32, Mike Frysinger a écrit : On Friday 10 February 2012 14:39:12 Albert ARIBAUD wrote: Le 07/02/2012 16:20, Mike Frysinger a écrit : On Monday 06 February 2012 16:01:56 Albert ARIBAUD wrote: Le 06/02/2012 21:57, Mike Frysinger a écrit : Is there a keep attribute like the link

Re: [U-Boot] [PATCH 2/2] RFC: Let linker create phy array

2012-02-10 Thread Mike Frysinger
On Friday 10 February 2012 14:39:12 Albert ARIBAUD wrote: > Le 07/02/2012 16:20, Mike Frysinger a écrit : > > On Monday 06 February 2012 16:01:56 Albert ARIBAUD wrote: > >> Le 06/02/2012 21:57, Mike Frysinger a écrit : > Is there a keep attribute like the linker has for sections? > >>> > >>>

Re: [U-Boot] [PATCH 2/2] RFC: Let linker create phy array

2012-02-10 Thread Albert ARIBAUD
Le 07/02/2012 16:20, Mike Frysinger a écrit : On Monday 06 February 2012 16:01:56 Albert ARIBAUD wrote: Le 06/02/2012 21:57, Mike Frysinger a écrit : The would give me compiler warnings for unused variables. How does that help? does gcc issue warnings ? doesn't seem to do so for me. Some d

Re: [U-Boot] [PATCH 2/2] RFC: Let linker create phy array

2012-02-07 Thread Mike Frysinger
On Monday 06 February 2012 16:44:36 Troy Kisky wrote: > On 2/6/2012 1:57 PM, Mike Frysinger wrote: > > On Monday 06 February 2012 15:17:32 Troy Kisky wrote: > >> On 2/6/2012 12:07 PM, Mike Frysinger wrote: > >>> On Monday 06 February 2012 13:48:13 Troy Kisky wrote: > On 2/4/2012 8:38 PM, Mike

Re: [U-Boot] [PATCH 2/2] RFC: Let linker create phy array

2012-02-07 Thread Mike Frysinger
On Monday 06 February 2012 16:01:56 Albert ARIBAUD wrote: > Le 06/02/2012 21:57, Mike Frysinger a écrit : > >> The would give me compiler warnings for unused variables. How does that > >> help? > > > > does gcc issue warnings ? doesn't seem to do so for me. > > Some do, and some will. vague ...

Re: [U-Boot] [PATCH 2/2] RFC: Let linker create phy array

2012-02-06 Thread Troy Kisky
On 2/6/2012 1:57 PM, Mike Frysinger wrote: On Monday 06 February 2012 15:17:32 Troy Kisky wrote: On 2/6/2012 12:07 PM, Mike Frysinger wrote: On Monday 06 February 2012 13:48:13 Troy Kisky wrote: On 2/4/2012 8:38 PM, Mike Frysinger wrote: On Saturday 04 February 2012 22:02:46 Troy Kisky wrote:

Re: [U-Boot] [PATCH 2/2] RFC: Let linker create phy array

2012-02-06 Thread Albert ARIBAUD
Le 06/02/2012 21:57, Mike Frysinger a écrit : The would give me compiler warnings for unused variables. How does that help? does gcc issue warnings ? doesn't seem to do so for me. Some do, and some will. Is there a keep attribute like the linker has for sections? yes, __attribute__((use

Re: [U-Boot] [PATCH 2/2] RFC: Let linker create phy array

2012-02-06 Thread Mike Frysinger
On Monday 06 February 2012 15:17:32 Troy Kisky wrote: > On 2/6/2012 12:07 PM, Mike Frysinger wrote: > > On Monday 06 February 2012 13:48:13 Troy Kisky wrote: > >> On 2/4/2012 8:38 PM, Mike Frysinger wrote: > >>> On Saturday 04 February 2012 22:02:46 Troy Kisky wrote: > --- a/drivers/net/phy/br

Re: [U-Boot] [PATCH 2/2] RFC: Let linker create phy array

2012-02-06 Thread Albert ARIBAUD
Le 06/02/2012 21:17, Troy Kisky a écrit : On 2/6/2012 12:07 PM, Mike Frysinger wrote: On Monday 06 February 2012 13:48:13 Troy Kisky wrote: On 2/4/2012 8:38 PM, Mike Frysinger wrote: On Saturday 04 February 2012 22:02:46 Troy Kisky wrote: --- a/drivers/net/phy/broadcom.c +++ b/drivers/net/phy

Re: [U-Boot] [PATCH 2/2] RFC: Let linker create phy array

2012-02-06 Thread Albert ARIBAUD
Le 05/02/2012 21:40, Mike Frysinger a écrit : On Sunday 05 February 2012 08:26:57 Albert ARIBAUD wrote: Le 05/02/2012 04:38, Mike Frysinger a écrit : On Saturday 04 February 2012 22:02:46 Troy Kisky wrote: --- a/include/phy.h +++ b/include/phy.h +extern struct phy_driver __phy_entry_start, __

Re: [U-Boot] [PATCH 2/2] RFC: Let linker create phy array

2012-02-06 Thread Troy Kisky
On 2/6/2012 12:07 PM, Mike Frysinger wrote: On Monday 06 February 2012 13:48:13 Troy Kisky wrote: On 2/4/2012 8:38 PM, Mike Frysinger wrote: On Saturday 04 February 2012 22:02:46 Troy Kisky wrote: --- a/drivers/net/phy/broadcom.c +++ b/drivers/net/phy/broadcom.c -static struct phy_driver BCM5

Re: [U-Boot] [PATCH 2/2] RFC: Let linker create phy array

2012-02-06 Thread Mike Frysinger
On Monday 06 February 2012 13:48:13 Troy Kisky wrote: > On 2/4/2012 8:38 PM, Mike Frysinger wrote: > > On Saturday 04 February 2012 22:02:46 Troy Kisky wrote: > >> --- a/drivers/net/phy/broadcom.c > >> +++ b/drivers/net/phy/broadcom.c > >> > >> -static struct phy_driver BCM5461S_driver = { > >> +s

Re: [U-Boot] [PATCH 2/2] RFC: Let linker create phy array

2012-02-06 Thread Troy Kisky
On 2/4/2012 8:38 PM, Mike Frysinger wrote: On Saturday 04 February 2012 22:02:46 Troy Kisky wrote: --- a/drivers/net/phy/broadcom.c +++ b/drivers/net/phy/broadcom.c -static struct phy_driver BCM5461S_driver = { +struct phy_driver BCM5461S_driver __phy_entry = { why do you have to remove the st

Re: [U-Boot] [PATCH 2/2] RFC: Let linker create phy array

2012-02-05 Thread Mike Frysinger
On Sunday 05 February 2012 08:26:57 Albert ARIBAUD wrote: > Le 05/02/2012 04:38, Mike Frysinger a écrit : > > On Saturday 04 February 2012 22:02:46 Troy Kisky wrote: > >> --- a/include/phy.h > >> +++ b/include/phy.h > >> > >> +extern struct phy_driver __phy_entry_start, __phy_entry_end; > > > > l

Re: [U-Boot] [PATCH 2/2] RFC: Let linker create phy array

2012-02-05 Thread Albert ARIBAUD
Hi Mike, Le 05/02/2012 04:38, Mike Frysinger a écrit : On Saturday 04 February 2012 22:02:46 Troy Kisky wrote: --- a/drivers/net/phy/broadcom.c +++ b/drivers/net/phy/broadcom.c -static struct phy_driver BCM5461S_driver = { +struct phy_driver BCM5461S_driver __phy_entry = { why do you have to

Re: [U-Boot] [PATCH 2/2] RFC: Let linker create phy array

2012-02-04 Thread Dirk Behme
On 05.02.2012 04:38, Mike Frysinger wrote: On Saturday 04 February 2012 22:02:46 Troy Kisky wrote: --- a/drivers/net/phy/broadcom.c +++ b/drivers/net/phy/broadcom.c -static struct phy_driver BCM5461S_driver = { +struct phy_driver BCM5461S_driver __phy_entry = { why do you have to remove the s

Re: [U-Boot] [PATCH 2/2] RFC: Let linker create phy array

2012-02-04 Thread Mike Frysinger
On Saturday 04 February 2012 22:02:46 Troy Kisky wrote: > --- a/drivers/net/phy/broadcom.c > +++ b/drivers/net/phy/broadcom.c > > -static struct phy_driver BCM5461S_driver = { > +struct phy_driver BCM5461S_driver __phy_entry = { why do you have to remove the static ? that shouldn't affect the se

[U-Boot] [PATCH 2/2] RFC: Let linker create phy array

2012-02-04 Thread Troy Kisky
Signed-off-by: Troy Kisky --- drivers/net/phy/atheros.c|9 + drivers/net/phy/broadcom.c | 15 +++ drivers/net/phy/davicom.c|9 + drivers/net/phy/lxt.c|9 + drivers/net/phy/marvell.c| 24 ++-- drivers/net