Subject: Re: [PATCH] [MTD] CHIPS: oops in cfi_amdstd_sync
From: Josh Boyer <jwboyer@linux.vnet.ibm.com>
To: =?ISO-8859-1?Q?J=F6rn?= Engel <joern@lazybastard.org>
Cc: Vijay Sampath <vsampath@gmail.com>, linux-mtd@lists.infradead.org,
       linux-kernel@vger.kernel.org
In-Reply-To: <20070221145420.GD464@lazybastard.org>
References: <758a2bbf0702201746p3959235ele3db9b600a9f8104@mail.gmail.com>
	 <20070221145420.GD464@lazybastard.org>
Content-Type: text/plain; charset=utf-8
Date: Wed, 21 Feb 2007 09:18:40 -0600
Message-Id: <1172071120.24204.167.camel@zod.rchland.ibm.com>
Mime-Version: 1.0
Content-Transfer-Encoding: 8bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 901
Lines: 22

On Wed, 2007-02-21 at 14:54 +0000, Jörn Engel wrote:
> On Tue, 20 February 2007 17:46:13 -0800, Vijay Sampath wrote:
> > 
> > The files cfi_cmdset_0002.c and cfi_cmdset_0020.c do not initialize
> > their wait queues like is done in cfi_cmdset_0001.c. This causes an
> > oops when the wait queue is accessed. I have copied the code from
> > cfi_cmdset_0001.c that is pertinent to initialization of the wait
> > queue.
> 
> Patch looks good, but I can no longer test it.  Josh may still have
> access to some commandset 20 chips.  Josh, any objections?

The patch looks good to me as well.  No access to those chips anymore,
but I have no objections.

josh

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/