From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:53287) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TTEXH-0005fz-7S for qemu-devel@nongnu.org; Tue, 30 Oct 2012 12:19:36 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TTEXB-00071V-9V for qemu-devel@nongnu.org; Tue, 30 Oct 2012 12:19:27 -0400 Received: from mx1.redhat.com ([209.132.183.28]:13885) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TTEXA-00071O-VT for qemu-devel@nongnu.org; Tue, 30 Oct 2012 12:19:21 -0400 Message-ID: <508FFDF5.6070901@redhat.com> Date: Tue, 30 Oct 2012 17:19:01 +0100 From: Gerd Hoffmann MIME-Version: 1.0 References: <66b81e2e5ffe78ac446fe4ba0a28e748595b7bf6.1351561225.git.jbaron@redhat.com> <508F74F2.4060204@redhat.com> <20121030151923.GB2744@redhat.com> In-Reply-To: <20121030151923.GB2744@redhat.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v1 12/13] q35: fill in usb pci slots with -usb List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Jason Baron Cc: aliguori@us.ibm.com, juzhang@redhat.com, mst@redhat.com, jan.kiszka@siemens.com, qemu-devel@nongnu.org, agraf@suse.de, blauwirbel@gmail.com, yamahata@valinux.co.jp, alex.williamson@redhat.com, kevin@koconnor.net, avi@redhat.com, gsomlo@gmail.com, mkletzan@redhat.com, pbonzini@redhat.com, lcapitulino@redhat.com, afaerber@suse.de, armbru@redhat.com On 10/30/12 16:19, Jason Baron wrote: > On Tue, Oct 30, 2012 at 07:34:26AM +0100, Gerd Hoffmann wrote: >> Hi, >> >>> + uhci_devname[sizeof(uhci_devname) - 2] = ((char)'1') + i; >> >> snprintf(devname, sizeof(devname), "...%d", i) is more readable. > > ok. > >> >>> + qdev_prop_set_string(usb_qdev, "masterbus", "ich9-usb-bus.0"); >> >> Any reason why you rename the usb bus? >> > > I wasn't sure if the user created usb devices on the command-line via > -device if that would break naming here. Thus, I added a 'private' name. > If the naming is stable, that works. It would be 'usb-bus.0', in that > case? "usb.0" would be the default name, but you don't need to know it, you can just look up what qdev created. See here: http://www.kraxel.org/cgit/qemu/commit/?h=rebase/usb-next&id=70b9867011c4793787c5acee3d2005a6bc951f59 [ This is part of the "usb patch queue" patch series posted today, depending on how the qom discussions go and how fast it goes in you might just call the function the patch provides. Or do something simliar in pc_q35.c and I'll drop the patch. ] -usb for -M pc creates a "usb.0" bus too, so I don't expect trouble. cheers, Gerd