From patchwork Thu Nov 9 18:10:04 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ladislav Michl X-Patchwork-Id: 10051617 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id D955560631 for ; Thu, 9 Nov 2017 18:10:09 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id CB27F2928D for ; Thu, 9 Nov 2017 18:10:09 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id BFECF2AF04; Thu, 9 Nov 2017 18:10:09 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.9 required=2.0 tests=BAYES_00, RCVD_IN_DNSWL_HI, UNPARSEABLE_RELAY autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 122772928D for ; Thu, 9 Nov 2017 18:10:08 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752549AbdKISKH (ORCPT ); Thu, 9 Nov 2017 13:10:07 -0500 Received: from eddie.linux-mips.org ([148.251.95.138]:57094 "EHLO cvs.linux-mips.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751878AbdKISKH (ORCPT ); Thu, 9 Nov 2017 13:10:07 -0500 Received: (from localhost user: 'ladis' uid#1021 fake: STDIN (ladis@eddie.linux-mips.org)) by eddie.linux-mips.org id S23990506AbdKISKFDLwZS (ORCPT ); Thu, 9 Nov 2017 19:10:05 +0100 Date: Thu, 9 Nov 2017 19:10:04 +0100 From: Ladislav Michl To: Tony Lindgren Cc: linux-mtd@lists.infradead.org, linux-omap@vger.kernel.org, Roger Quadros , Boris Brezillon , Kyungmin Park Subject: Re: [PATCH v3 1/7] memory: omap-gpmc: Refactor OneNAND support Message-ID: <20171109181004.g55vx4iveo5sulpt@lenoch> References: <20171109091155.6a6azfvjarwvlfh2@lenoch> <20171109091253.lvrzi5kbaykkxlcb@lenoch> <20171109175626.GD28152@atomide.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20171109175626.GD28152@atomide.com> User-Agent: NeoMutt/20170113 (1.7.2) Sender: linux-omap-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-omap@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP On Thu, Nov 09, 2017 at 09:56:26AM -0800, Tony Lindgren wrote: > Hi, > > * Ladislav Michl [171109 09:14]: > > Use generic probe function to deal with OneNAND node and remove now useless > > gpmc_probe_onenand_child function. > > Import sync mode timing calculation function from mach-omap2/gpmc-onenand.c > > and prepare for MTD driver DTfication. > > I tried giving this series a try on n900, but looks like onenand is no longer > seen on n900 after this first patch. This first patch makes original driver stop working as it removes special OneNAND handling. If it doesn't work even after applying whole serie, then: - verify onenand node has compatible 'ti,omap2-onenand' property - verify timings On IGEPv2 bootlog shows: [ 1.544464] omap-gpmc 6e000000.gpmc: GPMC revision 5.0 [ 1.550415] gpmc_mem_init: disabling cs 0 mapped at 0x0-0x1000000 [ 1.560485] omap2-onenand 30000000.onenand: initializing on CS0, phys base 0x30000000, virtual base e0080000 [ 1.571014] Muxed OneNAND(DDP) 512MB 1.8V 16-bit (0x58) [ 1.576507] OneNAND version = 0x0031 [ 1.583435] Scanning device for bad blocks [ 1.620971] OneNAND eraseblock 597 is an initial bad block [ 1.657470] OneNAND eraseblock 1159 is an initial bad block [ 1.754577] OneNAND eraseblock 2812 is an initial bad block [ 1.832214] omap2-onenand 30000000.onenand: optimized timings for 83 MHz [ 1.842620] 2 ofpart partitions found on MTD device 30000000.onenand [ 1.849426] Creating 2 MTD partitions on "30000000.onenand": [ 1.855651] 0x000000000000-0x000000080000 : "SPL" [ 1.863464] 0x000000080000-0x000020000000 : "UBI" For a start: --- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/arch/arm/boot/dts/omap3-n900.dts b/arch/arm/boot/dts/omap3-n900.dts index 4acd32a1c4ef..aa5b1a439564 100644 --- a/arch/arm/boot/dts/omap3-n900.dts +++ b/arch/arm/boot/dts/omap3-n900.dts @@ -838,6 +838,7 @@ onenand@0,0 { #address-cells = <1>; #size-cells = <1>; + compatible = "ti,omap2-onenand"; reg = <0 0 0x20000>; /* CS0, offset 0, IO size 128K */ gpmc,sync-read;