Re: [PATCH] p54: fix SoftLED compile dependencies

[John W. Linville]

On Sat, Mar 21, 2009 at 01:22:01PM +0100, Christian Lamparter wrote:
> On Saturday 21 March 2009 07:36:33 Kalle Valo wrote:
> > Christian Lamparter <chunkeey@xxxxxx> writes:
> > 
> > > This patch fixes a compile problem when the MAC80211_LEDS
> > > triggers are enabled but not LED class itself. 
> > > (which is sort of pointless, but anyway...)
> > 
> > [...]
> > 
> > > @@ -17,31 +18,6 @@ config P54_USB
> > >  	select CRC32
> > >  	---help---
> > >  	  This driver is for USB isl38xx based wireless cards.
> > > -	  These are USB based adapters found in devices such as:
> > > -
> > > -	  3COM 3CRWE254G72
> > > -	  SMC 2862W-G
> > > -	  Accton 802.11g WN4501 USB
> > > -	  Siemens Gigaset USB
> > > -	  Netgear WG121
> > > -	  Netgear WG111
> > > -	  Medion 40900, Roper Europe
> > > -	  Shuttle PN15, Airvast WM168g, IOGear GWU513
> > > -	  Linksys WUSB54G
> > > -	  Linksys WUSB54G Portable
> > > -	  DLink DWL-G120 Spinnaker
> > > -	  DLink DWL-G122
> > > -	  Belkin F5D7050 ver 1000
> > > -	  Cohiba Proto board
> > > -	  SMC 2862W-G version 2
> > > -	  U.S. Robotics U5 802.11g Adapter
> > > -	  FUJITSU E-5400 USB D1700
> > > -	  Sagem XG703A
> > > -	  DLink DWL-G120 Cohiba
> > > -	  Spinnaker Proto board
> > > -	  Linksys WUSB54AG
> > > -	  Inventel UR054G
> > > -	  Spinnaker DUT
> > 
> > This change wasn't mentioned in the commit log. Did this get included
> > by accident?
> > 
> heh true... should I add them back? after all this list hasn't changed in years
> and we have a much nicer one on http://www.prism54.org/
> ( http://wireless.kernel.org/en/users/Drivers/p54/devices )

Getting rid of them is fine, but it should probably be done in a different patch.

John
-- 
John W. Linville		Someday the world will need a hero, and you
linville@xxxxxxxxxxxxx			might be all we have.  Be ready.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/