From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Morton Subject: Re: [PATCH 2/3] device.h drivers/base/core.c Convert dev_ macros to functions Date: Thu, 4 Mar 2010 23:29:28 -0800 Message-ID: <20100304232928.2e45bdd1.akpm@linux-foundation.org> References: <20100304231047.4d76e0ef.akpm@linux-foundation.org> <1267773815.2136.10.camel@Joe-Laptop.home> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: Linus Torvalds , Greg Kroah-Hartman , linux-kernel@vger.kernel.org, netdev@vger.kernel.org To: Joe Perches Return-path: Received: from smtp1.linux-foundation.org ([140.211.169.13]:33101 "EHLO smtp1.linux-foundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754038Ab0CEH3g (ORCPT ); Fri, 5 Mar 2010 02:29:36 -0500 In-Reply-To: <1267773815.2136.10.camel@Joe-Laptop.home> Sender: netdev-owner@vger.kernel.org List-ID: On Thu, 04 Mar 2010 23:23:35 -0800 Joe Perches wrote: > On Thu, 2010-03-04 at 23:10 -0800, Andrew Morton wrote: > > On Thu, 4 Mar 2010 22:56:53 -0800 Joe Perches wrote: > > > Perhaps over time the struct and local uses of dev_info should be renamed. > > aww, c'mon. > > y:/usr/src/linux-2.6.33> grep -r '\*dev_info;' . | wc -l > > 30 > > No doubt I could submit all the required changes in > an hour or so, but some of the maintainers seem > less than open to what they consider "churn". Well, that has to be one of the worst possible reasons? > $ grep -rP --include=*.[ch] -l "dev_info\b\s*[^\(]" * > arch/powerpc/platforms/iseries/dt.c > drivers/usb/host/xhci-dbg.c > drivers/usb/host/xhci.h > drivers/usb/wusbcore/wusbhc.h > drivers/scsi/scsi_devinfo.c > drivers/staging/comedi/drivers/das08_cs.c > drivers/staging/comedi/drivers/ni_daq_dio24.c > drivers/staging/comedi/drivers/quatech_daqp_cs.c > drivers/staging/comedi/drivers/ni_daq_700.c > drivers/staging/comedi/drivers/ni_labpc_cs.c > drivers/staging/comedi/drivers/cb_das16_cs.c > drivers/staging/udlfb/udlfb.c > drivers/media/video/sh_mobile_ceu_camera.c > drivers/md/linear.h > drivers/net/ksz884x.c > drivers/net/bnx2x_link.c > drivers/net/wireless/wl3501_cs.c > drivers/net/cnic.c > drivers/net/bnx2x_main.c > include/linux/sfi.h > include/net/bluetooth/rfcomm.h Send 'em over, please. I'll merge any stragglers directly.