From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([209.51.188.92]:48811) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gjlbg-0004W4-Ev for qemu-devel@nongnu.org; Wed, 16 Jan 2019 08:51:49 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gjlbc-0003k0-Qu for qemu-devel@nongnu.org; Wed, 16 Jan 2019 08:51:46 -0500 Received: from mx1.redhat.com ([209.132.183.28]:35036) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1gjlba-0003bu-Nf for qemu-devel@nongnu.org; Wed, 16 Jan 2019 08:51:44 -0500 From: Thomas Huth References: <20190115141108.934-1-yang.zhong@intel.com> <20190115141108.934-25-yang.zhong@intel.com> <4fe7900d-13c9-348e-3e14-89dc80251c3e@redhat.com> Message-ID: <310f0d5d-c2e3-c46c-1d06-4fb2df486303@redhat.com> Date: Wed, 16 Jan 2019 14:51:37 +0100 MIME-Version: 1.0 In-Reply-To: <4fe7900d-13c9-348e-3e14-89dc80251c3e@redhat.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [RFC PATCH v2 24/37] build: convert sound.mak to Kconfig List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Yang Zhong , qemu-devel@nongnu.org Cc: peter.maydell@linaro.org, sameo@linux.intel.com, ehabkost@redhat.com, pbonzini@redhat.com On 2019-01-16 14:48, Thomas Huth wrote: > On 2019-01-15 15:10, Yang Zhong wrote: >> From: Paolo Bonzini >> >> There is really nothing special in these devices; they are just >> ISA devices. Instead of including them for each target, >> set CONFIG_ISA to true, and make the devices default to present >> whenever ISA is available. More conversion of ISA devices will >> follow. > [...] >> diff --git a/default-configs/ppc-softmmu.mak b/default-configs/ppc-softmmu.mak >> index 301cc83279..c7003e4284 100644 >> --- a/default-configs/ppc-softmmu.mak >> +++ b/default-configs/ppc-softmmu.mak >> @@ -1,8 +1,8 @@ >> # Default configuration for ppc-softmmu >> >> -include sound.mak >> include usb.mak >> CONFIG_PCI=y >> +ONFIG_ISA=y > > Typo: CONFIG_ISA please Or should it even be CONFIG_ISA_BUS ? At least that's the name of the switch that you use in the other patches ... Thomas