Message ID | 20180803182942.4de98df2@canb.auug.org.au (mailing list archive) |
---|---|
State | RFC |
Delegated to: | Kalle Valo |
Headers | show |
Series | linux-next: build failure after merge of the wireless-drivers-next tree | expand |
Stephen Rothwell <sfr@canb.auug.org.au> writes: > After merging the wireless-drivers-next tree, today's linux-next build > (powerpc allyesconfig) failed like this: > > ld: drivers/net/wireless/mediatek/mt76/mt76x0/trace.o:(__tracepoints+0x0): multiple definition of `__tracepoint_set_shared_key'; drivers/net/wireless/mediatek/mt7601u/trace.o:(__tracepoints+0x0): first defined here [...] > Caused by commit > > 7b4859026ccd ("mt76x0: core files") > > and following ones. > > I have just marked that driver broken for now: Thanks Stephen for the report. Odd that kbuild bot didn't catch this, I did have these patches in my pending branch for several days. Stanislaw, could you please send a fix for this ASAP? I can't submit a pull request to Dave until this is fixed.
Hi On Fri, Aug 03, 2018 at 06:29:42PM +1000, Stephen Rothwell wrote: > After merging the wireless-drivers-next tree, today's linux-next build > (powerpc allyesconfig) failed like this: [snip] > diff --git a/drivers/net/wireless/mediatek/mt76/Kconfig b/drivers/net/wireless/mediatek/mt76/Kconfig > index 850611ad347a..ad56b3ed2be2 100644 > --- a/drivers/net/wireless/mediatek/mt76/Kconfig > +++ b/drivers/net/wireless/mediatek/mt76/Kconfig > @@ -13,6 +13,7 @@ config MT76x0U > tristate "MediaTek MT76x0U (USB) support" > depends on MAC80211 > depends on USB > + depends on BROKEN Would be better to limit driver to be build as module. Hopefully that will fix the issue. I'll post patch in a moment. Thanks Stanislaw
On Fri, Aug 03, 2018 at 11:11:40AM +0200, Stanislaw Gruszka wrote: > Hi > > On Fri, Aug 03, 2018 at 06:29:42PM +1000, Stephen Rothwell wrote: > > After merging the wireless-drivers-next tree, today's linux-next build > > (powerpc allyesconfig) failed like this: > [snip] > > diff --git a/drivers/net/wireless/mediatek/mt76/Kconfig b/drivers/net/wireless/mediatek/mt76/Kconfig > > index 850611ad347a..ad56b3ed2be2 100644 > > --- a/drivers/net/wireless/mediatek/mt76/Kconfig > > +++ b/drivers/net/wireless/mediatek/mt76/Kconfig > > @@ -13,6 +13,7 @@ config MT76x0U > > tristate "MediaTek MT76x0U (USB) support" > > depends on MAC80211 > > depends on USB > > + depends on BROKEN > Would be better to limit driver to be build as module. > Hopefully that will fix the issue. I'll post patch in a moment. Ok, I made config allyesconfig and .config file looks like this: CONFIG_WLAN_VENDOR_MEDIATEK=y CONFIG_MT7601U=y CONFIG_MT76_CORE=y CONFIG_MT76_USB=y CONFIG_MT76x2_COMMON=y CONFIG_MT76x0U=m However I still not sure if kernel will build this way. Because there can be conflict between kernel symbols and module symbols. So looks as part of quick fix mt7601u should be build also as module. And slower fix will be to rename symbosl names in mt76x0 driver. Regards Stanislaw
diff --git a/drivers/net/wireless/mediatek/mt76/Kconfig b/drivers/net/wireless/mediatek/mt76/Kconfig index 850611ad347a..ad56b3ed2be2 100644 --- a/drivers/net/wireless/mediatek/mt76/Kconfig +++ b/drivers/net/wireless/mediatek/mt76/Kconfig @@ -13,6 +13,7 @@ config MT76x0U tristate "MediaTek MT76x0U (USB) support" depends on MAC80211 depends on USB + depends on BROKEN help This adds support for MT7610U-based wireless USB dongles.