From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752099AbcFVKgy (ORCPT ); Wed, 22 Jun 2016 06:36:54 -0400 Received: from bear.ext.ti.com ([198.47.19.11]:58951 "EHLO bear.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751262AbcFVKgr (ORCPT ); Wed, 22 Jun 2016 06:36:47 -0400 Subject: Re: [PATCH 12/15] net: davinci_mdio: document missed "ti,am4372-mdio" compat string To: Rob Herring References: <20160615115603.4897-1-grygorii.strashko@ti.com> <20160615115603.4897-13-grygorii.strashko@ti.com> <20160619143543.GA14844@rob-hp-laptop> CC: "David S. Miller" , , Sekhar Nori , Mugunthan V N , , , Ivan Khoronzhuk , From: Grygorii Strashko Message-ID: <576A67E5.5050106@ti.com> Date: Wed, 22 Jun 2016 13:26:45 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.8.0 MIME-Version: 1.0 In-Reply-To: <20160619143543.GA14844@rob-hp-laptop> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 06/19/2016 05:35 PM, Rob Herring wrote: > On Wed, Jun 15, 2016 at 02:56:00PM +0300, Grygorii Strashko wrote: >> Document missed "ti,am4372-mdio" compat string used for TI am437x SoC >> (am4372.dtsi). >> >> Signed-off-by: Grygorii Strashko >> --- >> Documentation/devicetree/bindings/net/davinci-mdio.txt | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/Documentation/devicetree/bindings/net/davinci-mdio.txt b/Documentation/devicetree/bindings/net/davinci-mdio.txt >> index 0369e25..f2bba50 100644 >> --- a/Documentation/devicetree/bindings/net/davinci-mdio.txt >> +++ b/Documentation/devicetree/bindings/net/davinci-mdio.txt >> @@ -2,7 +2,8 @@ TI SoC Davinci/Keystone2 MDIO Controller Device Tree Bindings >> --------------------------------------------------- >> >> Required properties: >> -- compatible : Should be "ti,davinci_mdio" or "ti,keystone_mdio" >> +- compatible : Should be "ti,davinci_mdio", "ti,keystone_mdio", >> + "ti,am4372-mdio" > > This is still an OR relationship? I think it's AND relation. It should be smth. like this: [SoC,][IP family,] "ti,davinci_mdio" where "ti,davinci_mdio" is mandatory always [IP family] - mandatory for IP/SOC family and can be "ti,keystone_mdio" or "ti,cpsw-mdio" (last one is added by subsequent patch) [SoC] - mandatory for SoC. now can be "ti,am4372-mdio" I can update it as below: - compatible : Should be "ti,davinci_mdio" and "ti,keystone_mdio" for Keystone 2 SoCs and "ti,am4372-mdio" for am472x SoC and in subsequent patch and "ti,cpsw-mdio" for am335x, am472x, am57xx/dra7, dm814x SoCs and "ti,am4372-mdio" for am472x SoC > > It's preferred to list one per line. ok -- regards, -grygorii