From owner-freebsd-embedded@FreeBSD.ORG  Thu Mar 18 15:18:36 2010
Return-Path: <owner-freebsd-embedded@FreeBSD.ORG>
Delivered-To: embedded@freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 985A51065670
	for <embedded@freebsd.org>; Thu, 18 Mar 2010 15:18:36 +0000 (UTC)
	(envelope-from gjb@semihalf.com)
Received: from smtp.semihalf.com (smtp.semihalf.com [213.17.239.109])
	by mx1.freebsd.org (Postfix) with ESMTP id 417E78FC21
	for <embedded@freebsd.org>; Thu, 18 Mar 2010 15:18:36 +0000 (UTC)
Received: from localhost (unknown [213.17.239.109])
	by smtp.semihalf.com (Postfix) with ESMTP id 3B784C42E3;
	Thu, 18 Mar 2010 16:21:20 +0100 (CET)
X-Virus-Scanned: by amavisd-new at semihalf.com
Received: from smtp.semihalf.com ([213.17.239.109])
	by localhost (smtp.semihalf.com [213.17.239.109]) (amavisd-new,
	port 10024)
	with ESMTP id Ft4krRjVH43f; Thu, 18 Mar 2010 16:21:19 +0100 (CET)
Received: from [10.0.0.75] (cardhu.semihalf.com [213.17.239.108])
	by smtp.semihalf.com (Postfix) with ESMTPA id AAC2CC42DB;
	Thu, 18 Mar 2010 16:21:19 +0100 (CET)
Message-ID: <4BA24449.6090400@semihalf.com>
Date: Thu, 18 Mar 2010 16:18:33 +0100
From: Grzegorz Bernacki <gjb@semihalf.com>
User-Agent: Thunderbird 2.0.0.16 (X11/20090618)
MIME-Version: 1.0
To: Luiz Otavio O Souza <loos.br@gmail.com>
References: <0AE04EFA-A3EB-4939-BD81-607C00355B67@semihalf.com>	<20100314165825.121d346b@fubar.geek.nz>	<CC419602-A9E8-4FE2-A5A5-0BFBD8240EDD@gmail.com>	<4B9E1697.9090602@semihalf.com>
	<20100316101044.0401295e@fubar.geek.nz>
	<4B9F72BC.1050609@semihalf.com>
	<776ADF3A-4C68-486A-8640-DE0DD6B8874E@gmail.com>
	<4B9FB5B0.5000007@semihalf.com>
In-Reply-To: <4B9FB5B0.5000007@semihalf.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Cc: embedded@freebsd.org, Andrew Turner <andrew@fubar.geek.nz>
Subject: Re: NAND Flash Framework for review
X-BeenThere: freebsd-embedded@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: Dedicated and Embedded Systems <freebsd-embedded.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-embedded>, 
	<mailto:freebsd-embedded-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-embedded>
List-Post: <mailto:freebsd-embedded@freebsd.org>
List-Help: <mailto:freebsd-embedded-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-embedded>, 
	<mailto:freebsd-embedded-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Thu, 18 Mar 2010 15:18:36 -0000

Hi,

> 
> I will try to implement new mpc8572 driver and see how it looks. Maybe 
> I'm exaggerating a little bit.

ok, it seems that I was... I've implemented driver and it doesn't look so bad.
I just need to test it more.

> 
> Thanks for the explanation of ready/busy pin and for your patch. I will 
> review it and test it tomorrow and let you know if I have any comments.

But your patch looks good. I tested it and found no bugs. I just need to find
time to update nandsim code, cause it is affected by the patch (unless you did
it already). Then we will push all changes to perforce.

grzesiek