From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mtiwmhc11.worldnet.att.net ([204.127.131.115]:48107 "EHLO mtiwmhc11.worldnet.att.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932395AbXCCVEP (ORCPT ); Sat, 3 Mar 2007 16:04:15 -0500 Message-ID: <45E9E2BD.9070106@lwfinger.net> Date: Sat, 03 Mar 2007 15:03:57 -0600 From: Larry Finger MIME-Version: 1.0 To: Michael Buesch CC: Sam Ravnborg , linux-wireless@vger.kernel.org, LKML , Andrew Morton Subject: Re: [PATCH] bcm43xx: do not rebuild when kernel version changes References: <20070303190328.GA30463@uranus.ravnborg.org> <200703032121.43136.mb@bu3sch.de> In-Reply-To: <200703032121.43136.mb@bu3sch.de> Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-wireless-owner@vger.kernel.org List-ID: Michael Buesch wrote: > On Saturday 03 March 2007 20:03, Sam Ravnborg wrote: >> Replacing use of UTS_RELEASE with utsname()->release >> avoids that this module is rebuild each >> time the kernel version changes. >> >> Compile tested only. > > I vote to get rid of the whole bcm43xx_ethtool.c file, as it's useless. > It will vanish with the advent of d80211, anyway. > Larry? Is bcm43xx_ethtool.c really useless? If so, I vote to remove it. I have never tried to use ethtool on bcm43xx. Similarly, it isn't very often that I recompile the kernel when bcm43xx hasn't changed and would not have noticed the bug that Sam reported. Larry