From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 15634 invoked by alias); 27 Oct 2009 20:02:24 -0000 Received: (qmail 15480 invoked by uid 22791); 27 Oct 2009 20:02:23 -0000 X-SWARE-Spam-Status: No, hits=-2.2 required=5.0 tests=AWL,BAYES_00 X-Spam-Check-By: sourceware.org Received: from smtp-vbr7.xs4all.nl (HELO smtp-vbr7.xs4all.nl) (194.109.24.27) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 27 Oct 2009 20:02:18 +0000 Received: from [192.168.1.66] (cust.7.108.adsl.cistron.nl [82.95.157.21]) by smtp-vbr7.xs4all.nl (8.13.8/8.13.8) with ESMTP id n9RK29H8063736; Tue, 27 Oct 2009 21:02:15 +0100 (CET) (envelope-from rutger@cs.vu.nl) Message-ID: <4AE75206.1080504@cs.vu.nl> Date: Tue, 27 Oct 2009 20:02:00 -0000 From: Rutger Hofman User-Agent: Thunderbird 2.0.0.23 (X11/20090817) MIME-Version: 1.0 To: Jonathan Larmour CC: eCos developers Subject: Re: NAND technical review References: <4AC6218C.20407@jifvik.org> <4ACB4B58.2040804@ecoscentric.com> <4ACC0722.9020601@jifvik.org> <4ACCC13F.40009@cs.vu.nl> <4AD69BBE.6070103@jifvik.org> <4AD73386.4030300@cs.vu.nl> <4AD7CD29.1050701@jifvik.org> <4ADC777F.4020506@cs.vu.nl> <4ADD2CAB.4010000@jifvik.org> <4ADDAC7A.1070206@cs.vu.nl> <4ADE679D.1050900@jifvik.org> <4ADEFCFE.9060603@cs.vu.nl> <4AE1B864.1040409@jifvik.org> <4AE1CAD0.4080206@cs.vu.nl> In-Reply-To: <4AE1CAD0.4080206@cs.vu.nl> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit X-IsSubscribed: yes Mailing-List: contact ecos-devel-help@ecos.sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Post: List-Help: , Sender: ecos-devel-owner@ecos.sourceware.org X-SW-Source: 2009-10/txt/msg00065.txt.bz2 Rutger Hofman wrote: > Jonathan Larmour wrote: >> Rutger Hofman wrote: >>> Jonathan Larmour wrote: >>> >>>> Rutger Hofman wrote: > [on adding support for other NAND chips than raw NAND] >> >>> I guess that this refactoring will take something like one or a few >>> days' work, including having ANC call the controller over a dispatch >>> table. I'll be glad to do it (ETA: somewhere in the next 1 to 1.5 >>> months). >> >> I would be very surprised by a day! > > Yesterday, there was an unexpected lull in the usual storm of work. > Basically, the refactoring is done so R can support hardware other than > raw NAND. I must still update the documentation, though. The structure > is a bit different than I first thought; there is a package IO_NAND for > the general stuff (anc, ecc, bbt), and a package IO_NAND_RAW for the raw > NAND. So, if somebody wants NAND but not raw NAND, that package isn't > included so no raw NAND code. > > I will put up a next release when the documentation is done. I am aware > that changing the code while you are reviewing it is not very polite; > maybe you prefer to stick with the code that you have right now, and > just acknowledge any updates/refactoring I did. The documentation has been overhauled, too. The new NAND layout, with a package for Common NAND besides a package for raw NAND, is published on http://www.cs.vu.nl/~rutger/software/ecos/nand-flash . The Common NAND should be able to harbour other kinds of NAND than raw NAND. I fully understand if you will not want to review the changed code again. But maybe you are willing to quickly browse through the documentation to see if it looks OK at first glance. Rutger