From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Steven J. Hill" Subject: Re: [PATCH v12 01/10] dt-bindings: Add Cavium Octeon Common Ethernet Interface. Date: Fri, 6 Jul 2018 17:10:39 -0500 Message-ID: References: <1530134719-19407-1-git-send-email-steven.hill@cavium.com> <1530134719-19407-2-git-send-email-steven.hill@cavium.com> <20180628083530.GE16727@lunn.ch> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org, Chandrakala Chavva To: Andrew Lunn Return-path: Received: from mail-sn1nam01on0066.outbound.protection.outlook.com ([104.47.32.66]:32698 "EHLO NAM01-SN1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S932412AbeGFWKr (ORCPT ); Fri, 6 Jul 2018 18:10:47 -0400 In-Reply-To: <20180628083530.GE16727@lunn.ch> Content-Language: en-US Sender: netdev-owner@vger.kernel.org List-ID: On 06/28/2018 03:35 AM, Andrew Lunn wrote: > >> +- cavium,rx-clk-delay-bypass: Set to <1> to bypass the rx clock delay setting. >> + Needed by the Micrel PHY. > > Could you explain this some more. Is it anything to do with RGMII delays? > Andrew, One of my colleagues tracked this down for me. This device tree option is in place because there are several different ways to do the clock and data with respect to RGMII. This controls the delay introduced for the RX clock with respect to the data. Without this, RX will not work with Micrel PHYs. Thanks. Steve