From mboxrd@z Thu Jan  1 00:00:00 1970
From: Albert ARIBAUD <albert.u.boot@aribaud.net>
Date: Sun, 05 Feb 2012 14:26:57 +0100
Subject: [U-Boot] [PATCH 2/2] RFC: Let linker create phy array
In-Reply-To: <201202042238.42038.vapier@gentoo.org>
References: <1328410966-13946-1-git-send-email-troy.kisky@boundarydevices.com>
	<1328410966-13946-2-git-send-email-troy.kisky@boundarydevices.com>
	<201202042238.42038.vapier@gentoo.org>
Message-ID: <4F2E83A1.8020503@aribaud.net>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

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 remove the static ?  that shouldn't affect the section name
> that it gets placed into.
>
>> --- a/include/phy.h
>> +++ b/include/phy.h
>>
>> +extern struct phy_driver __phy_entry_start, __phy_entry_end;
>
> linker symbols should be declared like:
> 	extern char __phy_entry_start[];

Why should they?

Amicalement,
-- 
Albert.