From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757896Ab2F0P1Z (ORCPT ); Wed, 27 Jun 2012 11:27:25 -0400 Received: from antcom.de ([188.40.178.216]:54415 "EHLO chuck.antcom.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754358Ab2F0PZd (ORCPT ); Wed, 27 Jun 2012 11:25:33 -0400 Message-ID: <4FEB25E8.5060706@antcom.de> Date: Wed, 27 Jun 2012 17:25:28 +0200 From: Roland Stigge Organization: ANTCOM IT Research & Development User-Agent: Mozilla/5.0 (X11; Linux i686 on x86_64; rv:10.0.5) Gecko/20120624 Icedove/10.0.5 MIME-Version: 1.0 To: Alexandre Pereira da Silva CC: dedekind1@gmail.com, linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, devicetree-discuss@lists.ozlabs.org, dwmw2@infradead.org, kevin.wells@nxp.com, srinivas.bakki@nxp.com, linux-arm-kernel@lists.infradead.org, hechtb@googlemail.com, lars@metafoo.de, b32955@freescale.com, leiwen@marvell.com, linux@arm.linux.org.uk Subject: Re: [PATCH v7] MTD: LPC32xx SLC NAND driver References: <1339064536-20553-1-git-send-email-stigge@antcom.de> <1340794007.29342.50.camel@sauron.fi.intel.com> <4FEAF90B.5030809@antcom.de> In-Reply-To: X-Enigmail-Version: 1.4 OpenPGP: url=subkeys.pgp.net Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi! On 06/27/2012 02:26 PM, Alexandre Pereira da Silva wrote: > I think it's best in case of *_raw to use nand base default functions, > like in the MLC patch. Problem here (specific for this SLC controller) is that it expects us to access the data register via 16bit access while the actual data is only 8bit wide. This can't be done with the default nand functions, neither with the 8bit nor 16bit variant. Roland PS: Nevertheless, I will post our SLC driver update series against the l2-mtd branch. From mboxrd@z Thu Jan 1 00:00:00 1970 From: Roland Stigge Subject: Re: [PATCH v7] MTD: LPC32xx SLC NAND driver Date: Wed, 27 Jun 2012 17:25:28 +0200 Message-ID: <4FEB25E8.5060706@antcom.de> References: <1339064536-20553-1-git-send-email-stigge@antcom.de> <1340794007.29342.50.camel@sauron.fi.intel.com> <4FEAF90B.5030809@antcom.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Sender: "devicetree-discuss" To: Alexandre Pereira da Silva Cc: srinivas.bakki-3arQi8VN3Tc@public.gmane.org, lars-Qo5EllUWu/uELgA04lAiVw@public.gmane.org, linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org, linux-doc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, leiwen-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org, dedekind1-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, kevin.wells-3arQi8VN3Tc@public.gmane.org, b32955-KZfg59tc24xl57MIdRCFDg@public.gmane.org, linux-mtd-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, hechtb-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org, dwmw2-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org List-Id: devicetree@vger.kernel.org Hi! On 06/27/2012 02:26 PM, Alexandre Pereira da Silva wrote: > I think it's best in case of *_raw to use nand base default functions, > like in the MLC patch. Problem here (specific for this SLC controller) is that it expects us to access the data register via 16bit access while the actual data is only 8bit wide. This can't be done with the default nand functions, neither with the 8bit nor 16bit variant. Roland PS: Nevertheless, I will post our SLC driver update series against the l2-mtd branch. From mboxrd@z Thu Jan 1 00:00:00 1970 Message-ID: <4FEB25E8.5060706@antcom.de> Date: Wed, 27 Jun 2012 17:25:28 +0200 From: Roland Stigge MIME-Version: 1.0 To: Alexandre Pereira da Silva Subject: Re: [PATCH v7] MTD: LPC32xx SLC NAND driver References: <1339064536-20553-1-git-send-email-stigge@antcom.de> <1340794007.29342.50.camel@sauron.fi.intel.com> <4FEAF90B.5030809@antcom.de> In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Cc: srinivas.bakki@nxp.com, lars@metafoo.de, linux@arm.linux.org.uk, linux-doc@vger.kernel.org, leiwen@marvell.com, devicetree-discuss@lists.ozlabs.org, dedekind1@gmail.com, linux-kernel@vger.kernel.org, kevin.wells@nxp.com, b32955@freescale.com, linux-mtd@lists.infradead.org, hechtb@googlemail.com, dwmw2@infradead.org, linux-arm-kernel@lists.infradead.org List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Hi! On 06/27/2012 02:26 PM, Alexandre Pereira da Silva wrote: > I think it's best in case of *_raw to use nand base default functions, > like in the MLC patch. Problem here (specific for this SLC controller) is that it expects us to access the data register via 16bit access while the actual data is only 8bit wide. This can't be done with the default nand functions, neither with the 8bit nor 16bit variant. Roland PS: Nevertheless, I will post our SLC driver update series against the l2-mtd branch. From mboxrd@z Thu Jan 1 00:00:00 1970 From: stigge@antcom.de (Roland Stigge) Date: Wed, 27 Jun 2012 17:25:28 +0200 Subject: [PATCH v7] MTD: LPC32xx SLC NAND driver In-Reply-To: References: <1339064536-20553-1-git-send-email-stigge@antcom.de> <1340794007.29342.50.camel@sauron.fi.intel.com> <4FEAF90B.5030809@antcom.de> Message-ID: <4FEB25E8.5060706@antcom.de> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi! On 06/27/2012 02:26 PM, Alexandre Pereira da Silva wrote: > I think it's best in case of *_raw to use nand base default functions, > like in the MLC patch. Problem here (specific for this SLC controller) is that it expects us to access the data register via 16bit access while the actual data is only 8bit wide. This can't be done with the default nand functions, neither with the 8bit nor 16bit variant. Roland PS: Nevertheless, I will post our SLC driver update series against the l2-mtd branch.