Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932194Ab1BWRzJ (ORCPT ); Wed, 23 Feb 2011 12:55:09 -0500 Received: from mail-yi0-f46.google.com ([209.85.218.46]:63161 "EHLO mail-yi0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754599Ab1BWRzG (ORCPT ); Wed, 23 Feb 2011 12:55:06 -0500 Date: Wed, 23 Feb 2011 10:54:59 -0700 From: Grant Likely To: Scott Wood Cc: Richard Cochran , Thomas Gleixner , Rodolfo Giometti , Arnd Bergmann , Peter Zijlstra , linux-api@vger.kernel.org, devicetree-discuss@lists.ozlabs.org, linux-kernel@vger.kernel.org, Russell King , Paul Mackerras , John Stultz , Alan Cox , netdev@vger.kernel.org, Mike Frysinger , Christoph Lameter , linuxppc-dev@lists.ozlabs.org, David Miller , linux-arm-kernel@lists.infradead.org, Krzysztof Halasa Subject: Re: [PATCH V11 2/4] ptp: Added a clock that uses the eTSEC found on the MPC85xx. Message-ID: <20110223175459.GH14597@angua.secretlab.ca> References: <20110223165058.GE14597@angua.secretlab.ca> <20110223112612.30071995@schlenkerla> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20110223112612.30071995@schlenkerla> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2510 Lines: 63 On Wed, Feb 23, 2011 at 11:26:12AM -0600, Scott Wood wrote: > On Wed, 23 Feb 2011 09:50:58 -0700 > Grant Likely wrote: > > > On Wed, Feb 23, 2011 at 11:38:17AM +0100, Richard Cochran wrote: > > > + > > > +* Gianfar PTP clock nodes > > > + > > > +General Properties: > > > + > > > + - compatible Should be "fsl,etsec-ptp" > > > > Should specify an *exact* part; ie: "fsl,mpc8313-etsec-ptp" instead of > > trying to define a generic catchall. The reason is that the same > > marketing name can end up getting applied to a wide range of parts. > > > > Instead, choose one specific device to stand in as the 'common' > > implementation and get all parts with the same core to claim > > compatibility with it. ie: a p2020 might have: > > > > compatible = "fsl,mpc2020-etsec-ptp", "fsl,mpc8313-etsec-ptp"; > > eTSEC is versioned, that's more reliable than the chip name since chips > have revisions (rev 2.1 of mpc8313 has eTSEC 1.6, not sure about previous > revs of mpc8313). Logic blocks can be and have been uprevved between one > revision of a chip to the next. I think "fsl,mpc8313rev2.1-etsec-ptp" > would be taking things a bit too far (and there could be board-level bugs > too...). > > If you really need to know the exact SoC you're on, look in SVR (which > will provide revision info as well). Isn't the device tree for things that > can't be probed? This is far more about the binding than it is about the chip revision. When documenting a binding it makes far more sense to anchor it to a specific implementation than to try and come up with a 'generic' catchall. A new binding means new compatible value and dropping any claims of being compatible with the old. It is not about the logic block version, particularly when it is detectable by the driver as you say. > > The eTSEC revision is probeable as well, but due the way PTP is described as > a separate node, the driver doesn't have straightforward access to those > registers. Ignorant question: Should the ptp be described as a separate node? > > Insisting on an explicit chip also encourages people to claim compatibility > with that chip without ensuring that it really is fully compatible. In practise, I've not seen this to be an issue. g. -- 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/