From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43)
	id 1MbetU-0006fp-78
	for qemu-devel@nongnu.org; Thu, 13 Aug 2009 14:19:20 -0400
Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43)
	id 1MbetT-0006ey-QD
	for qemu-devel@nongnu.org; Thu, 13 Aug 2009 14:19:19 -0400
Received: from [199.232.76.173] (port=51544 helo=monty-python.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1MbetT-0006ec-Fs
	for qemu-devel@nongnu.org; Thu, 13 Aug 2009 14:19:19 -0400
Received: from fg-out-1718.google.com ([72.14.220.155]:29645)
	by monty-python.gnu.org with esmtp (Exim 4.60)
	(envelope-from <blauwirbel@gmail.com>) id 1MbetT-0007A0-7M
	for qemu-devel@nongnu.org; Thu, 13 Aug 2009 14:19:19 -0400
Received: by fg-out-1718.google.com with SMTP id d23so253698fga.8
	for <qemu-devel@nongnu.org>; Thu, 13 Aug 2009 11:19:18 -0700 (PDT)
MIME-Version: 1.0
In-Reply-To: <4A7F7E91.5000903@earthlink.net>
References: <4A7F7E91.5000903@earthlink.net>
From: Blue Swirl <blauwirbel@gmail.com>
Date: Thu, 13 Aug 2009 21:18:58 +0300
Message-ID: <f43fc5580908131118q3a5ec9dcm4195b6f6b8d2c90c@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Subject: [Qemu-devel] Re: sparc sun4m changes
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/pipermail/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Robert Reif <reif@earthlink.net>
Cc: qemu-devel@nongnu.org

On Mon, Aug 10, 2009 at 4:57 AM, Robert Reif<reif@earthlink.net> wrote:
> I just took a look at the sun4m interrupt controller and noticed
> the recent changes. =C2=A0There are at least 3 different interrupt
> controllers used by sun4m: ss600mp with VME and MBUS
> specific support, ss 10/20 with MBUS support and ss 4/5/lx
> with slavio support. =C2=A0This doesn't even address the java stations.
>
> intbit_to_level in slavio_intctrl.c needs to be different for each
> controller chip used. =C2=A0There also needs to be a controller specific
> interrupt mask used.
>
> The NCR slavio chip is only used for single processor systems
> (ss 4/5/lx). =C2=A0It is a subset of the ss 10/20 which is a subset of th=
e
> ss600mp. =C2=A0We really should address these differences.

I think the differences should be supported with a "model" or
"version" property rather than the intbit table, which didn't fit the
qdev model well.