mips BCM47XX compile error

mips BCM47XX compile error

[Adrian Bunk]

Commit aab547ce0d1493d400b6468c521a0137cd8c1edf
(ssb: Add Gigabit Ethernet driver) causes the following
build error with bcm47xx_defconfig:

<--  snip  -->

...
  LD      .tmp_vmlinux1
arch/mips/pci/built-in.o: In function `pcibios_enable_device':
(.text+0x1f8): undefined reference to `pcibios_plat_dev_init'
arch/mips/pci/built-in.o: In function `pcibios_enable_device':
(.text+0x1f8): relocation truncated to fit: R_MIPS_26 against `pcibios_plat_dev_init'
arch/mips/pci/built-in.o: In function `pcibios_init':
pci.c:(.init.text+0x14c): undefined reference to `pcibios_map_irq'
pci.c:(.init.text+0x158): undefined reference to `pcibios_map_irq'
make[1]: *** [.tmp_vmlinux1] Error 1

<--  snip  -->

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed

Re: mips BCM47XX compile error

[Michael Buesch]

On Sunday 20 April 2008 12:03:47 Adrian Bunk wrote:
> Commit aab547ce0d1493d400b6468c521a0137cd8c1edf
> (ssb: Add Gigabit Ethernet driver) causes the following
> build error with bcm47xx_defconfig:
> 
> <--  snip  -->
> 
> ...
>   LD      .tmp_vmlinux1
> arch/mips/pci/built-in.o: In function `pcibios_enable_device':
> (.text+0x1f8): undefined reference to `pcibios_plat_dev_init'
> arch/mips/pci/built-in.o: In function `pcibios_enable_device':
> (.text+0x1f8): relocation truncated to fit: R_MIPS_26 against `pcibios_plat_dev_init'
> arch/mips/pci/built-in.o: In function `pcibios_init':
> pci.c:(.init.text+0x14c): undefined reference to `pcibios_map_irq'
> pci.c:(.init.text+0x158): undefined reference to `pcibios_map_irq'
> make[1]: *** [.tmp_vmlinux1] Error 1

Some parts of the new 47xx arch code are not ported, yet.
Somebody should port all the new code from openwrt SVN over to mainline.
I don't really have time for that, at the moment, though.

-- 
Greetings Michael.

Re: mips BCM47XX compile error

[Aurelien Jarno]

Michael Buesch a écrit :
> On Sunday 20 April 2008 12:03:47 Adrian Bunk wrote:
>> Commit aab547ce0d1493d400b6468c521a0137cd8c1edf
>> (ssb: Add Gigabit Ethernet driver) causes the following
>> build error with bcm47xx_defconfig:
>>
>> <--  snip  -->
>>
>> ...
>>   LD      .tmp_vmlinux1
>> arch/mips/pci/built-in.o: In function `pcibios_enable_device':
>> (.text+0x1f8): undefined reference to `pcibios_plat_dev_init'
>> arch/mips/pci/built-in.o: In function `pcibios_enable_device':
>> (.text+0x1f8): relocation truncated to fit: R_MIPS_26 against `pcibios_plat_dev_init'
>> arch/mips/pci/built-in.o: In function `pcibios_init':
>> pci.c:(.init.text+0x14c): undefined reference to `pcibios_map_irq'
>> pci.c:(.init.text+0x158): undefined reference to `pcibios_map_irq'
>> make[1]: *** [.tmp_vmlinux1] Error 1
> 
> Some parts of the new 47xx arch code are not ported, yet.

It would have been nice at least to warn before breaking a platform.

> Somebody should port all the new code from openwrt SVN over to mainline.
> I don't really have time for that, at the moment, though.

I gave a quick look at openwrt SVN, and only find 2.6.23 patches. I
can't find a possible fix among them. Do you have a better pointer?

-- 
  .''`.  Aurelien Jarno	            | GPG: 1024D/F1BCDB73
 : :' :  Debian developer           | Electrical Engineer
 `. `'   aurel32@debian.org         | aurelien@aurel32.net
   `-    people.debian.org/~aurel32 | www.aurel32.net

Re: mips BCM47XX compile error

[Aurelien Jarno]

On Mon, Apr 21, 2008 at 12:49:42AM +0200, Aurelien Jarno wrote:
> Michael Buesch a écrit :
> > On Sunday 20 April 2008 12:03:47 Adrian Bunk wrote:
> >> Commit aab547ce0d1493d400b6468c521a0137cd8c1edf
> >> (ssb: Add Gigabit Ethernet driver) causes the following
> >> build error with bcm47xx_defconfig:
> >>
> >> <--  snip  -->
> >>
> >> ...
> >>   LD      .tmp_vmlinux1
> >> arch/mips/pci/built-in.o: In function `pcibios_enable_device':
> >> (.text+0x1f8): undefined reference to `pcibios_plat_dev_init'
> >> arch/mips/pci/built-in.o: In function `pcibios_enable_device':
> >> (.text+0x1f8): relocation truncated to fit: R_MIPS_26 against `pcibios_plat_dev_init'
> >> arch/mips/pci/built-in.o: In function `pcibios_init':
> >> pci.c:(.init.text+0x14c): undefined reference to `pcibios_map_irq'
> >> pci.c:(.init.text+0x158): undefined reference to `pcibios_map_irq'
> >> make[1]: *** [.tmp_vmlinux1] Error 1
> > 
> > Some parts of the new 47xx arch code are not ported, yet.
> 
> It would have been nice at least to warn before breaking a platform.
> 
> > Somebody should port all the new code from openwrt SVN over to mainline.
> > I don't really have time for that, at the moment, though.
> 
> I gave a quick look at openwrt SVN, and only find 2.6.23 patches. I
> can't find a possible fix among them. Do you have a better pointer?
> 

I have finally found it, the code was not in a form of a patch, but in
form of a C code file.

I am working on a fix.

-- 
  .''`.  Aurelien Jarno	            | GPG: 1024D/F1BCDB73
 : :' :  Debian developer           | Electrical Engineer
 `. `'   aurel32@debian.org         | aurelien@aurel32.net
   `-    people.debian.org/~aurel32 | www.aurel32.net

Re: mips BCM47XX compile error

[Michael Buesch]

On Monday 21 April 2008 00:49:42 Aurelien Jarno wrote:
> Michael Buesch a écrit :
> > On Sunday 20 April 2008 12:03:47 Adrian Bunk wrote:
> >> Commit aab547ce0d1493d400b6468c521a0137cd8c1edf
> >> (ssb: Add Gigabit Ethernet driver) causes the following
> >> build error with bcm47xx_defconfig:
> >>
> >> <--  snip  -->
> >>
> >> ...
> >>   LD      .tmp_vmlinux1
> >> arch/mips/pci/built-in.o: In function `pcibios_enable_device':
> >> (.text+0x1f8): undefined reference to `pcibios_plat_dev_init'
> >> arch/mips/pci/built-in.o: In function `pcibios_enable_device':
> >> (.text+0x1f8): relocation truncated to fit: R_MIPS_26 against `pcibios_plat_dev_init'
> >> arch/mips/pci/built-in.o: In function `pcibios_init':
> >> pci.c:(.init.text+0x14c): undefined reference to `pcibios_map_irq'
> >> pci.c:(.init.text+0x158): undefined reference to `pcibios_map_irq'
> >> make[1]: *** [.tmp_vmlinux1] Error 1
> > 
> > Some parts of the new 47xx arch code are not ported, yet.
> 
> It would have been nice at least to warn before breaking a platform.

You know, I intentionally did this, because I'm evil...

> > Somebody should port all the new code from openwrt SVN over to mainline.
> > I don't really have time for that, at the moment, though.
> 
> I gave a quick look at openwrt SVN, and only find 2.6.23 patches. I
> can't find a possible fix among them. Do you have a better pointer?

It's fixed in the plain files that are copied to the build directory.

-- 
Greetings Michael.

Re: mips BCM47XX compile error

[Michael Buesch]

On Monday 21 April 2008 09:28:51 Aurelien Jarno wrote:
> On Mon, Apr 21, 2008 at 12:49:42AM +0200, Aurelien Jarno wrote:
> > Michael Buesch a écrit :
> > > On Sunday 20 April 2008 12:03:47 Adrian Bunk wrote:
> > >> Commit aab547ce0d1493d400b6468c521a0137cd8c1edf
> > >> (ssb: Add Gigabit Ethernet driver) causes the following
> > >> build error with bcm47xx_defconfig:
> > >>
> > >> <--  snip  -->
> > >>
> > >> ...
> > >>   LD      .tmp_vmlinux1
> > >> arch/mips/pci/built-in.o: In function `pcibios_enable_device':
> > >> (.text+0x1f8): undefined reference to `pcibios_plat_dev_init'
> > >> arch/mips/pci/built-in.o: In function `pcibios_enable_device':
> > >> (.text+0x1f8): relocation truncated to fit: R_MIPS_26 against `pcibios_plat_dev_init'
> > >> arch/mips/pci/built-in.o: In function `pcibios_init':
> > >> pci.c:(.init.text+0x14c): undefined reference to `pcibios_map_irq'
> > >> pci.c:(.init.text+0x158): undefined reference to `pcibios_map_irq'
> > >> make[1]: *** [.tmp_vmlinux1] Error 1
> > > 
> > > Some parts of the new 47xx arch code are not ported, yet.
> > 
> > It would have been nice at least to warn before breaking a platform.
> > 
> > > Somebody should port all the new code from openwrt SVN over to mainline.
> > > I don't really have time for that, at the moment, though.
> > 
> > I gave a quick look at openwrt SVN, and only find 2.6.23 patches. I
> > can't find a possible fix among them. Do you have a better pointer?
> > 
> 
> I have finally found it, the code was not in a form of a patch, but in
> form of a C code file.
> 
> I am working on a fix.
> 

What is the status of the fix?
We have a bugzilla entry
http://bugzilla.kernel.org/show_bug.cgi?id=10493
So if it's not ready, yet, I will have to fix it by myself.

-- 
Greetings Michael.

Re: mips BCM47XX compile error

[Aurelien Jarno]

On Sun, May 25, 2008 at 11:53:46AM +0200, Michael Buesch wrote:
> On Monday 21 April 2008 09:28:51 Aurelien Jarno wrote:
> > On Mon, Apr 21, 2008 at 12:49:42AM +0200, Aurelien Jarno wrote:
> > > Michael Buesch a écrit :
> > > > On Sunday 20 April 2008 12:03:47 Adrian Bunk wrote:
> > > >> Commit aab547ce0d1493d400b6468c521a0137cd8c1edf
> > > >> (ssb: Add Gigabit Ethernet driver) causes the following
> > > >> build error with bcm47xx_defconfig:
> > > >>
> > > >> <--  snip  -->
> > > >>
> > > >> ...
> > > >>   LD      .tmp_vmlinux1
> > > >> arch/mips/pci/built-in.o: In function `pcibios_enable_device':
> > > >> (.text+0x1f8): undefined reference to `pcibios_plat_dev_init'
> > > >> arch/mips/pci/built-in.o: In function `pcibios_enable_device':
> > > >> (.text+0x1f8): relocation truncated to fit: R_MIPS_26 against `pcibios_plat_dev_init'
> > > >> arch/mips/pci/built-in.o: In function `pcibios_init':
> > > >> pci.c:(.init.text+0x14c): undefined reference to `pcibios_map_irq'
> > > >> pci.c:(.init.text+0x158): undefined reference to `pcibios_map_irq'
> > > >> make[1]: *** [.tmp_vmlinux1] Error 1
> > > > 
> > > > Some parts of the new 47xx arch code are not ported, yet.
> > > 
> > > It would have been nice at least to warn before breaking a platform.
> > > 
> > > > Somebody should port all the new code from openwrt SVN over to mainline.
> > > > I don't really have time for that, at the moment, though.
> > > 
> > > I gave a quick look at openwrt SVN, and only find 2.6.23 patches. I
> > > can't find a possible fix among them. Do you have a better pointer?
> > > 
> > 
> > I have finally found it, the code was not in a form of a patch, but in
> > form of a C code file.
> > 
> > I am working on a fix.
> > 
> 
> What is the status of the fix?
> We have a bugzilla entry
> http://bugzilla.kernel.org/show_bug.cgi?id=10493
> So if it's not ready, yet, I will have to fix it by myself.
> 

A patch [1] has been sent 1 month ago, it waits with the other BCM47xx
patches.

Aurelien

[1] http://marc.info/?l=linux-kernel&m=120876451216558&w=2

-- 
  .''`.  Aurelien Jarno	            | GPG: 1024D/F1BCDB73
 : :' :  Debian developer           | Electrical Engineer
 `. `'   aurel32@debian.org         | aurelien@aurel32.net
   `-    people.debian.org/~aurel32 | www.aurel32.net

Re: mips BCM47XX compile error

[Michael Buesch]

On Sunday 25 May 2008 13:36:34 Aurelien Jarno wrote:
> On Sun, May 25, 2008 at 11:53:46AM +0200, Michael Buesch wrote:
> > On Monday 21 April 2008 09:28:51 Aurelien Jarno wrote:
> > > On Mon, Apr 21, 2008 at 12:49:42AM +0200, Aurelien Jarno wrote:
> > > > Michael Buesch a écrit :
> > > > > On Sunday 20 April 2008 12:03:47 Adrian Bunk wrote:
> > > > >> Commit aab547ce0d1493d400b6468c521a0137cd8c1edf
> > > > >> (ssb: Add Gigabit Ethernet driver) causes the following
> > > > >> build error with bcm47xx_defconfig:
> > > > >>
> > > > >> <--  snip  -->
> > > > >>
> > > > >> ...
> > > > >>   LD      .tmp_vmlinux1
> > > > >> arch/mips/pci/built-in.o: In function `pcibios_enable_device':
> > > > >> (.text+0x1f8): undefined reference to `pcibios_plat_dev_init'
> > > > >> arch/mips/pci/built-in.o: In function `pcibios_enable_device':
> > > > >> (.text+0x1f8): relocation truncated to fit: R_MIPS_26 against `pcibios_plat_dev_init'
> > > > >> arch/mips/pci/built-in.o: In function `pcibios_init':
> > > > >> pci.c:(.init.text+0x14c): undefined reference to `pcibios_map_irq'
> > > > >> pci.c:(.init.text+0x158): undefined reference to `pcibios_map_irq'
> > > > >> make[1]: *** [.tmp_vmlinux1] Error 1
> > > > > 
> > > > > Some parts of the new 47xx arch code are not ported, yet.
> > > > 
> > > > It would have been nice at least to warn before breaking a platform.
> > > > 
> > > > > Somebody should port all the new code from openwrt SVN over to mainline.
> > > > > I don't really have time for that, at the moment, though.
> > > > 
> > > > I gave a quick look at openwrt SVN, and only find 2.6.23 patches. I
> > > > can't find a possible fix among them. Do you have a better pointer?
> > > > 
> > > 
> > > I have finally found it, the code was not in a form of a patch, but in
> > > form of a C code file.
> > > 
> > > I am working on a fix.
> > > 
> > 
> > What is the status of the fix?
> > We have a bugzilla entry
> > http://bugzilla.kernel.org/show_bug.cgi?id=10493
> > So if it's not ready, yet, I will have to fix it by myself.
> > 
> 
> A patch [1] has been sent 1 month ago, it waits with the other BCM47xx
> patches.
> 
> Aurelien
> 
> [1] http://marc.info/?l=linux-kernel&m=120876451216558&w=2
> 

Thanks a lot, Aurelien :)


-- 
Greetings Michael.