From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut To: Mark Brown Subject: Re: [PATCH V1 3/5] mtd: m25p80: add the quad-read support Date: Fri, 23 Aug 2013 01:58:05 +0200 References: <1376885403-12156-1-git-send-email-b32955@freescale.com> <201308222229.01778.marex@denx.de> <20130822233611.GA25263@sirena.org.uk> In-Reply-To: <20130822233611.GA25263@sirena.org.uk> MIME-Version: 1.0 Content-Type: Text/Plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Message-Id: <201308230158.05909.marex@denx.de> Cc: devicetree@vger.kernel.org, shawn.guo@linaro.org, b44548@freescale.com, dedekind1@gmail.com, b18965@freescale.com, linux-spi@vger.kernel.org, Huang Shijie , linux-mtd@lists.infradead.org, kernel@pengutronix.de, Brian Norris , dwmw2@infradead.org, wangyuhang , linux-arm-kernel@lists.infradead.org List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Dear Mark Brown, > On Thu, Aug 22, 2013 at 10:29:01PM +0200, Marek Vasut wrote: > > > On Thu, Aug 22, 2013 at 12:34:53PM -0700, Brian Norris wrote: > > > > > > This should be a generic property for all quad devices to use, though, > > > since the same thing applies everywhere. > > > > Full ACK, "m25p,dual-read" and "m25p,quad-read" sound like a good prop > > names? > > I was actually thinking something more generic than that - putting the > property at the SPI generic bindings level. Though if all flashes with > this dual/quad read functionality have the prefix m25p the above would > work also, at the minute this does seem to be mostly used by flash (I > bet someone's got some DSPs or something though). Ah! So you mean the SPI controller would provide information that it can do dual/quad transfers? But then, the additional pins can only be wired to certain chips (controller by certain CS lines). Best regards, Marek Vasut From mboxrd@z Thu Jan 1 00:00:00 1970 From: marex@denx.de (Marek Vasut) Date: Fri, 23 Aug 2013 01:58:05 +0200 Subject: [PATCH V1 3/5] mtd: m25p80: add the quad-read support In-Reply-To: <20130822233611.GA25263@sirena.org.uk> References: <1376885403-12156-1-git-send-email-b32955@freescale.com> <201308222229.01778.marex@denx.de> <20130822233611.GA25263@sirena.org.uk> Message-ID: <201308230158.05909.marex@denx.de> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Dear Mark Brown, > On Thu, Aug 22, 2013 at 10:29:01PM +0200, Marek Vasut wrote: > > > On Thu, Aug 22, 2013 at 12:34:53PM -0700, Brian Norris wrote: > > > > > > This should be a generic property for all quad devices to use, though, > > > since the same thing applies everywhere. > > > > Full ACK, "m25p,dual-read" and "m25p,quad-read" sound like a good prop > > names? > > I was actually thinking something more generic than that - putting the > property at the SPI generic bindings level. Though if all flashes with > this dual/quad read functionality have the prefix m25p the above would > work also, at the minute this does seem to be mostly used by flash (I > bet someone's got some DSPs or something though). Ah! So you mean the SPI controller would provide information that it can do dual/quad transfers? But then, the additional pins can only be wired to certain chips (controller by certain CS lines). Best regards, Marek Vasut