From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:48726) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YDKOi-0005gr-Dt for qemu-devel@nongnu.org; Mon, 19 Jan 2015 17:02:13 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YDKOf-0000Ve-6v for qemu-devel@nongnu.org; Mon, 19 Jan 2015 17:02:12 -0500 Message-ID: <54BD7E35.40108@reactos.org> Date: Mon, 19 Jan 2015 22:59:17 +0100 From: =?UTF-8?B?SGVydsOpIFBvdXNzaW5lYXU=?= MIME-Version: 1.0 References: <1421667328-11800-1-git-send-email-mark.cave-ayland@ilande.co.uk> In-Reply-To: <1421667328-11800-1-git-send-email-mark.cave-ayland@ilande.co.uk> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH 0/2] m48t59: add year offset and MMIO ISA mapping List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Mark Cave-Ayland , qemu-devel@nongnu.org, qemu-ppc@nongnu.org, agraf@suse.de, afaerber@suse.de, atar4qemu@gmail.com Le 19/01/2015 12:35, Mark Cave-Ayland a =C3=A9crit : > This patch lays the groundwork for switching sun4u over from ioport NVR= AM > access to MMIO NVRAM access. > > Patch 1 introduces a new year_offset property which is the offset betwe= en the > year value stored in hardware and the actual year. In particular, Sun h= ardware > has a 68 year offset used to extend the date range of the IC. While the > kernel sources I have viewed contain this offset within a #ifdef CONFIG= _SPARC > block, I've updated all the callers so that no change in behaviour will= be > seen across all platforms. PPC users may wish to alter the callers to c= hange > this behaviour as required. > > Patch 2 mimics the mem_base parameter from m48t59_init() to m48t59_init= _isa() > so that MMIO can be used for sun4u where the NVRAM is attached to the e= bus > which is essentially the same as an ISA bus. I've a patch which QOM'ifies m48t59, that I'll send to the list. If you apply it, you'll be then able to create a sysbus-m48t02 device, an= d then to add it to ebus memory region. IMO, there is no need to add a new parameter to m48t59_init_isa device. Tell me what do you think of it. Herv=C3=A9