* [U-Boot] Microblaze and Sparc boards are broken because of common timer func
@ 2013-11-12 2:12 Masahiro Yamada
2013-11-12 8:19 ` Michal Simek
0 siblings, 1 reply; 12+ messages in thread
From: Masahiro Yamada @ 2013-11-12 2:12 UTC (permalink / raw)
To: u-boot
Hello, Rob, Tom, Michal, Daniel.
Commit 8dfafdde88eb (Introduce common timer functions)
broke Microblaze and Sparc boards.
$ git checkout 8dfafdde88eb
$ CROSS_COMPILE=microblaze-unknown-linux-gnu- ./MAKEALL -a microblaze
<snipped>
lib/time.c:45: undefined reference to `timer_read_counter'
lib/libgeneric.o: In function `__udelay':
lib/time.c:86: relocation truncated to fit: R_MICROBLAZE_32_PCREL_LO
against symbol `get_ticks' defined in .text section in
arch/microblaze/cpu/libmicroblaze.o
lib/time.c:88: relocation truncated to fit: R_MICROBLAZE_32_PCREL_LO
against symbol `get_ticks' defined in .text section in
arch/microblaze/cpu/libmicroblaze.o
lib/libgeneric.o: In function `udelay':
lib/time.c:101: relocation truncated to fit: R_MICROBLAZE_32_PCREL_LO
against symbol `__udelay' defined in .text section in
arch/microblaze/cpu/libmicroblaze.o
lib/libgeneric.o: In function `timer_get_us':
lib/time.c:70: relocation truncated to fit: R_MICROBLAZE_32_PCREL_LO
against symbol `get_ticks' defined in .text section in
arch/microblaze/cpu/libmicroblaze.o
lib/libgeneric.o: In function `get_timer':
lib/time.c:65: relocation truncated to fit: R_MICROBLAZE_32_PCREL_LO
against symbol `get_ticks' defined in .text section in
arch/microblaze/cpu/libmicroblaze.o
$ CROSS_COMPILE=sparc-elf- ./MAKEALL grsim_leon2
Configuring for grsim_leon2 board...
sparc-elf-size: './u-boot': No such file
lib/libgeneric.o: In function `tick_to_time':
/home/yamada/workspace/arm-linux-pf/u-boot/lib/time.c:56: undefined reference to `get_tbclk'
lib/libgeneric.o: In function `get_ticks':
/home/yamada/workspace/arm-linux-pf/u-boot/lib/time.c:45: undefined reference to `timer_read_counter'
lib/libgeneric.o: In function `usec_to_tick':
/home/yamada/workspace/arm-linux-pf/u-boot/lib/time.c:74: undefined reference to `get_tbclk'
/home/yamada/workspace/arm-linux-pf/u-boot/lib/time.c:75: undefined reference to `get_tbclk'
(I can notice broken boards because I quite often build all boards
for the test of Kbuild porting.)
Best Regards
Masahiro Yamada
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] Microblaze and Sparc boards are broken because of common timer func
2013-11-12 2:12 [U-Boot] Microblaze and Sparc boards are broken because of common timer func Masahiro Yamada
@ 2013-11-12 8:19 ` Michal Simek
2013-11-12 9:04 ` Masahiro Yamada
0 siblings, 1 reply; 12+ messages in thread
From: Michal Simek @ 2013-11-12 8:19 UTC (permalink / raw)
To: u-boot
On 11/12/2013 03:12 AM, Masahiro Yamada wrote:
> Hello, Rob, Tom, Michal, Daniel.
>
> Commit 8dfafdde88eb (Introduce common timer functions)
> broke Microblaze and Sparc boards.
>
> $ git checkout 8dfafdde88eb
> $ CROSS_COMPILE=microblaze-unknown-linux-gnu- ./MAKEALL -a microblaze
> <snipped>
> lib/time.c:45: undefined reference to `timer_read_counter'
> lib/libgeneric.o: In function `__udelay':
> lib/time.c:86: relocation truncated to fit: R_MICROBLAZE_32_PCREL_LO
> against symbol `get_ticks' defined in .text section in
> arch/microblaze/cpu/libmicroblaze.o
> lib/time.c:88: relocation truncated to fit: R_MICROBLAZE_32_PCREL_LO
> against symbol `get_ticks' defined in .text section in
> arch/microblaze/cpu/libmicroblaze.o
> lib/libgeneric.o: In function `udelay':
> lib/time.c:101: relocation truncated to fit: R_MICROBLAZE_32_PCREL_LO
> against symbol `__udelay' defined in .text section in
> arch/microblaze/cpu/libmicroblaze.o
> lib/libgeneric.o: In function `timer_get_us':
> lib/time.c:70: relocation truncated to fit: R_MICROBLAZE_32_PCREL_LO
> against symbol `get_ticks' defined in .text section in
> arch/microblaze/cpu/libmicroblaze.o
> lib/libgeneric.o: In function `get_timer':
> lib/time.c:65: relocation truncated to fit: R_MICROBLAZE_32_PCREL_LO
> against symbol `get_ticks' defined in .text section in
> arch/microblaze/cpu/libmicroblaze.o
>
>
>
> $ CROSS_COMPILE=sparc-elf- ./MAKEALL grsim_leon2
> Configuring for grsim_leon2 board...
> sparc-elf-size: './u-boot': No such file
> lib/libgeneric.o: In function `tick_to_time':
> /home/yamada/workspace/arm-linux-pf/u-boot/lib/time.c:56: undefined reference to `get_tbclk'
> lib/libgeneric.o: In function `get_ticks':
> /home/yamada/workspace/arm-linux-pf/u-boot/lib/time.c:45: undefined reference to `timer_read_counter'
> lib/libgeneric.o: In function `usec_to_tick':
> /home/yamada/workspace/arm-linux-pf/u-boot/lib/time.c:74: undefined reference to `get_tbclk'
> /home/yamada/workspace/arm-linux-pf/u-boot/lib/time.c:75: undefined reference to `get_tbclk'
>
>
>
> (I can notice broken boards because I quite often build all boards
> for the test of Kbuild porting.)
This should be fixed by this patch - at least for microblaze.
commit 65ba7add0d609bbd035b8d42fafdaf428ac24751
Author: Rob Herring <rob.herring@calxeda.com>
Date: Fri Nov 8 08:40:43 2013 -0600
time: add weak annotation to timer_read_counter declaration
A weak annotation is needed in order to prevent link errors when
get_ticks is overridden. This fixes sandbox build.
Signed-off-by: Rob Herring <rob.herring@calxeda.com>
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 263 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20131112/e0ebb7ae/attachment.pgp>
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] Microblaze and Sparc boards are broken because of common timer func
2013-11-12 8:19 ` Michal Simek
@ 2013-11-12 9:04 ` Masahiro Yamada
2013-11-12 12:32 ` Michal Simek
0 siblings, 1 reply; 12+ messages in thread
From: Masahiro Yamada @ 2013-11-12 9:04 UTC (permalink / raw)
To: u-boot
Hello Michal
> This should be fixed by this patch - at least for microblaze.
>
> commit 65ba7add0d609bbd035b8d42fafdaf428ac24751
> Author: Rob Herring <rob.herring@calxeda.com>
> Date: Fri Nov 8 08:40:43 2013 -0600
>
> time: add weak annotation to timer_read_counter declaration
>
> A weak annotation is needed in order to prevent link errors when
> get_ticks is overridden. This fixes sandbox build.
>
> Signed-off-by: Rob Herring <rob.herring@calxeda.com>
No.
I tried both the current u-boot/master and commit 65ba7add0d,
but Microblaze board failed with the same error message.
Best Regards
Masahiro Yamada
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] Microblaze and Sparc boards are broken because of common timer func
2013-11-12 9:04 ` Masahiro Yamada
@ 2013-11-12 12:32 ` Michal Simek
2013-11-12 13:46 ` Rob Herring
2013-11-14 0:37 ` Masahiro Yamada
0 siblings, 2 replies; 12+ messages in thread
From: Michal Simek @ 2013-11-12 12:32 UTC (permalink / raw)
To: u-boot
Hi Masahiro,
On 11/12/2013 10:04 AM, Masahiro Yamada wrote:
> Hello Michal
>
>
>> This should be fixed by this patch - at least for microblaze.
>>
>> commit 65ba7add0d609bbd035b8d42fafdaf428ac24751
>> Author: Rob Herring <rob.herring@calxeda.com>
>> Date: Fri Nov 8 08:40:43 2013 -0600
>>
>> time: add weak annotation to timer_read_counter declaration
>>
>> A weak annotation is needed in order to prevent link errors when
>> get_ticks is overridden. This fixes sandbox build.
>>
>> Signed-off-by: Rob Herring <rob.herring@calxeda.com>
>
> No.
> I tried both the current u-boot/master and commit 65ba7add0d,
> but Microblaze board failed with the same error message.
>
Look at this log - uboot section.
http://www.monstr.eu/wiki/doku.php?id=log:2013-11-12_13_29_22
Which toolchain do you use?
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 263 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20131112/3ff2419f/attachment.pgp>
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] Microblaze and Sparc boards are broken because of common timer func
2013-11-12 12:32 ` Michal Simek
@ 2013-11-12 13:46 ` Rob Herring
2013-11-12 15:37 ` Michal Simek
2013-11-14 0:37 ` Masahiro Yamada
1 sibling, 1 reply; 12+ messages in thread
From: Rob Herring @ 2013-11-12 13:46 UTC (permalink / raw)
To: u-boot
On 11/12/2013 06:32 AM, Michal Simek wrote:
> Hi Masahiro,
>
> On 11/12/2013 10:04 AM, Masahiro Yamada wrote:
>> Hello Michal
>>
>>
>>> This should be fixed by this patch - at least for microblaze.
>>>
>>> commit 65ba7add0d609bbd035b8d42fafdaf428ac24751 Author: Rob
>>> Herring <rob.herring@calxeda.com> Date: Fri Nov 8 08:40:43
>>> 2013 -0600
>>>
>>> time: add weak annotation to timer_read_counter declaration
>>>
>>> A weak annotation is needed in order to prevent link errors
>>> when get_ticks is overridden. This fixes sandbox build.
>>>
>>> Signed-off-by: Rob Herring <rob.herring@calxeda.com>
>>
>> No. I tried both the current u-boot/master and commit
>> 65ba7add0d, but Microblaze board failed with the same error
>> message.
>>
>
> Look at this log - uboot section.
> http://www.monstr.eu/wiki/doku.php?id=log:2013-11-12_13_29_22
>
> Which toolchain do you use?
It builds for me with 4.8.0 from kernel.org, but I had to disable
systemace driver otherwise I get this error:
/opt/gcc-4.8.0-nolibc/microblaze-linux/bin/microblaze-linux-gcc -g
-Os -D__KERNEL__ -DCONFIG_SYS_TEXT_BASE=0x29000000
-I/home/rob/proj/git/u-boot-arm/include -fno-builtin -ffreestanding
-nostdinc -isystem
/opt/gcc-4.8.0-nolibc/microblaze-linux/bin/../lib/gcc/microblaze-linux/4.8.0/include
-pipe -ffixed-r31 -D__microblaze__ -mno-xl-soft-mul -mno-xl-soft-div
-mxl-barrel-shift -Wall -Wstrict-prototypes -fno-stack-protector
-Wno-format-nonliteral -Wno-format-security -fstack-usage -o
systemace.o systemace.c -c
systemace.c: In function 'ace_readw':
systemace.c:70:1: warning: stack usage computation not supported for
this target [enabled by default]
}
^
{standard input}: Assembler messages:
{standard input}:2499: Error: operation combines symbols in different
segments
{standard input}:2500: Error: operation combines symbols in different
segments
{standard input}:2503: Error: operation combines symbols in different
segments
{standard input}:2504: Error: operation combines symbols in different
segments
{standard input}:2509: Error: operation combines symbols in different
segments
{standard input}:2510: Error: operation combines symbols in different
segments
{standard input}:2519: Error: operation combines symbols in different
segments
{standard input}:2520: Error: operation combines symbols in different
segments
{standard input}:2523: Error: operation combines symbols in different
segments
{standard input}:2524: Error: operation combines symbols in different
segments
{standard input}:2533: Error: operation combines symbols in different
segments
{standard input}:2534: Error: operation combines symbols in different
segments
{standard input}:2537: Error: operation combines symbols in different
segments
{standard input}:2538: Error: operation combines symbols in different
segments
{standard input}:2543: Error: operation combines symbols in different
segments
{standard input}:2544: Error: operation combines symbols in different
segments
{standard input}:2553: Error: operation combines symbols in different
segments
{standard input}:2554: Error: operation combines symbols in different
segments
make[3]: *** [systemace.o] Error 1
Rob
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] Microblaze and Sparc boards are broken because of common timer func
2013-11-12 13:46 ` Rob Herring
@ 2013-11-12 15:37 ` Michal Simek
2013-11-18 13:24 ` Tom Rini
0 siblings, 1 reply; 12+ messages in thread
From: Michal Simek @ 2013-11-12 15:37 UTC (permalink / raw)
To: u-boot
On 11/12/2013 02:46 PM, Rob Herring wrote:
> On 11/12/2013 06:32 AM, Michal Simek wrote:
>> Hi Masahiro,
>>
>> On 11/12/2013 10:04 AM, Masahiro Yamada wrote:
>>> Hello Michal
>>>
>>>
>>>> This should be fixed by this patch - at least for microblaze.
>>>>
>>>> commit 65ba7add0d609bbd035b8d42fafdaf428ac24751 Author: Rob
>>>> Herring <rob.herring@calxeda.com> Date: Fri Nov 8 08:40:43
>>>> 2013 -0600
>>>>
>>>> time: add weak annotation to timer_read_counter declaration
>>>>
>>>> A weak annotation is needed in order to prevent link errors
>>>> when get_ticks is overridden. This fixes sandbox build.
>>>>
>>>> Signed-off-by: Rob Herring <rob.herring@calxeda.com>
>>>
>>> No. I tried both the current u-boot/master and commit
>>> 65ba7add0d, but Microblaze board failed with the same error
>>> message.
>>>
>>
>> Look at this log - uboot section.
>> http://www.monstr.eu/wiki/doku.php?id=log:2013-11-12_13_29_22
>>
>> Which toolchain do you use?
>
> It builds for me with 4.8.0 from kernel.org, but I had to disable
> systemace driver otherwise I get this error:
yes. That's my behaviour too when I use this toolchain.
I have to ping toolchain people to find out status of stack support
in mainline gcc.
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 263 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20131112/3c4f361b/attachment.pgp>
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] Microblaze and Sparc boards are broken because of common timer func
2013-11-12 12:32 ` Michal Simek
2013-11-12 13:46 ` Rob Herring
@ 2013-11-14 0:37 ` Masahiro Yamada
2013-11-18 9:36 ` Michal Simek
1 sibling, 1 reply; 12+ messages in thread
From: Masahiro Yamada @ 2013-11-14 0:37 UTC (permalink / raw)
To: u-boot
Hello Michal.
> >> This should be fixed by this patch - at least for microblaze.
> >>
> >> commit 65ba7add0d609bbd035b8d42fafdaf428ac24751
> >> Author: Rob Herring <rob.herring@calxeda.com>
> >> Date: Fri Nov 8 08:40:43 2013 -0600
> >>
> >> time: add weak annotation to timer_read_counter declaration
> >>
> >> A weak annotation is needed in order to prevent link errors when
> >> get_ticks is overridden. This fixes sandbox build.
> >>
> >> Signed-off-by: Rob Herring <rob.herring@calxeda.com>
> >
> > No.
> > I tried both the current u-boot/master and commit 65ba7add0d,
> > but Microblaze board failed with the same error message.
> >
>
> Look at this log - uboot section.
> http://www.monstr.eu/wiki/doku.php?id=log:2013-11-12_13_29_22
>
> Which toolchain do you use?
Microblaze crosstools I use is available at:
http://dev.gentoo.org/~vapier/u-boot/microblaze.tar.xz
Before commit 8dfafdde88eb, I could build the microblaze-generic
board with it.
But I can't build for the current u-boot/master.
Best Regards
Masahiro Yamada
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] Microblaze and Sparc boards are broken because of common timer func
2013-11-14 0:37 ` Masahiro Yamada
@ 2013-11-18 9:36 ` Michal Simek
2013-11-18 10:50 ` Masahiro Yamada
0 siblings, 1 reply; 12+ messages in thread
From: Michal Simek @ 2013-11-18 9:36 UTC (permalink / raw)
To: u-boot
On 11/14/2013 01:37 AM, Masahiro Yamada wrote:
> Hello Michal.
>
>>>> This should be fixed by this patch - at least for microblaze.
>>>>
>>>> commit 65ba7add0d609bbd035b8d42fafdaf428ac24751
>>>> Author: Rob Herring <rob.herring@calxeda.com>
>>>> Date: Fri Nov 8 08:40:43 2013 -0600
>>>>
>>>> time: add weak annotation to timer_read_counter declaration
>>>>
>>>> A weak annotation is needed in order to prevent link errors when
>>>> get_ticks is overridden. This fixes sandbox build.
>>>>
>>>> Signed-off-by: Rob Herring <rob.herring@calxeda.com>
>>>
>>> No.
>>> I tried both the current u-boot/master and commit 65ba7add0d,
>>> but Microblaze board failed with the same error message.
>>>
>>
>> Look at this log - uboot section.
>> http://www.monstr.eu/wiki/doku.php?id=log:2013-11-12_13_29_22
>>
>> Which toolchain do you use?
>
>
> Microblaze crosstools I use is available at:
> http://dev.gentoo.org/~vapier/u-boot/microblaze.tar.xz
Not sure if Mike maintains this toolchain.
> Before commit 8dfafdde88eb, I could build the microblaze-generic
> board with it.
> But I can't build for the current u-boot/master.
You can use this one and follow Rob's comments.
https://www.kernel.org/pub/tools/crosstool/files/bin/x86_64/4.8.0/
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 263 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20131118/c3cd71e4/attachment.pgp>
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] Microblaze and Sparc boards are broken because of common timer func
2013-11-18 9:36 ` Michal Simek
@ 2013-11-18 10:50 ` Masahiro Yamada
2013-11-18 10:53 ` Michal Simek
0 siblings, 1 reply; 12+ messages in thread
From: Masahiro Yamada @ 2013-11-18 10:50 UTC (permalink / raw)
To: u-boot
Hello Michal
> >> Which toolchain do you use?
> >
> >
> > Microblaze crosstools I use is available at:
> > http://dev.gentoo.org/~vapier/u-boot/microblaze.tar.xz
>
> Not sure if Mike maintains this toolchain.
Stefan Roses told me about this pre-built cross tool in a thread:
http://u-boot.10912.n7.nabble.com/Suggestion-for-building-all-boards-td165285.html
Accoding to this thread, it was built by Mike Frysinger.
> > Before commit 8dfafdde88eb, I could build the microblaze-generic
> > board with it.
> > But I can't build for the current u-boot/master.
>
> You can use this one and follow Rob's comments.
> https://www.kernel.org/pub/tools/crosstool/files/bin/x86_64/4.8.0/
Worked for me.
OK. Going forward I will use this. Thanks!
Best Regards
Masahiro Yamada
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] Microblaze and Sparc boards are broken because of common timer func
2013-11-18 10:50 ` Masahiro Yamada
@ 2013-11-18 10:53 ` Michal Simek
0 siblings, 0 replies; 12+ messages in thread
From: Michal Simek @ 2013-11-18 10:53 UTC (permalink / raw)
To: u-boot
On 11/18/2013 11:50 AM, Masahiro Yamada wrote:
> Hello Michal
>
>
>>>> Which toolchain do you use?
>>>
>>>
>>> Microblaze crosstools I use is available at:
>>> http://dev.gentoo.org/~vapier/u-boot/microblaze.tar.xz
>>
>> Not sure if Mike maintains this toolchain.
>
> Stefan Roses told me about this pre-built cross tool in a thread:
> http://u-boot.10912.n7.nabble.com/Suggestion-for-building-all-boards-td165285.html
>
> Accoding to this thread, it was built by Mike Frysinger.
>
>
>
>>> Before commit 8dfafdde88eb, I could build the microblaze-generic
>>> board with it.
>>> But I can't build for the current u-boot/master.
>>
>> You can use this one and follow Rob's comments.
>> https://www.kernel.org/pub/tools/crosstool/files/bin/x86_64/4.8.0/
>
> Worked for me.
> OK. Going forward I will use this. Thanks!
Great.
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 263 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20131118/07af1aca/attachment.pgp>
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] Microblaze and Sparc boards are broken because of common timer func
2013-11-12 15:37 ` Michal Simek
@ 2013-11-18 13:24 ` Tom Rini
2013-11-18 13:50 ` Michal Simek
0 siblings, 1 reply; 12+ messages in thread
From: Tom Rini @ 2013-11-18 13:24 UTC (permalink / raw)
To: u-boot
On Tue, Nov 12, 2013 at 04:37:34PM +0100, Michal Simek wrote:
> On 11/12/2013 02:46 PM, Rob Herring wrote:
> > On 11/12/2013 06:32 AM, Michal Simek wrote:
> >> Hi Masahiro,
> >>
> >> On 11/12/2013 10:04 AM, Masahiro Yamada wrote:
> >>> Hello Michal
> >>>
> >>>
> >>>> This should be fixed by this patch - at least for microblaze.
> >>>>
> >>>> commit 65ba7add0d609bbd035b8d42fafdaf428ac24751 Author: Rob
> >>>> Herring <rob.herring@calxeda.com> Date: Fri Nov 8 08:40:43
> >>>> 2013 -0600
> >>>>
> >>>> time: add weak annotation to timer_read_counter declaration
> >>>>
> >>>> A weak annotation is needed in order to prevent link errors
> >>>> when get_ticks is overridden. This fixes sandbox build.
> >>>>
> >>>> Signed-off-by: Rob Herring <rob.herring@calxeda.com>
> >>>
> >>> No. I tried both the current u-boot/master and commit
> >>> 65ba7add0d, but Microblaze board failed with the same error
> >>> message.
> >>>
> >>
> >> Look at this log - uboot section.
> >> http://www.monstr.eu/wiki/doku.php?id=log:2013-11-12_13_29_22
> >>
> >> Which toolchain do you use?
> >
> > It builds for me with 4.8.0 from kernel.org, but I had to disable
> > systemace driver otherwise I get this error:
>
> yes. That's my behaviour too when I use this toolchain.
> I have to ping toolchain people to find out status of stack support
> in mainline gcc.
Well, why is the assembler part failing? And what is the toolchain
people should use here?
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20131118/47bac817/attachment.pgp>
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] Microblaze and Sparc boards are broken because of common timer func
2013-11-18 13:24 ` Tom Rini
@ 2013-11-18 13:50 ` Michal Simek
0 siblings, 0 replies; 12+ messages in thread
From: Michal Simek @ 2013-11-18 13:50 UTC (permalink / raw)
To: u-boot
On 11/18/2013 02:24 PM, Tom Rini wrote:
> On Tue, Nov 12, 2013 at 04:37:34PM +0100, Michal Simek wrote:
>> On 11/12/2013 02:46 PM, Rob Herring wrote:
>>> On 11/12/2013 06:32 AM, Michal Simek wrote:
>>>> Hi Masahiro,
>>>>
>>>> On 11/12/2013 10:04 AM, Masahiro Yamada wrote:
>>>>> Hello Michal
>>>>>
>>>>>
>>>>>> This should be fixed by this patch - at least for microblaze.
>>>>>>
>>>>>> commit 65ba7add0d609bbd035b8d42fafdaf428ac24751 Author: Rob
>>>>>> Herring <rob.herring@calxeda.com> Date: Fri Nov 8 08:40:43
>>>>>> 2013 -0600
>>>>>>
>>>>>> time: add weak annotation to timer_read_counter declaration
>>>>>>
>>>>>> A weak annotation is needed in order to prevent link errors
>>>>>> when get_ticks is overridden. This fixes sandbox build.
>>>>>>
>>>>>> Signed-off-by: Rob Herring <rob.herring@calxeda.com>
>>>>>
>>>>> No. I tried both the current u-boot/master and commit
>>>>> 65ba7add0d, but Microblaze board failed with the same error
>>>>> message.
>>>>>
>>>>
>>>> Look at this log - uboot section.
>>>> http://www.monstr.eu/wiki/doku.php?id=log:2013-11-12_13_29_22
>>>>
>>>> Which toolchain do you use?
>>>
>>> It builds for me with 4.8.0 from kernel.org, but I had to disable
>>> systemace driver otherwise I get this error:
>>
>> yes. That's my behaviour too when I use this toolchain.
>> I have to ping toolchain people to find out status of stack support
>> in mainline gcc.
>
> Well, why is the assembler part failing? And what is the toolchain
> people should use here?
>
That's not question to me but to our toolchain guys what it is missing
in the mainline gcc. I have sent them an email. Will see what I get.
I do use these two toolchains[1] depends if you want to compile little
or big endian targets. Mainline u-boot, because of ll_temac driver,
must be compiled by big endian toolchains.
[1] http://www.monstr.eu/20131118-microblaze-toolchains.tar.gz
md5sum 20131118-microblaze-toolchains.tar.gz
940b8464154d3f1b58fe977156203b2a 20131118-microblaze-toolchains.tar.gz
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 263 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20131118/2fe1df41/attachment.pgp>
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2013-11-18 13:50 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-11-12 2:12 [U-Boot] Microblaze and Sparc boards are broken because of common timer func Masahiro Yamada
2013-11-12 8:19 ` Michal Simek
2013-11-12 9:04 ` Masahiro Yamada
2013-11-12 12:32 ` Michal Simek
2013-11-12 13:46 ` Rob Herring
2013-11-12 15:37 ` Michal Simek
2013-11-18 13:24 ` Tom Rini
2013-11-18 13:50 ` Michal Simek
2013-11-14 0:37 ` Masahiro Yamada
2013-11-18 9:36 ` Michal Simek
2013-11-18 10:50 ` Masahiro Yamada
2013-11-18 10:53 ` Michal Simek
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox