Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757450AbcJXLr4 (ORCPT ); Mon, 24 Oct 2016 07:47:56 -0400 Received: from mail-out.m-online.net ([212.18.0.10]:43713 "EHLO mail-out.m-online.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756858AbcJXLrx (ORCPT ); Mon, 24 Oct 2016 07:47:53 -0400 X-Auth-Info: wk37HhbaxFX8jnNQykZJ0l23anTIFjiggx7igzRg5a8= Subject: Re: [PATCH v2 0/9] mtd: spi-nor: parse SFDP tables to setup (Q)SPI memories To: Jagan Teki References: Cc: Cyrille Pitchen , Brian Norris , "linux-mtd@lists.infradead.org" , nicolas.ferre@atmel.com, boris.brezillon@free-electrons.com, "linux-kernel@vger.kernel.org" From: Marek Vasut Message-ID: Date: Mon, 24 Oct 2016 13:41:49 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Icedove/45.4.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1920 Lines: 45 On 10/24/2016 09:41 AM, Jagan Teki wrote: > On Sun, Oct 23, 2016 at 2:03 AM, Marek Vasut wrote: >> On 10/22/2016 01:00 PM, Jagan Teki wrote: >>> On Wed, Oct 5, 2016 at 5:30 PM, Cyrille Pitchen >>> wrote: >>>> Hi all, >>>> >>>> This series extends support of SPI protocols to new protocols such as >>>> SPI x-2-2 and SPI x-4-4. Also spi_nor_scan() tries now to select the right >>>> op codes, timing parameters (number of mode and dummy cycles) and erase >>>> sector size by parsing the Serial Flash Discoverable Parameter (SFDP) >>>> tables, when available, as defined in the JEDEC JESD216 specifications. >>>> >>>> When SFDP tables are not available, legacy settings are still used for >>>> backward compatibility (SPI and earlier QSPI memories). >>>> >>>> Support of SPI memories >128Mbits is also improved by using the 4byte >>>> address instruction set, when available. Using those dedicated op codes >>>> is stateless as opposed to enter the 4byte address mode, hence a better >>>> compatibility with some boot loaders which expect to use 3byte address >>>> op codes. >>> >>> The memories which are > 128Mbits should have 4-bytes addressing >>> support based on my experience, do you think BAR is also required >>> atleast from spi-nor side? >> >> Yes, I believe BAR is still required for broken/dumb flash chips. >> Not all chips > 16 MiB support dedicated 4-byte addressing opcodes :-( > > Do you have list for those broken chips? No. > because I never find any > chips which has > 16 MiB with not support of 4-byte address opcodes > and I've seen the controller has dependable with BAR though it can > access > 16MiB ex: zynq qspi/ IIRC some of the 32 MiB micron chips were pretty flaky in that aspect, they used the same opcodes for 4byte access as for 3byte access and their behavior was switched by writing some non-volatile bit. -- Best regards, Marek Vasut