From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:36497) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TuwXD-0007rO-M7 for qemu-devel@nongnu.org; Mon, 14 Jan 2013 21:45:56 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TuwXC-0007hy-OI for qemu-devel@nongnu.org; Mon, 14 Jan 2013 21:45:55 -0500 Received: from smtp4.mundo-r.com ([212.51.32.151]:22049) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TuwXC-0007gP-HF for qemu-devel@nongnu.org; Mon, 14 Jan 2013 21:45:54 -0500 Date: Tue, 15 Jan 2013 03:45:36 +0100 From: Alberto Garcia Message-ID: <20130115024536.GA3514@igalia.com> References: <50F07600.1070100@suse.de> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <50F07600.1070100@suse.de> Subject: Re: [Qemu-devel] [PATCH v5 1/2] Add TEWS TPCI200 IndustryPack emulation List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Andreas =?iso-8859-1?Q?F=E4rber?= Cc: Anthony Liguori , qemu-devel@nongnu.org On Fri, Jan 11, 2013 at 09:28:48PM +0100, Andreas Färber wrote: > I do wonder if adjust_value() can be replaced by switching > MemoryRegion endianness between DEVICE_LITTLE_ENDIAN and > DEVICE_BIG_ENDIAN rather than having it as DEVICE_NATIVE_ENDIAN and > doing a manual bswap16()? Avi had actually suggested to have two memory regions with different endianness and switch between them. I had taken a look back then and it didn't seem to make things any simpler due to how the TPCI200 works in big-endian mode. Berto