Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S935017AbaGXJ4t (ORCPT ); Thu, 24 Jul 2014 05:56:49 -0400 Received: from mail.aswsp.com ([193.34.35.150]:31839 "EHLO mail.aswsp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932600AbaGXJ4q convert rfc822-to-8bit (ORCPT ); Thu, 24 Jul 2014 05:56:46 -0400 Date: Thu, 24 Jul 2014 11:56:50 +0200 From: Matthieu CASTET To: Boris BREZILLON CC: David Woodhouse , Brian Norris , , Lee Jones , Subject: Re: [PATCH 1/2] mtd: nand: define struct nand_timings Message-ID: <20140724115650.46cbb3cf@parrot.com> In-Reply-To: <20140722141219.7f2c2890@bbrezillon> References: <1405064982-11456-1-git-send-email-boris.brezillon@free-electrons.com> <1405064982-11456-2-git-send-email-boris.brezillon@free-electrons.com> <20140722120346.0f2d0e77@parrot.com> <20140722141219.7f2c2890@bbrezillon> X-Mailer: Claws Mail 3.10.1 (GTK+ 2.24.24; i486-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-15" Content-Transfer-Encoding: 8BIT X-Originating-IP: [46.218.109.88] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Boris, Le Tue, 22 Jul 2014 14:12:19 +0200, Boris BREZILLON a ?crit : > Hi Matthieu > > On Tue, 22 Jul 2014 12:03:46 +0200 > Matthieu CASTET wrote: > > > Hi, > > > > > > > > I did a similar patch [1] (that wasn't merged :( ), and I used reduced > > timing info. > > I'm sorry it didn't make it to mainline, do you know why ? For the omap part there was a gpmc code rewrite that conflict with the patch. For the mtd stuff, I don't know/remember (I think for was no reply). > Could you point out where "reduced timing info" is defined in the ONFI > specification ? It is not defined on onfi. This was more a simplification of timings in order to simplify the driver side (avoid code duplication). Most controller allow to control nRE and nWE pulse and the setup time. Do you have drivers that use onfi timings ? > > > > > I also have support for the omap driver > > (http://article.gmane.org/gmane.linux.ports.arm.omap/88606/match=) and > > a controller we use in our chip (not upstream). > > It should be pretty easy to convert the full timings list into a > reduced one (actually, that's what your patch is doing), and you can > thus port your work on top of these patches. Yes I think an helper will be useful in order to help driver to use these timings. It can be a function that return the reduced version for a onfi mode and edo support. > > Anyway, I'm not sure what you have in mind, but unless there is a strong > reason to drop full timings in favor of reduced ones I'd like to > keep them (even if this implies adding a new converter to get reduced > timings list). > No problem. I have nothing special in mind. I hope this could give ideas how to use the onfi timings in mtd drivers (understanding how to use ONFI timings can be tricky). Matthieu -- 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/