Bug 759109 - kernel-3.1.2-1.fc16 build failure on PPC64, drivers/staging/brcm80211/brcmsmac/ampdu.c
Summary: kernel-3.1.2-1.fc16 build failure on PPC64, drivers/staging/brcm80211/brcmsma...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: kernel
Version: 16
Hardware: powerpc
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Josh Boyer
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-12-01 12:53 UTC by Karsten Hopp
Modified: 2011-12-22 01:18 UTC (History)
7 users (show)

Fixed In Version: kernel-3.1.5-1.fc16
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-12-12 21:57:33 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Karsten Hopp 2011-12-01 12:53:49 UTC
Description of problem:
drivers/staging/brcm80211/brcmsmac/ampdu.c: In function 'brcms_c_ampdu_dotxstatus':
drivers/staging/brcm80211/brcmsmac/ampdu.c:833:17: error: invalid operands to binary ^ (have 'volatile u32 *' and 'int')
drivers/staging/brcm80211/brcmsmac/ampdu.c:833:17: error: invalid operands to binary ^ (have 'volatile u32 *' and 'int')
drivers/staging/brcm80211/brcmsmac/ampdu.c:841:8: error: invalid operands to binary ^ (have 'volatile u32 *' and 'int')
drivers/staging/brcm80211/brcmsmac/ampdu.c:841:8: error: invalid operands to binary ^ (have 'volatile u32 *' and 'int')


Version-Release number of selected component (if applicable):
kernel-3.1.2-1.fc16 

How reproducible:
always

Steps to Reproduce:
1. ppc-koji build --scratch dist-f16 kernel-3.1.2-1.fc16.src.rpm
2.
3.
  
Actual results:
http://ppc.koji.fedoraproject.org/koji/getfile?taskID=322550&name=build.log

Comment 1 Josh Boyer 2011-12-01 13:58:45 UTC
So.  This is because we supposedly have backported brcm80211 drivers from 3.2-rc2 in brcm80211.patch.  Which we obviously do.  That backport contains commit:

commit b60987a15628046259be17471fd80ba92cf35ed2
Author: Roland Vossen <rvossen>
Date:   Mon Aug 8 15:58:49 2011 +0200

    staging: brcm80211: simplified register access macro's in softmac
    

which should fix the build error.

However, the brcm80211 drivers are still sitting in drivers/staging/ as well in f15/f16.  For some reason, those are getting picked up during the ppc build.

Looking in a local x86_64 build, they're also being built there but not installed:

[jwboyer@host brcmsmac]$ pwd
/ssd/jwboyer/kernel/kernel-3.1.fc16/linux-3.1.x86_64/drivers/staging/brcm80211/brcmsmac
[jwboyer@ihatethathostname brcmsmac]$ ls
aiutils.c  antsel.h    d11.h          Makefile       phy_shim.h  srom.c
aiutils.h  antsel.o    dma.c          modules.order  phy_shim.o  srom.h
aiutils.o  bmac.c      dma.h          nicpci.c       pmu.c       srom.o
alloc.c    bmac.h      dma.o          nicpci.h       pmu.h       stf.c
alloc.h    bmac.o      mac80211_if.c  nicpci.o       pmu.o       stf.h
alloc.o    brcmsmac.o  mac80211_if.h  otp.c          pub.h       stf.o
ampdu.c    built-in.o  mac80211_if.o  otp.h          rate.c      types.h
ampdu.h    channel.c   main.c         otp.o          rate.h      ucode_loader.c
ampdu.o    channel.h   main.h         phy            rate.o      ucode_loader.h
antsel.c   channel.o   main.o         phy_shim.c     scb.h       ucode_loader.o
[jwboyer@host brcmsmac]$

John, unless you say otherwise I'm going to add the removal of the Kconfig and Makefile in drivers/staging/brcm80211/* to brcm80211.patch on f15/f16.

Comment 2 John W. Linville 2011-12-01 14:40:39 UTC
Sure, that seems reasonable -- sorry I didn't think about that!

Comment 3 Josh Boyer 2011-12-01 15:06:23 UTC
Done and committed to git for f15/f16.

Comment 4 Fedora Update System 2011-12-09 20:23:45 UTC
kernel-3.1.5-1.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/kernel-3.1.5-1.fc16

Comment 5 Harald Reindl 2011-12-10 00:14:02 UTC
> * Thu Dec 01 2011 Josh Boyer <jwboyer> 
> - Apply patch to revert mac80211 scan optimizations (rhbz #731365) 
> - Disable the existing brcm80211 staging drivers (rhbz #759109)

does this mean WLAN stops working again with 2.6.41.5-1.fc15?
one of my co-workers was so happy to get his WLAN supported with the last kernel

Comment 6 Josh Boyer 2011-12-10 02:12:59 UTC
(In reply to comment #5)
> > * Thu Dec 01 2011 Josh Boyer <jwboyer> 
> > - Apply patch to revert mac80211 scan optimizations (rhbz #731365) 
> > - Disable the existing brcm80211 staging drivers (rhbz #759109)
> 
> does this mean WLAN stops working again with 2.6.41.5-1.fc15?
> one of my co-workers was so happy to get his WLAN supported with the last
> kernel

No, not at all.  There were two separate copies of the brcm80211 drivers in the kernel tree.  One existed in the drivers/staging directory and the other is a backport of a newer version of those drivers that live in drivers/net/wireless.  We removed the now stale drivers/staging copy of the drivers (really just the build files).

The brcm80211 drivers are still built and still included in the kernel.  You can see them listed in koji here:

http://koji.fedoraproject.org/koji/rpminfo?fileStart=1600&rpmID=2826611&fileOrder=name&buildrootOrder=-id&buildrootStart=0#filelist

Comment 7 Harald Reindl 2011-12-10 02:45:14 UTC
thank you for your reply!

i thought if it goes away i should prepare him sicne he needs WLAn at home and is glad that currently no kmod-packages are needed!

BTW: 2.6.41.5-1.fc15 is running smooth on my HP Compaq 8200 Elite CMT PC

Comment 8 Fedora Update System 2011-12-10 19:34:00 UTC
Package kernel-3.1.5-1.fc16:
* should fix your issue,
* was pushed to the Fedora 16 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing kernel-3.1.5-1.fc16'
as soon as you are able to, then reboot.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2011-16840/kernel-3.1.5-1.fc16
then log in and leave karma (feedback).

Comment 9 Fedora Update System 2011-12-12 12:20:59 UTC
kernel-2.6.41.5-1.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/kernel-2.6.41.5-1.fc15

Comment 10 Fedora Update System 2011-12-12 21:57:33 UTC
kernel-3.1.5-1.fc16 has been pushed to the Fedora 16 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 11 Fedora Update System 2011-12-15 18:54:43 UTC
kernel-2.6.41.5-4.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/kernel-2.6.41.5-4.fc15

Comment 12 Fedora Update System 2011-12-22 01:18:31 UTC
kernel-2.6.41.6-1.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/kernel-2.6.41.6-1.fc15


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