From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alex Subject: Re: net: ti: cpsw-phy-sel: RGMII is not working on AM335x Date: Wed, 11 Jan 2017 12:18:54 -0800 Message-ID: References: <1484122463.3777.1.camel@phytec.de> <20170111151407.GW22820@lunn.ch> <1484150255.3777.13.camel@phytec.de> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1484150255.3777.13.camel@phytec.de> Sender: netdev-owner@vger.kernel.org To: Teresa Remmet , Andrew Lunn Cc: "David S. Miller" , Mugunthan V N , Grygorii Strashko , linux-omap@vger.kernel.org, netdev@vger.kernel.org List-Id: linux-omap@vger.kernel.org Hi Teresa On 01/11/2017 07:57 AM, Teresa Remmet wrote: > The phy we use is a KSZ9021. And yes we add delays to the > phy, as you can see. When looking to the dts documentation I probably > need to set the phy-mode to "rgmii-id" instead, as the phy is providing > the delays. Before the change I submitted, every "rgmii*" link was being set up as "rgmii-id", regardless of phy-mode. I suspected there might have been the odd board here or there that used "rgmii" when the actual mode is "rgmii-id". What confused me the first time around was that I didn't realize the delay applies to the phy, not the mac. So "-id" means "the PHY provides the delays, so don't add delays on the MAC", just like you realized. > I make a quick test with that change and it is working. So this seems > to solve my problem. Thank you for the hint. I'm glad it was an easy fix. Last time I had issues with RGMII, I had to pull out the oscilloscope. Alex > Regards, > Teresa >