From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1LqqgL-0008LA-2y for qemu-devel@nongnu.org; Mon, 06 Apr 2009 11:24:17 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1LqqgG-0008Do-9a for qemu-devel@nongnu.org; Mon, 06 Apr 2009 11:24:16 -0400 Received: from [199.232.76.173] (port=52569 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1LqqgF-0008DV-TW for qemu-devel@nongnu.org; Mon, 06 Apr 2009 11:24:12 -0400 Received: from main.gmane.org ([80.91.229.2]:42620 helo=ciao.gmane.org) by monty-python.gnu.org with esmtps (TLS-1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.60) (envelope-from ) id 1LqqgF-0000UY-Hs for qemu-devel@nongnu.org; Mon, 06 Apr 2009 11:24:11 -0400 Received: from list by ciao.gmane.org with local (Exim 4.43) id 1LqqgD-0003au-4B for qemu-devel@nongnu.org; Mon, 06 Apr 2009 15:24:09 +0000 Received: from h86-62-88-129.ln.rinet.ru ([86.62.88.129]) by main.gmane.org with esmtp (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Mon, 06 Apr 2009 15:24:09 +0000 Received: from vladimir by h86-62-88-129.ln.rinet.ru with local (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Mon, 06 Apr 2009 15:24:09 +0000 From: Vladimir Prus Date: Mon, 06 Apr 2009 19:23:57 +0400 Message-ID: References: <200904022129.02385.vladimir@codesourcery.com> <49DA0064.2070909@juno.dti.ne.jp> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7Bit Sender: news Subject: [Qemu-devel] Re: SH: support 7785 serial Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Shin-ichiro KAWASAKI wrote: > Hi, Vladimir. > Thank you for your work. > > Vladimir Prus wrote: >> These 3 patches make sh_serial.c support 7785 serial. The primary >> difference is that on 7785, instead of single fifo count register >> there are two separate ones -- for rx and tx. Patch 3 adds necessary >> conditional logic. Patches 1 and 2 are cleanups that I've done alone >> the way: >> >> - Patch 1 makes it possible to change the size of RX fifo. It's 16 >> on 7751 and 64 on 7785. While I do not know of any breakage if we >> use wrong size, it's still best to be correct. >> - Patch 2 replaces a pile of hardcoded constants with macroses. >> Also, the FSR register was handled strangely -- when written, we'd >> look at the written value and set bits in the 'flags' field, and on >> read, we'd reconstruct the value from the 'flags' field. There does >> not seem to be any reason for such roundabout, so I've made the code >> work with 'sr' directly. >> >> This patch was tested both with r2d, using kernel and userland found >> at: >> >> thttp://www.assembla.com/wiki/show/qemu-sh4/BuildingEnvironment >> >> and with 7785, using a hand-made kernel. > > Patch 2 produces a trouble in my environment. > For r2d, the output to SCIF from kernel is OK, but output from > shell is broken by inserted white space, like follows. > > (before applying patch 2) > # ls > > (after applying patch2) > # l s > > Do you have time to investigate it? Shin-ichiro, I could not reproduce this locally. Did you tried typing this in virtual console that QEMU pops up, or in the console where QEMU was started. For me, the former does not accept any input at all. Would it be possible for you to provide me with - Source tree of qemu that you've tested this with? Pointer to some git commit in some branch in some git repository will be OK. - The kernel zImage you have used it - The disk image you have used - The command line for qemu For the latter three, I have used what you have put at: http://www.assembla.com/wiki/show/qemu-sh4/BuildingEnvironment but I want to be sure there's no accidental differences. And for qemu source tree, I want to make sure I did not unintentially based by patch on some wrong state. Thanks, Volodya