From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.bootlin.com ([62.4.15.54]) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1fBdBU-000109-Oi for linux-mtd@lists.infradead.org; Thu, 26 Apr 2018 09:27:31 +0000 Date: Thu, 26 Apr 2018 11:27:11 +0200 From: Boris Brezillon To: Vignesh R Cc: David Woodhouse , Brian Norris , Marek Vasut , Richard Weinberger , Cyrille Pitchen , , Miquel Raynal , Mark Brown , , Peter Pan , Frieder Schrempf , Yogesh Gaur , =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= , Kamal Dasu , Maxime Chevallier Subject: Re: [PATCH v3 0/9] spi: Extend the framework to generically support memory devices Message-ID: <20180426112711.218fea51@bbrezillon> In-Reply-To: <354d5c48-be1b-0412-8cb0-f6e4daebccc5@ti.com> References: <20180422183522.11118-1-boris.brezillon@bootlin.com> <354d5c48-be1b-0412-8cb0-f6e4daebccc5@ti.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Thu, 26 Apr 2018 14:37:10 +0530 Vignesh R wrote: > On Monday 23 April 2018 12:05 AM, Boris Brezillon wrote: > > Hello, > > > > Shrinking a bit the explanation on why the spi-mem abstraction is > > needed (a detailed explanation is available here [2]). In addition to > > what as been said in my initial explanation I'll add that making it part > > of the SPI framework instead of as an extra independent layer is > > justified by the fact that some controllers support both SPI memory > > operations and regular SPI transfers, and it's cleaner to have both > > features exposed through a single driver. > > > > For those who want to have the full picture, here is a branch [1] > > containing the SPI NAND framework based on top of this spi-mem layer. > > > > Thanks, > > > > Boris > > > > [1]https://github.com/bbrezillon/linux/tree/spi-mem > > [2]https://www.spinics.net/lists/linux-spi/msg12058.html > > > > > > I am trying to apply this on top of linux-next, but > [PATCH v2 01/10] spi: Check presence the of ->transfer[_xxx]() before > registering a controller > from v2 is missing in this series and it does not seem to be in spi.git > as well? Hm, that's weird. I'm almost sure I received a notification saying it had been applied. I'll double check.