From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:33051)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <afaerber@suse.de>) id 1VE0OR-0005W4-65
	for qemu-devel@nongnu.org; Mon, 26 Aug 2013 13:16:01 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <afaerber@suse.de>) id 1VE0OJ-0006ae-NB
	for qemu-devel@nongnu.org; Mon, 26 Aug 2013 13:15:55 -0400
Received: from cantor2.suse.de ([195.135.220.15]:33284 helo=mx2.suse.de)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <afaerber@suse.de>) id 1VE0OJ-0006ZZ-Bj
	for qemu-devel@nongnu.org; Mon, 26 Aug 2013 13:15:47 -0400
Message-ID: <521B8D3E.4080106@suse.de>
Date: Mon, 26 Aug 2013 19:15:42 +0200
From: =?UTF-8?B?QW5kcmVhcyBGw6RyYmVy?= <afaerber@suse.de>
MIME-Version: 1.0
References: <1377471536-12423-1-git-send-email-akoskovacs@gmx.com>
	<1377471536-12423-22-git-send-email-akoskovacs@gmx.com>
	<521B316E.6070703@redhat.com>
In-Reply-To: <521B316E.6070703@redhat.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Subject: Re: [Qemu-devel] [PATCH 21/47] hw/char/Kconfig: Add Kconfig file
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: =?UTF-8?B?w4Frb3MgS292w6Fjcw==?= <akoskovacs@gmx.com>, Alberto Garcia <agarcia@igalia.com>, qemu-devel@nongnu.org

Am 26.08.2013 12:43, schrieb Paolo Bonzini:
> Il 26/08/2013 00:58, =C3=81kos Kov=C3=A1cs ha scritto:
>> Signed-off-by: =C3=81kos Kov=C3=A1cs <akoskovacs@gmx.com>
>> ---
>>  hw/char/Kconfig |   24 ++++++++++++++++++++++++
>>  1 file changed, 24 insertions(+)
>>  create mode 100644 hw/char/Kconfig
>>
>> diff --git a/hw/char/Kconfig b/hw/char/Kconfig
>> new file mode 100644
>> index 0000000..7ad0bd3
>> --- /dev/null
>> +++ b/hw/char/Kconfig
>> @@ -0,0 +1,24 @@
>> +config IPACK=20
>> +    bool
>> +    depends on PCI
>=20
> PCI devices are generally configurable, so you need to add prompts to t=
hem.

IndustryPack is really misplaced in hw/char/ and I believe I posted
patches to remedy that and let one actually find it in our source tree.
There were no objections against hw/ipack/, alternatively it could go
into hw/gpio/. (Currently my patch series is waiting to be respun due to
changed QOM realize requirements from Anthony.)

That having being said, IndustryPack does not depend on PCI, only the
TPCI2000(?) PCI-IndustryPack bridge does.

Andreas

>=20
>> +config ESCC
>> +    bool
>> +
>> +config PARALLEL
>> +    bool
>> +    #select ISA
>=20
> Also "depends on" here.  You probably also want to make all these ISA
> devices configurable, so add a prompt.
>=20
> Paolo
>=20
>> +config SERIAL
>> +    bool
>> +    #select ISA
>> +
>> +config SERIAL_PCI
>> +    bool
>> +    depends on PCI
>> +
>> +config SCLPCONSOLE
>> +    bool
>> +
>> +config PL011
>> +    bool
>>
>=20
>=20


--=20
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N=C3=BCrnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imend=C3=B6rffer; HRB 16746 AG N=C3=BC=
rnberg