Re: [PATCH v4 34/34] wireless: mark prism54.ko as deprecated and mark for removal

From: "Luis R. Rodriguez" <lrodriguez@atheros.com>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: "Luis Rodriguez" <Luis.Rodriguez@Atheros.com>,
	"linville@tuxdriver.com" <linville@tuxdriver.com>,
	"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
	"Larry.Finger@lwfinger.net" <Larry.Finger@lwfinger.net>,
	"aquilaver@yahoo.com" <aquilaver@yahoo.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"Dan Williams" <dcbw@redhat.com>,
	"Kai Engert" <kengert@redhat.com>,
	"Jean Tourrilhes" <jt@hpl.hp.com>,
	"Tim de Waal" <tim.dewaal@yahoo.com>,
	"Roy Marples" <uberlord@gentoo.org>,
	"Alan Cox" <alan@lxorguk.ukuu.org.uk>,
	"Christian Lamparter" <chunkeey@web.de>,
	"Björn Steinbrink" <B.Steinbrink@gmx.de>,
	"Tim Gardner" <tim.gardner@canonical.com>
Subject: Re: [PATCH v4 34/34] wireless: mark prism54.ko as deprecated and mark for removal
Date: Mon, 17 Aug 2009 10:06:44 -0700	[thread overview]
Message-ID: <20090817170644.GE4352@mosca> (raw)
In-Reply-To: <1250323182.15069.0.camel@johannes.local>

On Sat, Aug 15, 2009 at 12:59:42AM -0700, Johannes Berg wrote:
> On Fri, 2009-08-14 at 16:25 -0400, Luis R. Rodriguez wrote:
> > The preferred modules is p54.ko, which also supports FullMAC
> > PCI / Cardbus devices. We schedule removal for 2.6.34. Reason
> > to remove this is no one really is testing prism54.ko anymore,
> > and while it works p54.ko provides support for the same hardware.
> > It should be noted I have been told some FullMAC devices may not
> > have worked with the SoftMAC driver but to date we have yet to
> > recieve a single bug report regarding this. If there are users
> > out there please let us know!
> 
> Not that I care about the changelog, but I do find it illogical to refer
> to modules w/o .ko everywhere and then with .ko in the changelog. Of
> course you were consistent ;)

Heh I've gone and cleaned this up in the series, I just removed the
.ko usage when I refer to modules in the entries and commit log.

> > -	  The module will be called prism54.
> > +	  When built as module you get prism54.ko
> 
> NAK this change.

Heh ok, changed it.

  Luis