From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul Brook Subject: Re: [Qemu-devel] [PATCH 3/4] LSI53C895A: Implement TARGET RESET message Date: Mon, 8 Dec 2008 23:36:33 +0000 Message-ID: <200812082336.34006.paul@codesourcery.com> References: <1228759670-31113-1-git-send-email-ryanh@us.ibm.com> <200812081838.23464.paul@codesourcery.com> <20081208185827.GL13481@us.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Cc: Ryan Harper , kvm@vger.kernel.org To: qemu-devel@nongnu.org Return-path: Received: from mail.codesourcery.com ([65.74.133.4]:39983 "EHLO mail.codesourcery.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753809AbYLHXgg (ORCPT ); Mon, 8 Dec 2008 18:36:36 -0500 In-Reply-To: <20081208185827.GL13481@us.ibm.com> Content-Disposition: inline Sender: kvm-owner@vger.kernel.org List-ID: > Which I figured was because lsi_soft_reset doesn't initialize > current_dma_len. I added current_dma_len to soft_reset and now we can > probe with out failing, and existing partitions on the device show up, > but any further use of the device results in broken behavior. >... > Because the driver is issueing bus resets, we're clobbering the scratch > registers. In that case I'm still very suspicious of your parch. Why are you clearing some of the scratch registers? It sounds like you need to separate a bus reset from a hard reset. Paul