Modify

Opened 9 years ago

Closed 7 years ago

Last modified 6 years ago

#36 closed defect (wontfix)

build brcom6358 error

Reported by: sniperpr@… Owned by: amain
Priority: minor Milestone: milestone:
Component: openwrt Version: 2.0
Severity: severity 4 Keywords:
Cc:

Description

only svn co openwrt_backfire

make brcom6358 is OK.

but user DebWrt, error.
display :

In file included from drivers/net/phy/bcm63xx.c:10:
include/linux/phy.h:337: warning: 'struct sk_buff' declared inside parameter list
include/linux/phy.h:337: warning: its scope is only this definition or declaration, which is probably not what you want
include/linux/phy.h:338: warning: 'struct sk_buff' declared inside parameter list
  LD      drivers/net/phy/built-in.o
  LD      drivers/net/wireless/built-in.o
  CC      drivers/net/Space.o
  CC      drivers/net/loopback.o
  CC      drivers/net/bcm63xx_enet.o
drivers/net/bcm63xx_enet.c:1622: error: 'bcm_enet_netpoll' undeclared here (not in a function)
make[7]: *** [drivers/net/bcm63xx_enet.o] Error 1
make[6]: *** [drivers/net] Error 2
make[5]: *** [drivers] Error 2

Attachments (0)

Change History (6)

comment:1 Changed 9 years ago by amain

could be related to bug #45 where also some linking goes wrong

comment:2 Changed 9 years ago by stappers

FWIW, saturday January 8th 2011, svn rev140, still reproduceable, details:

make[5]: Entering directory `/usr/src/debwrt/build/openwrt-brcm63xx-Broadcom-wl-
trunk-default/build_dir/linux-brcm63xx/linux-2.6.35.9'
  CHK     include/linux/version.h
  CHK     include/generated/utsrelease.h
  CALL    scripts/checksyscalls.sh
  CHK     include/generated/compile.h
  CC      drivers/net/bcm63xx_enet.o
drivers/net/bcm63xx_enet.c:1614: error: 'bcm_enet_netpoll' undeclared here (not 
in a function)
make[7]: *** [drivers/net/bcm63xx_enet.o] Error 1
make[6]: *** [drivers/net] Error 2

( this comment is to say "we still care" ... )

comment:3 Changed 9 years ago by amain

  • Resolution set to wontfix
  • Status changed from new to closed

Reporter

  • use DebWrt with OpenWrt trunk and compare with OpenWrt Trunk
  • play with related menuconfig setting in make openwrt/menuconfig
  • grep through the code yourself and see why bcm_enet_netpoll is undeclared? Some ifdef not set?
  • be creative...if it is in DebWrt's OpenWrt it is probably also in OpenWrt

Setting bug to won't fix...takes to much time.

comment:4 Changed 8 years ago by amain

  • Resolution wontfix deleted
  • Severity set to severity 4
  • Status changed from closed to reopened

comment:5 Changed 7 years ago by amain

  • Resolution set to wontfix
  • Status changed from reopened to closed

Assuming this has been fixed upstream somehow. If not, please reopen.

comment:6 Changed 6 years ago by quackquack147@…

this is the solution as of now disable the undeclared variable. and save it in /path/to-your/debwrt-12.09-rnd/build/openwrt-brcm63xx-Atheros-trunk-HEAD/target/linux/brcm63xx/patches-3.10/
as 803-debwrt_support.patch

and it will continue to build.

cat 803-debwrt_support.patch
diff -rupN linux-3.10.18-old/drivers/net/ethernet/broadcom/bcm63xx_enet.c linux-3.10.18-new/drivers/net/ethernet/broadcom/bcm63xx_enet.c
--- linux-3.10.18-old/drivers/net/ethernet/broadcom/bcm63xx_enet.c 2013-11-29 04:50:24.000000000 +0530
+++ linux-3.10.18-new/drivers/net/ethernet/broadcom/bcm63xx_enet.c 2013-11-29 04:54:16.158012762 +0530
@@ -1684,7 +1684,7 @@ static const struct net_device_ops bcm_e

.ndo_do_ioctl = bcm_enet_ioctl,
.ndo_change_mtu = bcm_enet_change_mtu,

#ifdef CONFIG_NET_POLL_CONTROLLER

  • .ndo_poll_controller = bcm_enet_netpoll,

+/* .ndo_poll_controller = bcm_enet_netpoll, */

#endif
};


if you select jffs package instead of squashfs it wont boot but squashfs boots fine. i am working on a fix. will update soon. thanks in advance.
regards!
quackquack147
(partho)

Add Comment

Modify Ticket

Action
as closed The owner will remain amain.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.