From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1LiCB3-00059g-4a for qemu-devel@nongnu.org; Fri, 13 Mar 2009 14:32:13 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1LiCAy-00053p-Bo for qemu-devel@nongnu.org; Fri, 13 Mar 2009 14:32:12 -0400 Received: from [199.232.76.173] (port=33287 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1LiCAy-00053O-3s for qemu-devel@nongnu.org; Fri, 13 Mar 2009 14:32:08 -0400 Received: from mx20.gnu.org ([199.232.41.8]:63207) by monty-python.gnu.org with esmtps (TLS-1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.60) (envelope-from ) id 1LiCAw-0005Jo-Dm for qemu-devel@nongnu.org; Fri, 13 Mar 2009 14:32:06 -0400 Received: from mail.codesourcery.com ([65.74.133.4]) by mx20.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1LiCAu-0004Wo-IS for qemu-devel@nongnu.org; Fri, 13 Mar 2009 14:32:04 -0400 From: Vladimir Prus Subject: Re: [Qemu-devel] SH: Improve the interrupt controller Date: Fri, 13 Mar 2009 21:32:15 +0300 References: <200812112252.17620.vladimir@codesourcery.com> <200902192257.58689.vladimir@codesourcery.com> <200903131750.n2DHoMDi019632@smtp09.dti.ne.jp> In-Reply-To: <200903131750.n2DHoMDi019632@smtp09.dti.ne.jp> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Message-Id: <200903132132.15945.vladimir@codesourcery.com> Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: takasi-y@ops.dti.ne.jp Cc: qemu-devel@nongnu.org On Friday 13 March 2009 20:50:22 takasi-y@ops.dti.ne.jp wrote: > Hi, > > > On the CPU, INT2MSKR is initialized to all zeros, not masking anything. > > In QEMU, > > Please read HW manual. According to a SH7785 HW manual(REJ09B0261-0100), > Initial value of four mask registers are > {ff000000,ff000000,00000000,ffffffff}. There's no general rule for initial > value of registers. > But default = 0 not seems to be a bad idea. > Attached patch enables to supply initial value, otherwise 0. > > Tested for little endian r2d on PC with source below > svn head: svn://svn.sv.gnu.org/qemu/trunk@6838 > + --append patch : Sun, 8 Mar 2009 03:00:17 +0900 (JST) > + intc old patch : Mon, 9 Feb 2009 03:57:25 +0900 (JST) > + intc last patch : Wed, 18 Feb 2009 03:32:15 +0900 (JST) > + This patch. > > BTW, our(mostly "my", sorry) slow conversation seems to upset some people. > I don't care writing code without your source code, Hmm, maybe I am confused but I had an impression that you do have access to the sh4a qemu -- both binary and source. > because my reference > is always a HW manual. But, it is true that this style of development is > not efficient nor interesting. Indeed, mailing patches and revisions back and forth is cumbersome. If the above set of patches works for you for r2d, then maybe the best approach is to get them checked in -- and then I'll have a baseline to revise my patch series against? I assume that core qemu maintainers are not watching this thread closely. Do you think you can post the above patches -- either combined, or separately, for review? Thanks, Volodya