[U-Boot] [PATCH v5 1/3] net: phy: Optionally force master mode for RTL PHY

Joe Hershberger joe.hershberger at gmail.com
Mon Mar 28 18:12:25 CEST 2016


Hi Hans,

On Mon, Mar 28, 2016 at 7:54 AM, Hans de Goede <hdegoede at redhat.com> wrote:
> Hi,
>
>
> On 25-03-16 22:11, Joe Hershberger wrote:
>>
>> On Fri, Mar 25, 2016 at 12:22 PM, Michael Haas
>> <haas at computerlinguist.org> wrote:
>>>
>>> This patch introduces CONFIG_RTL8211X_PHY_FORCE_MASTER. If this
>>> define is set, RTL8211x PHYs (except for the RTL8211F) will have their
>>> 1000BASE-T master/slave autonegotiation disabled and forced to master
>>> mode.
>>>
>>> This is helpful for PHYs like the RTL8211C which produce unstable links
>>> in slave mode. Such problems have been found on the A20-Olimex-SOM-EVB
>>> and A20-OLinuXino-Lime2.
>>>
>>> There is no proper way to identify affected PHYs in software as the
>>> RTL8211C shares its UID with the RTL8211B. Thus, this fix requires
>>> the introduction of an #ifdef.
>>>
>>> CC: fradav at gmail.com
>>> CC: merker at debian.org
>>> CC: hdegoede at redhat.com
>>> CC: ijc at hellion.org.uk
>>> CC: joe.hershberger at ni.com
>>>
>>> Signed-off-by: Michael Haas <haas at computerlinguist.org>
>>> Tested-by: Karsten Merker <merker at debian.org>
>>
>>
>> Acked-by: Joe Hershberger <joe.hershberger at ni.com>
>
>
> Series looks good to me:
>
> Acked-by: Hans de Goede <hdegoede at redhat.com>
>
> Joe, how do you want to proceed with upstreaming this, shall I take
> all 3 patches upstream through u-boot-sunxi ?

Sounds good to me.

-Joe


More information about the U-Boot mailing list