Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754460AbaA0VNQ (ORCPT ); Mon, 27 Jan 2014 16:13:16 -0500 Received: from mail-pd0-f176.google.com ([209.85.192.176]:47095 "EHLO mail-pd0-f176.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754118AbaA0VNO (ORCPT ); Mon, 27 Jan 2014 16:13:14 -0500 MIME-Version: 1.0 In-Reply-To: References: <1390795167-6677-1-git-send-email-jcmvbkbc@gmail.com> <1390795167-6677-4-git-send-email-jcmvbkbc@gmail.com> From: Florian Fainelli Date: Mon, 27 Jan 2014 13:12:33 -0800 Message-ID: Subject: Re: [PATCH 3/3] net: ethoc: document OF bindings To: Max Filippov Cc: Rob Herring , "linux-xtensa@linux-xtensa.org" , netdev , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , Chris Zankel , Marc Gauthier , "David S. Miller" , Grant Likely , Rob Herring Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 2014-01-27 Max Filippov : > On Mon, Jan 27, 2014 at 11:45 PM, Florian Fainelli wrote: >> 2014-01-27 Max Filippov : >>> On Mon, Jan 27, 2014 at 6:10 PM, Rob Herring wrote: >>>> On Sun, Jan 26, 2014 at 9:59 PM, Max Filippov wrote: > > [...] > >>>>> --- /dev/null >>>>> +++ b/Documentation/devicetree/bindings/net/opencores-ethoc.txt >>>>> @@ -0,0 +1,25 @@ >>>>> +* OpenCores MAC 10/100 Mbps >>>>> + >>>>> +Required properties: >>>>> +- compatible: Should be "opencores,ethoc". >>>> >>>> There are not different versions of IP or is the version probeable? >>> >>> AFAIK there's single version of this 10/100 MAC. >>> It doesn't have any identification registers. >> >> Since this is an IP block that people can modify due to its open >> source nature, it would have been good to define a revision register >> or such which would allow software to gate specific code based on that >> revision. > > Probably yes, though I haven't heard of any modified versions of this MAC > out there. > > [...] > >>>>> +Examples: >>>>> + >>>>> + enet0: ethoc@fd030000 { >>>>> + compatible = "opencores,ethoc"; >>>>> + reg = <0xfd030000 0x4000 0xfd800000 0x4000>; >>>>> + interrupts = <1>; >>>>> + local-mac-address = [00 50 c2 13 6f 00]; >>>>> + clock-frequency = <50000000>; >>>>> + mii-mgmt-clock-frequency = <5000000>; >> >> 5Mhz management clock? Can't you make it work with the standard 2.5Mhz >> management clock? Is not there a risk not to be able to "talk" to some >> PHY chips out there which do not support > 2.5Mhz management clock? > > Yes I can, it just didn't occur to me that there is a standard 2.5MHz setting. > >> Since this is an ethoc specific property, should it be prefixed with "ethoc,"? > > Is it worth keeping this parameter at all, or just always default to 2.5MHz? Your example lists 5Mhz, so I assume this is of some use for you on the platform you are working with? My concern is that this might not be compatible with all PHY devices out there and might create hard to debug issues where you get some MDIO transactions to succeeds and some which don't. -- Florian -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/