public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [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