From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from top.free-electrons.com ([176.31.233.9] helo=mail.free-electrons.com) by merlin.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1VWOvJ-0004gM-Mb for linux-mtd@lists.infradead.org; Wed, 16 Oct 2013 11:05:54 +0000 Date: Wed, 16 Oct 2013 08:05:39 -0300 From: Ezequiel Garcia To: Brian Norris Subject: Re: [PATCH 0/2] pxa3xx: Data buffer dynamic allocation Message-ID: <20131016110538.GB2518@localhost> References: <1380911438-24240-1-git-send-email-ezequiel.garcia@free-electrons.com> <20131015204741.GW23337@ld-irv-0074.broadcom.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20131015204741.GW23337@ld-irv-0074.broadcom.com> Cc: Thomas Petazzoni , Lior Amsalem , Tawfik Bayouk , Daniel Mack , linux-mtd@lists.infradead.org, Gregory Clement List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Tue, Oct 15, 2013 at 01:47:41PM -0700, Brian Norris wrote: > Hi Ezequiel, > > On Fri, Oct 04, 2013 at 03:30:36PM -0300, Ezequiel Garcia wrote: > > As part of my work to support the NAND controllers in Armada 370/XP SoC > > here's a tiny set of patches to replace the currently data buffer allocation > > based in a hardcoded buffer size, into a dynamic scheme. > > > > We first use kmalloc to request a 256 bytes (big enough for the ONFI paramater > > page) and then re-allocate (either with kmalloc or using DMA allocators) using > > the detected page size. > > > > The first patch is required to perform the initial NAND operations (READID > > and friends) using programmable I/O since the DMA buffers are not ready > > at such early stage. > > > > The second patch performs to actual dynamic allocation and it's exactly > > the same patch I sent a few weeks ago: > > > > http://patchwork.ozlabs.org/patch/275982/ > > > > Daniel: Can you test these two If they look fine we can add them now, > > being an improvement to the driver pxa3xx beyond the Armada 370/XP effort. > > Thanks for sending these out separately from your other series. They are > independently useful and look good to me. And thanks, Daniel, for the > testing. > > Pushed to l2-mtd.git! > Great, thanks! -- Ezequiel GarcĂ­a, Free Electrons Embedded Linux, Kernel and Android Engineering http://free-electrons.com