[U-Boot] [PATCH] net: macb: sama5d4 is not gigabit capable

Nicolas Ferre nicolas.ferre at atmel.com
Wed Dec 16 10:19:04 CET 2015


Le 15/12/2015 20:59, Joe Hershberger a écrit :
> Hi Gregory,
> 
> On Mon, Dec 14, 2015 at 10:37 AM, Gregory CLEMENT
> <gregory.clement at free-electrons.com> wrote:
>> During the initialization of PHY the gigabit bit capable is set if the
>> controller is a GEM. However, for sama5d4, the GEM is not gigabit
>> capable. Improperly setting the GBE capability leads to an unresponsive
>> MAC controller. This patch fix this behavior allowing to use the gmac
>> with the sama5d4.
>>
>> Suggested-by: Nicolas Ferre <nicolas.ferre at atmel.com>
>> Signed-off-by: Gregory CLEMENT <gregory.clement at free-electrons.com>
>> ---
>>  drivers/net/macb.c | 7 +++++--
>>  1 file changed, 5 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/net/macb.c b/drivers/net/macb.c
>> index a5c1880..642717d 100644
>> --- a/drivers/net/macb.c
>> +++ b/drivers/net/macb.c
>> @@ -480,8 +480,11 @@ static int macb_phy_init(struct macb_device *macb)
>>                 return 0;
>>         }
>>
>> -       /* First check for GMAC */
>> -       if (macb_is_gem(macb)) {
>> +       /*
>> +        * First check for GMAC, but not the one on SAMA5D4 which is
>> +        * not gigabit capabale
>> +        */
>> +       if (macb_is_gem(macb) && ! cpu_is_sama5d4()) {
> 
> Is there not some other property that can identify the lack of Gigabit
> support in the "GEM"? It would be better to not have to keep track of
> the next processor and the one after that which has the same
> situation.

Actually, sama5d2 is in the same situation already...

Bye,

> 
>>                 lpa = macb_mdio_read(macb, MII_STAT1000);
>>
>>                 if (lpa & (LPA_1000FULL | LPA_1000HALF)) {
>> --
>> 2.5.0
>>
>> _______________________________________________
>> U-Boot mailing list
>> U-Boot at lists.denx.de
>> http://lists.denx.de/mailman/listinfo/u-boot
> 


-- 
Nicolas Ferre


More information about the U-Boot mailing list