Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933466AbbLRAle (ORCPT ); Thu, 17 Dec 2015 19:41:34 -0500 Received: from mail-pf0-f170.google.com ([209.85.192.170]:33068 "EHLO mail-pf0-f170.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753512AbbLRAlc (ORCPT ); Thu, 17 Dec 2015 19:41:32 -0500 Date: Thu, 17 Dec 2015 16:41:29 -0800 From: Brian Norris To: Bean Huo =?utf-8?B?6ZyN5paM5paMIChiZWFuaHVvKQ==?= Cc: Cyrille Pitchen , "linux-mtd@lists.infradead.org" , "nicolas.ferre@atmel.com" , "marex@denx.de" , "vigneshr@ti.com" , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "devicetree@vger.kernel.org" , "robh+dt@kernel.org" , "pawel.moll@arm.com" , "mark.rutland@arm.com" , "ijc+devicetree@hellion.org.uk" , "galak@codeaurora.org" Subject: Re: [PATCH linux-next 0/5] mtd: spi-nor: add driver for Atmel QSPI controller Message-ID: <20151218004129.GF10460@google.com> References: <20151207193441.GO120110@google.com> <20151218002901.GE10460@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20151218002901.GE10460@google.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1503 Lines: 35 On Thu, Dec 17, 2015 at 04:29:01PM -0800, Brian Norris wrote: > On Tue, Dec 08, 2015 at 06:21:00AM +0000, Bean Huo 霍斌斌 wrote: > OK, so I think your patch is broken: > > > > commit 548cd3ab54da ("mtd: spi-nor: Add quad I/O support for Micron > > > SPI NOR") > > How did you test this? Specifically, this can't possibly have worked > with a regular drivers/spi/ controllers, since: > > (a) you're enabling 4/4/4 (i.e., "Quad mode") on the flash but > (b) m25p80_read() only sets .rx_bits for the data; i.e., it's using > 1/1/4 (i.e., "Extended mode") > > I'm tempted to essentially revert that, as it looks essentially > untested. It would be nice to have a cleaner baseline before trying to > extend it with Cyrille's work. > > Cyrille, what do you think? Is my analysis at all correct here? (Sorry > if this is addressed elsewhere; there's a lot of text in this > conversation, but I'm getting hung up very early.) And if so, does it > hurt to just drop Micron "Quad mode" (4/4/4)? It looks like you address some of this in patch 2, where you (as I do) claim that Micron support is broken. It seems to me that it could be better to kill it than to try to fix it. But maybe that's just the frustrated maintainer in me speaking. Brian -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/