From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from perch.kroah.org (mail.kroah.org [69.55.234.183]) by ozlabs.org (Postfix) with ESMTP id 32094679E1 for ; Thu, 2 Jun 2005 09:52:11 +1000 (EST) Date: Wed, 1 Jun 2005 17:02:28 -0700 From: Greg KH To: Matt Porter Message-ID: <20050602000228.GB5678@kroah.com> References: <20050601110836.A16559@cox.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20050601110836.A16559@cox.net> Cc: akpm@osdl.org, torvalds@osdl.org, linux-kernel@vger.kernel.org, linuxppc-embedded@ozlabs.org Subject: Re: [PATCH][1/3] RapidIO support: core List-Id: Linux on Embedded PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Wed, Jun 01, 2005 at 11:08:36AM -0700, Matt Porter wrote: > Patch is 108KB and can be found here: > ftp://source.mvista.com/pub/rio/l26_rio_core.patch register_driver() does not return the number of devices bound to the driver. So your comment in rio_register_driver() is incorrect. Just return count. Hm, if the patch was inline it would be easier to comment on stuff like this, I'll wait till then for the rest :) thanks, greg k-h