From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752771AbbCHUYq (ORCPT ); Sun, 8 Mar 2015 16:24:46 -0400 Received: from down.free-electrons.com ([37.187.137.238]:45344 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752313AbbCHUYp (ORCPT ); Sun, 8 Mar 2015 16:24:45 -0400 Message-ID: <54FCAF6C.7010902@free-electrons.com> Date: Sun, 08 Mar 2015 17:22:04 -0300 From: Ezequiel Garcia User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0 MIME-Version: 1.0 To: Robert Jarzmik CC: Antoine Tenart , sebastian.hesselbarth@gmail.com, dwmw2@infradead.org, computersforpeace@gmail.com, thomas.petazzoni@free-electrons.com, boris.brezillon@free-electrons.com, linux-kernel@vger.kernel.org, zmxu@marvell.com, linux-mtd@lists.infradead.org, jszhang@marvell.com, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v3 5/9] mtd: pxa3xx_nand: add support for the Marvell Berlin nand controller References: <1425555085-29531-1-git-send-email-antoine.tenart@free-electrons.com> <1425555085-29531-6-git-send-email-antoine.tenart@free-electrons.com> <54FA6DFC.1070904@free-electrons.com> <87oao3xvh8.fsf@free.fr> In-Reply-To: <87oao3xvh8.fsf@free.fr> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 03/08/2015 02:14 PM, Robert Jarzmik wrote: > Ezequiel Garcia writes: > >> Hi Antoine, >> >> On 03/05/2015 08:31 AM, Antoine Tenart wrote: >> [..] >>> + >>> +static struct pxa3xx_nand_flash berlin_builtin_flash_types[] = { >>> +{ "4GiB 8-bit", 0xd7ec, 128, 8192, 8, 8, 4096 }, >>> +{ }, >> >> IMHO, supporting a specific flash shouldn't be part of this patch. >> >> In any case, why do you need this? If you can share the details about >> this device, it would be interesting for me to take a look. >> >> This driver's open-coded, legacy-style flash detection is nasty, and the >> only reason I've kept it is to avoid breaking some wacky user with some >> old board. In fact, maybe we can just kill it so nobody thinks it's sane. > > I think you'll kill the zylonite board, and I'll nack it if that's the case. At > least that's what happened when I tried to use onfi default values last time in > barebox development. > > I can test your changes, but if the specific zylonite nand (ie. nand id 0xba20, > ie. pxa310 embedded flash) gets broken, I'm against the removal of the legacy > timings removal. > I'm not speaking of any timing params here, but about the flash identification. Which flash do you have there? -- Ezequiel García, Free Electrons Embedded Linux, Kernel and Android Engineering http://free-electrons.com