From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1MgFMt-0003if-9R for qemu-devel@nongnu.org; Wed, 26 Aug 2009 06:04:39 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1MgFMo-0003fn-BC for qemu-devel@nongnu.org; Wed, 26 Aug 2009 06:04:38 -0400 Received: from [199.232.76.173] (port=56506 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1MgFMn-0003fc-Vy for qemu-devel@nongnu.org; Wed, 26 Aug 2009 06:04:34 -0400 Received: from mx1.redhat.com ([209.132.183.28]:40424) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1MgFMn-0002Wz-3L for qemu-devel@nongnu.org; Wed, 26 Aug 2009 06:04:33 -0400 Received: from int-mx05.intmail.prod.int.phx2.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.18]) by mx1.redhat.com (8.13.8/8.13.8) with ESMTP id n7QA4Vop020616 for ; Wed, 26 Aug 2009 06:04:31 -0400 Message-ID: <4A9508AC.5070205@redhat.com> Date: Wed, 26 Aug 2009 12:04:28 +0200 From: Gerd Hoffmann MIME-Version: 1.0 Subject: Re: [Qemu-devel] [PATCH] Compile usb_ohci only for targets that need it References: <1251224190-2423-1-git-send-email-quintela@redhat.com> In-Reply-To: <1251224190-2423-1-git-send-email-quintela@redhat.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Juan Quintela Cc: qemu-devel@nongnu.org On 08/25/09 20:16, Juan Quintela wrote: > It is only needed for arm, ppc and sh4. Ahem. It works just fine on a pc too. And once qdev-ified you can even add it via -device. While it probably makes sense to not build it by default because we use uhci for pc I certainly want to have the option to include it. What is the long-term plan this stuff? I think it isn't useful to hard-code all this in configure. How about having a target-$arch/default-config file with 'CONFIG_FOO=y' lines? Also look for (and prefer if present) a target-$arch/user-config file? cheers, Gerd