From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752688AbdLEIZI (ORCPT ); Tue, 5 Dec 2017 03:25:08 -0500 Received: from mail.linuxfoundation.org ([140.211.169.12]:53160 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750810AbdLEIZG (ORCPT ); Tue, 5 Dec 2017 03:25:06 -0500 Date: Tue, 5 Dec 2017 09:25:13 +0100 From: Greg Kroah-Hartman To: Finn Thain Cc: Geert Uytterhoeven , linux-m68k@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v3 14/14] nubus: Add support for the driver model Message-ID: <20171205082513.GA24114@kroah.com> References: <5c71aa72f8c6192ca472b103555cad7441974f71.1512454057.git.fthain@telegraphics.com.au> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <5c71aa72f8c6192ca472b103555cad7441974f71.1512454057.git.fthain@telegraphics.com.au> User-Agent: Mutt/1.9.1 (2017-09-22) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Dec 05, 2017 at 01:20:02AM -0500, Finn Thain wrote: > This patch brings basic support for the Linux Driver Model to the > NuBus subsystem. > > For flexibility, the matching of boards with drivers is left up to the > drivers. This is also the approach taken by NetBSD. A board may have > many functions, and drivers may have to consider many functional > resources and board resources in order to match a device. > > This implementation does not bind drivers to resources (nor does it bind > many drivers to the same board). Apple's NuBus declaration ROM design > is flexible enough to allow that, but I don't see a need to support it > as we don't use the "slot zero" resources (in the main logic board ROM). > > Eliminate the global nubus_boards linked list by rewriting the procfs > board iterator around bus_for_each_dev(). Hence the nubus device refcount > can be used to determine the lifespan of board objects. > > Cc: Greg Kroah-Hartman > Tested-by: Stan Johnson > Signed-off-by: Finn Thain Nice job: Reviewed-by: Greg Kroah-Hartman