From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [PATCH v2 4/4] Documentation: dt: OMAP: Update Overo/Tobi Date: Fri, 14 Feb 2014 08:39:42 -0800 Message-ID: <20140214163942.GB21592@atomide.com> References: <1392287116-4548-1-git-send-email-florian.vaussard@epfl.ch> <1392287116-4548-5-git-send-email-florian.vaussard@epfl.ch> <52FCDA29.90605@ti.com> <52FDCC9B.2050608@epfl.ch> <52FDD082.9090006@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mho-03-ewr.mailhop.org ([204.13.248.66]:21181 "EHLO mho-01-ewr.mailhop.org" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751822AbaBNQj5 (ORCPT ); Fri, 14 Feb 2014 11:39:57 -0500 Content-Disposition: inline In-Reply-To: <52FDD082.9090006@ti.com> Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Nishanth Menon Cc: florian.vaussard@epfl.ch, =?utf-8?Q?Beno=C3=AEt?= Cousson , Kevin Hilman , linux-omap , linux-arm-kernel@lists.infradead.org * Nishanth Menon [140214 00:17]: > On 02/14/2014 01:58 AM, Florian Vaussard wrote: > > Hi, > > > > On 02/13/2014 03:43 PM, Nishanth Menon wrote: > >> On 02/13/2014 04:25 AM, Florian Vaussard wrote: > >>> Update the compatible string for Overo/Tobi to reflect the latest > >>> changes. > >>> > >>> Signed-off-by: Florian Vaussard > >>> --- > >>> Documentation/devicetree/bindings/arm/omap/omap.txt | 2 +- > >>> 1 file changed, 1 insertion(+), 1 deletion(-) > >>> > >>> diff --git a/Documentation/devicetree/bindings/arm/omap/omap.txt b/Documentation/devicetree/bindings/arm/omap/omap.txt > >>> index 34dc40c..af9b4a0 100644 > >>> --- a/Documentation/devicetree/bindings/arm/omap/omap.txt > >>> +++ b/Documentation/devicetree/bindings/arm/omap/omap.txt > >>> @@ -91,7 +91,7 @@ Boards: > >>> compatible = "ti,omap3-beagle", "ti,omap3" > >>> > >>> - OMAP3 Tobi with Overo : Commercial expansion board with daughter board > >>> - compatible = "ti,omap3-tobi", "ti,omap3-overo", "ti,omap3" > >>> + compatible = "gumstix,omap3-overo-tobi", "gumstix,omap3-overo", "ti,omap3" > >> omap3-overo-storm-tobi.dts is not covered here, so, I wonder as the > >> number of boards supported keep increasing, is'nt it better we drop > >> the board information from omap.txt? > >> > > > > I am not opposed to this. We will just have checkpatch warnings due to > > undocumented compatible strings. > > Aaah, I seemed to have missed bff5da43 commit, thanks for pointing me > at it - unrelated to this patch, I think the board cleanups is a > little overdue :( > > Acked-by: Nishanth Menon OK thanks picking up this too. Tony From mboxrd@z Thu Jan 1 00:00:00 1970 From: tony@atomide.com (Tony Lindgren) Date: Fri, 14 Feb 2014 08:39:42 -0800 Subject: [PATCH v2 4/4] Documentation: dt: OMAP: Update Overo/Tobi In-Reply-To: <52FDD082.9090006@ti.com> References: <1392287116-4548-1-git-send-email-florian.vaussard@epfl.ch> <1392287116-4548-5-git-send-email-florian.vaussard@epfl.ch> <52FCDA29.90605@ti.com> <52FDCC9B.2050608@epfl.ch> <52FDD082.9090006@ti.com> Message-ID: <20140214163942.GB21592@atomide.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org * Nishanth Menon [140214 00:17]: > On 02/14/2014 01:58 AM, Florian Vaussard wrote: > > Hi, > > > > On 02/13/2014 03:43 PM, Nishanth Menon wrote: > >> On 02/13/2014 04:25 AM, Florian Vaussard wrote: > >>> Update the compatible string for Overo/Tobi to reflect the latest > >>> changes. > >>> > >>> Signed-off-by: Florian Vaussard > >>> --- > >>> Documentation/devicetree/bindings/arm/omap/omap.txt | 2 +- > >>> 1 file changed, 1 insertion(+), 1 deletion(-) > >>> > >>> diff --git a/Documentation/devicetree/bindings/arm/omap/omap.txt b/Documentation/devicetree/bindings/arm/omap/omap.txt > >>> index 34dc40c..af9b4a0 100644 > >>> --- a/Documentation/devicetree/bindings/arm/omap/omap.txt > >>> +++ b/Documentation/devicetree/bindings/arm/omap/omap.txt > >>> @@ -91,7 +91,7 @@ Boards: > >>> compatible = "ti,omap3-beagle", "ti,omap3" > >>> > >>> - OMAP3 Tobi with Overo : Commercial expansion board with daughter board > >>> - compatible = "ti,omap3-tobi", "ti,omap3-overo", "ti,omap3" > >>> + compatible = "gumstix,omap3-overo-tobi", "gumstix,omap3-overo", "ti,omap3" > >> omap3-overo-storm-tobi.dts is not covered here, so, I wonder as the > >> number of boards supported keep increasing, is'nt it better we drop > >> the board information from omap.txt? > >> > > > > I am not opposed to this. We will just have checkpatch warnings due to > > undocumented compatible strings. > > Aaah, I seemed to have missed bff5da43 commit, thanks for pointing me > at it - unrelated to this patch, I think the board cleanups is a > little overdue :( > > Acked-by: Nishanth Menon OK thanks picking up this too. Tony