From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from devils.ext.ti.com ([198.47.26.153]) by merlin.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1VltTz-0001h4-6x for linux-mtd@lists.infradead.org; Thu, 28 Nov 2013 04:45:43 +0000 Message-ID: <5296CA5A.90304@ti.com> Date: Thu, 28 Nov 2013 10:15:14 +0530 From: Sourav Poddar MIME-Version: 1.0 To: Mark Brown Subject: Re: [PATCH 03/17] spi/qspi: parse register by name. References: <1385451313-1875-1-git-send-email-sourav.poddar@ti.com> <1385451313-1875-4-git-send-email-sourav.poddar@ti.com> <20131126103029.GY14725@sirena.org.uk> <5294835D.5090902@ti.com> <20131126122442.GC14725@sirena.org.uk> In-Reply-To: <20131126122442.GC14725@sirena.org.uk> Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Cc: devicetree@vger.kernel.org, computersforpeace@gmail.com, balbi@ti.com, linux-spi@vger.kernel.org, linux-mtd@lists.infradead.org, bcousson@baylibre.com, linux-omap@vger.kernel.org List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Tuesday 26 November 2013 05:54 PM, Mark Brown wrote: > On Tue, Nov 26, 2013 at 04:47:49PM +0530, Sourav Poddar wrote: > >> I have added "reg-names" property in dt bindings. (17/17) patch of >> this series has that info. > Do device tree binding changes early on in the series rather than at the > end, one of the things that gets reviewed for is binding updates. > > Your code should also cope with the existing bindings, falling back on > the current behaviour if it can't get the names or the extra resources. My current code only has a single resouce, hence "*_byname* variants was not used. With extra resources, I am switching to "byname" variants, So, if a particular resource name is not found, it should just bypass that, thinking the resource is not required and should not resort to existing bindings.. From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sourav Poddar Subject: Re: [PATCH 03/17] spi/qspi: parse register by name. Date: Thu, 28 Nov 2013 10:15:14 +0530 Message-ID: <5296CA5A.90304@ti.com> References: <1385451313-1875-1-git-send-email-sourav.poddar@ti.com> <1385451313-1875-4-git-send-email-sourav.poddar@ti.com> <20131126103029.GY14725@sirena.org.uk> <5294835D.5090902@ti.com> <20131126122442.GC14725@sirena.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from devils.ext.ti.com ([198.47.26.153]:46939 "EHLO devils.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754471Ab3K1Epo (ORCPT ); Wed, 27 Nov 2013 23:45:44 -0500 In-Reply-To: <20131126122442.GC14725@sirena.org.uk> Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Mark Brown Cc: computersforpeace@gmail.com, linux-mtd@lists.infradead.org, linux-spi@vger.kernel.org, balbi@ti.com, linux-omap@vger.kernel.org, devicetree@vger.kernel.org, bcousson@baylibre.com On Tuesday 26 November 2013 05:54 PM, Mark Brown wrote: > On Tue, Nov 26, 2013 at 04:47:49PM +0530, Sourav Poddar wrote: > >> I have added "reg-names" property in dt bindings. (17/17) patch of >> this series has that info. > Do device tree binding changes early on in the series rather than at the > end, one of the things that gets reviewed for is binding updates. > > Your code should also cope with the existing bindings, falling back on > the current behaviour if it can't get the names or the extra resources. My current code only has a single resouce, hence "*_byname* variants was not used. With extra resources, I am switching to "byname" variants, So, if a particular resource name is not found, it should just bypass that, thinking the resource is not required and should not resort to existing bindings.. From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sourav Poddar Subject: Re: [PATCH 03/17] spi/qspi: parse register by name. Date: Thu, 28 Nov 2013 10:15:14 +0530 Message-ID: <5296CA5A.90304@ti.com> References: <1385451313-1875-1-git-send-email-sourav.poddar@ti.com> <1385451313-1875-4-git-send-email-sourav.poddar@ti.com> <20131126103029.GY14725@sirena.org.uk> <5294835D.5090902@ti.com> <20131126122442.GC14725@sirena.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Cc: , , , , , , To: Mark Brown Return-path: In-Reply-To: <20131126122442.GC14725@sirena.org.uk> Sender: linux-omap-owner@vger.kernel.org List-Id: linux-spi.vger.kernel.org On Tuesday 26 November 2013 05:54 PM, Mark Brown wrote: > On Tue, Nov 26, 2013 at 04:47:49PM +0530, Sourav Poddar wrote: > >> I have added "reg-names" property in dt bindings. (17/17) patch of >> this series has that info. > Do device tree binding changes early on in the series rather than at the > end, one of the things that gets reviewed for is binding updates. > > Your code should also cope with the existing bindings, falling back on > the current behaviour if it can't get the names or the extra resources. My current code only has a single resouce, hence "*_byname* variants was not used. With extra resources, I am switching to "byname" variants, So, if a particular resource name is not found, it should just bypass that, thinking the resource is not required and should not resort to existing bindings..