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

Troy Kisky troy.kisky at boundarydevices.com
Mon Feb 6 21:17:32 CET 2012


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 BCM5461S_driver = {
>>>> +struct phy_driver BCM5461S_driver __phy_entry = {
>>> why do you have to remove the static ?  that shouldn't affect the section
>>> name that it gets placed into.
>> I had static to start. But the compiler ate all of the code. No
>> references to any of the static symbols.
> sounds like you should change the __phy_entry define from "unused" to "used"
> -mike
The would give me compiler warnings for unused variables. How does that 
help?
Is there a keep attribute like the linker has for sections?

Troy





More information about the U-Boot mailing list