lkml.org 
[lkml]   [2019]   [Oct]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] ARM: dts: am335x-sancloud-bbe: Fix PHY mode for ethernet
From
Date


On 03/10/2019 11:16, Jeroen Hofstee wrote:
> Hello Grygorri,
>
> On 10/2/19 4:48 PM, Grygorii Strashko wrote:
>>
>>
>> On 02/10/2019 12:54, Jeroen Hofstee wrote:
>>> cd28d1d6e52e: ("net: phy: at803x: Disable phy delay for RGMII mode")
>>> broke
>>> the ethernet networking on the beaglebone enhanced.
>>
>> Above commit is incorrect (by itself) and there are few more commits
>> on top of
>> it, so pls. update reference to commit(s)
>>
>> bb0ce4c1517d net: phy: at803x: stop switching phy delay config needlessly
>> 6d4cd041f0af net: phy: at803x: disable delay only for RGMII mode
>>
>>
> I don't see why that is relevant. The mention patch introduces a
> backwards incompatibility for the device tree.

Pls read https://patchwork.kernel.org/patch/10773389/
The patch you've mentioned here is buggy by itself and not related to your
fix, but final at803x behavior actually defined by above two commits.

I've posted this commit because I was confused when i've checked commit you referenced.

The patches you
> mention don't fix that and hence are unrelated to this patch.

No. but they define new at803x behavior which is:
After commits (see above) at803x driver disable RX RGMII delay
if phy-mode = "rgmii-txid"
or will disable TX RGMII delay
if phy-mode = "rgmii-rxid"

Before above commits, the at803x driver was keeping RX or TX RGMII delay setting
untouched as per bootloader or bootstraping configuration for "rgmii-txid"/"rgmii-rxid".

>
> Furthermore 4.19 is fine, so there is no need to include it in stable
> and have a note to make sure also other patches are required etc.

Hence all above patches went in 5.1 it would be correct to mention only
6d4cd041f0af net: phy: at803x: disable delay only for RGMII mode

--
Best regards,
grygorii

\
 
 \ /
  Last update: 2019-10-03 11:32    [W:0.089 / U:0.012 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site