Bug 477927 - at76 is upstream in 2.6.28, patch linux-2.6-at76.patch thus superfluous
Summary: at76 is upstream in 2.6.28, patch linux-2.6-at76.patch thus superfluous
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: kernel
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: John W. Linville
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
: 484850 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-12-25 17:25 UTC by Thorsten Leemhuis
Modified: 2009-03-05 22:27 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-03-05 22:27:41 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Thorsten Leemhuis 2008-12-25 17:25:12 UTC
Quoting from kernel.spec:
> Patch690: linux-2.6-at76.patch
[...]
> # Add misc wireless bits from upstream wireless tree
> ApplyPatch linux-2.6-at76.patch
[...]
>* Fri Aug 01 2008 John W. Linville <linville>
>- Revert at76_usb to version from before attempted mac80211 port

This patch is unneeded, as at76 is upstream these days -- it's part of the staging tree/area that was merged into 2.6.28. The sources are apart from two small changes even identical with the ones we use

Version-Release number of selected component (if applicable):
kernel-2.6.28-1.fc11.x86_64

Suggested solution:
Remove patch and use the driver from the staging tree; that way the kernel will also get tainted as soon as the driver gets loaded (that's what upstream does and it's likely the best to follow that example)

Comment 1 John W. Linville 2009-01-05 14:37:58 UTC
Seems reasonable, except that there is virtually no guarantee that the staging version will remain working or that anyone will fix it when it breaks.  C'est la vie?

Also, I don't know what (if any) policy we have or need w.r.t. enabling staging drivers?

Comment 2 Thorsten Leemhuis 2009-01-05 19:36:14 UTC
(In reply to comment #1)
> Seems reasonable, except that there is virtually no guarantee that the staging
> version will remain working or that anyone will fix it when it breaks.  C'est
> la vie?

Well, the same afaics applies to the driver we include as patch right now, as it nearly the same code base ;-)

So maybe the real solution to for the at76-usb driver is to drop it completely from Fedora...

> Also, I don't know what (if any) policy we have or need w.r.t. enabling staging
> drivers?

That was one of the reasons why I opened this bug. But the decisions is likely up to Davej, Cebbert and Kylem.

Comment 3 John W. Linville 2009-01-05 20:43:59 UTC
One difference is that the staging driver will eventually go back to the (broken) mac80211 port of the driver, whereas the version in fedora will remain where it is until we have a working alternative.  As for dropping it, that might be a fine alternative provided you don't mind hurting those users.

Comment 4 Thorsten Leemhuis 2009-01-08 06:21:32 UTC
*sigh*

Seems
http://thread.gmane.org/gmane.linux.kernel.wireless.general/26045/focus=26487
was to late.

http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=02227c28391b5059a7710d6039c52912b0ee2c1d

>    Staging: at76_usb: update drivers/staging/at76_usb w/ mac80211 port
>    
>    This updates drivers/staging/at76_usb to correspond with the latest
>    version previously available in wireless-testing.

Which makes this bug mostly irrelevant afaics :-/

Comment 5 John W. Linville 2009-01-08 14:17:46 UTC
Crap, confusion...that commit reflects what I was hinting at in comment 3.  Of course, I thought the thread you quoted meant that we all agreed not to commit that -- Greg must have not connected the two points... :-(

Comment 6 Thorsten Leemhuis 2009-02-10 06:25:14 UTC
at76 change was reverted afaics:

http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=89cb7e7fd6c0917bb9236ea48bf538d4668ed009

So we could again do now what was suggested in the initial comment of this bug: drop our private at76_usb copy that we patch in and instead use the one from the staging tree

Comment 7 John W. Linville 2009-02-10 15:04:31 UTC
*** Bug 484850 has been marked as a duplicate of this bug. ***

Comment 8 John W. Linville 2009-02-11 20:18:42 UTC
Change checked-in to rawhide.


Note You need to log in before you can comment on or make changes to this bug.