From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753067AbbCEMzF (ORCPT ); Thu, 5 Mar 2015 07:55:05 -0500 Received: from down.free-electrons.com ([37.187.137.238]:57621 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751188AbbCEMzC (ORCPT ); Thu, 5 Mar 2015 07:55:02 -0500 Date: Thu, 5 Mar 2015 13:55:00 +0100 From: Antoine Tenart To: Thomas Petazzoni Cc: Antoine Tenart , sebastian.hesselbarth@gmail.com, ezequiel.garcia@free-electrons.com, dwmw2@infradead.org, computersforpeace@gmail.com, boris.brezillon@free-electrons.com, zmxu@marvell.com, jszhang@marvell.com, linux-arm-kernel@lists.infradead.org, linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v3 4/9] mtd: pxa3xx_nand: rework timings setup Message-ID: <20150305125500.GB5265@kwain> References: <1425555085-29531-1-git-send-email-antoine.tenart@free-electrons.com> <1425555085-29531-5-git-send-email-antoine.tenart@free-electrons.com> <20150305135023.45727776@free-electrons.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20150305135023.45727776@free-electrons.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Thomas, On Thu, Mar 05, 2015 at 01:50:23PM +0100, Thomas Petazzoni wrote: > On Thu, 5 Mar 2015 12:31:20 +0100, Antoine Tenart wrote: > > Use the nand framework helpers: onfi_get_async_timing_mode() and > > onfi_async_timing_mode_to_sdr_timings() to retrieve the timing > > configuration. Then update the pxa3xx timing setup function to use the > > timing configuration retrieved. > > > > Signed-off-by: Antoine Tenart > > --- > > drivers/mtd/nand/pxa3xx_nand.c | 101 ++++++++++++++++++-------- > > include/linux/platform_data/mtd-nand-pxa3xx.h | 2 - > > 2 files changed, 69 insertions(+), 34 deletions(-) > > After this commit, the pxa3xx_nand_timing structure type is no longer > used as far as I understand. So maybe it can be removed from > include/linux/platform_data/mtd-nand-pxa3xx.h ? Right, I'll remove it. Antoine -- Antoine Ténart, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com