From: "John W. Linville" <linville@tuxdriver.com>
To: Arend van Spriel <arend@broadcom.com>
Cc: linux-wireless <linux-wireless@vger.kernel.org>
Subject: Re: [PATCH 3/4] brcmsmac: extend hardware info shown in debugfs
Date: Tue, 25 Nov 2014 13:50:49 -0500 [thread overview]
Message-ID: <20141125185049.GE15476@tuxdriver.com> (raw)
In-Reply-To: <54747D76.6070207@broadcom.com>
On Tue, Nov 25, 2014 at 02:00:38PM +0100, Arend van Spriel wrote:
> On 11/25/14 13:12, Arend van Spriel wrote:
> >On 11/24/14 22:50, John W. Linville wrote:
> >>Hmmm...I'm not sure why, but this is causing problems for me...
> >>
> >>ERROR: "brcmu_boardrev_str"
> >>[drivers/net/wireless/brcm80211/brcmsmac/brcmsmac.ko] undefined!
> >
> >I am confused as well. I did not have such an issue. The function is
> >provided by brcmutil.ko (patch 2/4) and I did an EXPORT_SYMBOL and added
> >it to the include file. Missing something? Also I don't see this issue
> >over here. I will do a clean build on the wireless-next tree and see if
> >I can reproduce this.
>
> So I did a fresh checkout of wireless-next/master and applied patch 2 and 3.
> Here is my build output:
>
> $ make M=drivers/net/wireless/brcm80211 modules
> CC [M] drivers/net/wireless/brcm80211/brcmsmac/mac80211_if.o
> CC [M] drivers/net/wireless/brcm80211/brcmsmac/ucode_loader.o
> CC [M] drivers/net/wireless/brcm80211/brcmsmac/ampdu.o
> CC [M] drivers/net/wireless/brcm80211/brcmsmac/antsel.o
> CC [M] drivers/net/wireless/brcm80211/brcmsmac/channel.o
> CC [M] drivers/net/wireless/brcm80211/brcmsmac/main.o
> CC [M] drivers/net/wireless/brcm80211/brcmsmac/phy_shim.o
> CC [M] drivers/net/wireless/brcm80211/brcmsmac/pmu.o
> CC [M] drivers/net/wireless/brcm80211/brcmsmac/rate.o
> CC [M] drivers/net/wireless/brcm80211/brcmsmac/stf.o
> CC [M] drivers/net/wireless/brcm80211/brcmsmac/aiutils.o
> CC [M] drivers/net/wireless/brcm80211/brcmsmac/phy/phy_cmn.o
> CC [M] drivers/net/wireless/brcm80211/brcmsmac/phy/phy_lcn.o
> CC [M] drivers/net/wireless/brcm80211/brcmsmac/phy/phy_n.o
> CC [M] drivers/net/wireless/brcm80211/brcmsmac/phy/phytbl_lcn.o
> CC [M] drivers/net/wireless/brcm80211/brcmsmac/phy/phytbl_n.o
> CC [M] drivers/net/wireless/brcm80211/brcmsmac/phy/phy_qmath.o
> CC [M] drivers/net/wireless/brcm80211/brcmsmac/dma.o
> CC [M] drivers/net/wireless/brcm80211/brcmsmac/brcms_trace_events.o
> CC [M] drivers/net/wireless/brcm80211/brcmsmac/debug.o
> LD [M] drivers/net/wireless/brcm80211/brcmsmac/brcmsmac.o
> CC [M] drivers/net/wireless/brcm80211/brcmutil/utils.o
> CC [M] drivers/net/wireless/brcm80211/brcmutil/d11.o
> LD [M] drivers/net/wireless/brcm80211/brcmutil/brcmutil.o
> Building modules, stage 2.
> MODPOST 2 modules
> CC drivers/net/wireless/brcm80211/brcmsmac/brcmsmac.mod.o
> LD [M] drivers/net/wireless/brcm80211/brcmsmac/brcmsmac.ko
> CC drivers/net/wireless/brcm80211/brcmutil/brcmutil.mod.o
> LD [M] drivers/net/wireless/brcm80211/brcmutil/brcmutil.ko
>
> Anything else I should try over here?
/home/linville/git/wireless-testing
[linville-x1.hq.tuxdriver.com]:> make M=drivers/net/wireless/brcm80211 modules
Building modules, stage 2.
MODPOST 3 modules
WARNING: "brcmu_boardrev_str" [drivers/net/wireless/brcm80211/brcmsmac/brcmsmac.ko] undefined!
CC drivers/net/wireless/brcm80211/brcmfmac/brcmfmac.mod.o
LD [M] drivers/net/wireless/brcm80211/brcmfmac/brcmfmac.ko
CC drivers/net/wireless/brcm80211/brcmsmac/brcmsmac.mod.o
LD [M] drivers/net/wireless/brcm80211/brcmsmac/brcmsmac.ko
CC drivers/net/wireless/brcm80211/brcmutil/brcmutil.mod.o
LD [M] drivers/net/wireless/brcm80211/brcmutil/brcmutil.ko
--
John W. Linville Someday the world will need a hero, and you
linville@tuxdriver.com might be all we have. Be ready.
next prev parent reply other threads:[~2014-11-25 19:00 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-11-20 21:26 [PATCH 0/4] brcm80211: minor rework Arend van Spriel
2014-11-20 21:26 ` [PATCH 1/4] brcmfmac: fix static checker warning in pmklist handling Arend van Spriel
2014-11-20 21:27 ` [PATCH 2/4] brcmutil: add helper function to format board revision Arend van Spriel
2014-11-24 21:53 ` John W. Linville
2014-11-25 10:01 ` Arend van Spriel
2014-11-25 18:51 ` John W. Linville
2014-11-20 21:27 ` [PATCH 3/4] brcmsmac: extend hardware info shown in debugfs Arend van Spriel
2014-11-24 21:50 ` John W. Linville
2014-11-25 12:12 ` Arend van Spriel
2014-11-25 13:00 ` Arend van Spriel
2014-11-25 18:50 ` John W. Linville [this message]
2014-11-25 18:49 ` John W. Linville
2014-11-25 19:05 ` John W. Linville
2014-11-25 19:16 ` Arend van Spriel
2014-11-25 19:20 ` John W. Linville
2014-11-25 19:49 ` Arend van Spriel
2014-11-25 19:23 ` Arend van Spriel
2014-11-25 19:28 ` Arend van Spriel
2014-11-20 21:27 ` [PATCH 4/4] brcmfmac: correct .disconnect() callback while connecting Arend van Spriel
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20141125185049.GE15476@tuxdriver.com \
--to=linville@tuxdriver.com \
--cc=arend@broadcom.com \
--cc=linux-wireless@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.