From owner-cvs-all Mon Nov 22 6:55:52 1999 Delivered-To: cvs-all@freebsd.org Received: from ns.skylink.it (ns.skylink.it [194.177.113.1]) by hub.freebsd.org (Postfix) with ESMTP id 436A4158DE; Mon, 22 Nov 1999 06:55:35 -0800 (PST) (envelope-from n_hibma@skylink.it) Received: from skylink.it (va-166.skylink.it [194.185.55.166]) by ns.skylink.it (8.9.1/8.8.8) with ESMTP id PAA13878; Mon, 22 Nov 1999 15:55:32 +0100 Received: from localhost (localhost [127.0.0.1]) by skylink.it (8.9.3/8.9.3) with ESMTP id PAA66850; Mon, 22 Nov 1999 15:53:48 +0100 (CET) (envelope-from n_hibma@skylink.it) Date: Mon, 22 Nov 1999 15:53:48 +0100 (CET) From: Nick Hibma X-Sender: n_hibma@henny.jrc.it Reply-To: Nick Hibma To: Doug Rabson Cc: cvs-committers@freebsd.org, cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/pci uhci_pci.c ohci_pci.c In-Reply-To: Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk What you mean to say is that, the ones I kicked out were resource handling and unnecessary because usb\d+ does not require any specific resources. In that case, I think you are wrong. It's fine were it is. A bus handles resources for its children and it was just the fact that usb\d+ does not need any resources at all. Spread the methods out too thinly and people will get lost in what they need. At the moment they can simply walk through bus_if.m and pick the ones they need. Nick On Mon, 22 Nov 1999, Doug Rabson wrote: > On Sun, 21 Nov 1999, Nick Hibma wrote: > > > n_hibma 1999/11/21 19:22:44 PST > > > > Modified files: > > sys/pci uhci_pci.c ohci_pci.c > > Log: > > Remove some bogus bus methods peter added. We are hardly doing > > anything as a bus. > > In retrospect, all the resource handling methods in bus_if.m should have > been in a new interface specifically for devices which manage resources on > behalf of their children. > > -- > Doug Rabson Mail: dfr@nlsystems.com > Nonlinear Systems Ltd. Phone: +44 181 442 9037 > > To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message