From mboxrd@z Thu Jan 1 00:00:00 1970 From: Grant Likely Subject: Re: [RFC PATCH 06/17] phy_driver: Add and use phy_driver_[un]register_multiple() Date: Tue, 25 Oct 2011 07:00:03 +0200 Message-ID: <20111025050003.GC798@ponder.secretlab.ca> References: <1319144425-15547-1-git-send-email-Kyle.D.Moffett@boeing.com> <1319144425-15547-7-git-send-email-Kyle.D.Moffett@boeing.com> <201110201715.58441.vapier@gentoo.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Kyle Moffett , linux-kernel@vger.kernel.org, netdev@vger.kernel.org, "David S. Miller" , Greg Dietsche , Giuseppe Cavallaro , David Daney , Arnaud Patard , Baruch Siach , Stephen Hemminger , Lucas De Marchi , Marc Kleine-Budde , Andrew Morton To: Mike Frysinger Return-path: Content-Disposition: inline In-Reply-To: <201110201715.58441.vapier@gentoo.org> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Thu, Oct 20, 2011 at 05:15:54PM -0400, Mike Frysinger wrote: > On Thursday 20 October 2011 17:00:13 Kyle Moffett wrote: > > + return phy_driver_register_multiple(bcm63xx_drivers, > > + ARRAY_SIZE(bcm63xx_drivers)); > > if only the arm ARRAY_AND_SIZE() macro could gain traction in the wider tree: > return phy_driver_register_multiple(ARRAY_AND_SIZE(bcm63xx_drivers)); Propose a patch to move it from arch/arm to include/linux/kernel.h. This is the first I've heard of this macro, but I would probably use it. > ignoring that, this patch looks cool > > Acked-by: Mike Frysinger Indeed, I like the diffstat. :-) Acked-by: Grant Likely