Re: [U-Boot] [PATCH 0/2] improve hello_world standalone application for arm

[Max Krummenacher]

Dear Wolfgang

Am Samstag, den 12.08.2017, 20:32 +0200 schrieb Wolfgang Denk:
> Dear Max,
> 
> In message <20170812090346.7887-1-max.krummenac...@toradex.com> you wrote:
> > 
> > 
> > This series addresses
> > - hardcoded entry address, use LOADADDR if available as the entry point 
> > instead
> 
> I'm not sure which sort of problem you are trying to fix, but what
> you describe here is inherently broken.
> 
> The entry point of an image is almost never ever at the beginning of
> an image.

I was mislead by the table in point 6. of doc/README.standalone that it
is a design decision that the entry point is whatever is linked to the
beginning of the text segment. Thus the sloppy use of entry point and
load address.

Max
> 
> Naked-by: Wolfgang Denk 
> 
> Sorry!
> 
> 
> Best regards,
> 
> Wolfgang Denk
> 
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 0/2] improve hello_world standalone application for arm

[Wolfgang Denk]

Dear Max,

In message <20170812090346.7887-1-max.krummenac...@toradex.com> you wrote:
> 
> This series addresses
> - hardcoded entry address, use LOADADDR if available as the entry point 
> instead

I'm not sure which sort of problem you are trying to fix, but what
you describe here is inherently broken.

The entry point of an image is almost never ever at the beginning of
an image.

Naked-by: Wolfgang Denk 

Sorry!


Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,  Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
What is wanted is not the will to believe,  but the will to find out,
which is the exact opposite.
-- Bertrand Russell, "Skeptical Essays", 1928
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH 0/2] improve hello_world standalone application for arm

[Max Krummenacher]

This series addresses
- hardcoded entry address, use LOADADDR if available as the entry point instead
- fix thumb build, jumping with 'go' to the entry point expects arm code

Note that in addition to the two fixes I've seen random freezes
or 'random' printed stuff when using an early linaro gcc 6 compiler.
Adding an initialized variable helped in that case
static int dummy_var_in_text = 1;
I assume that this forces alignment of some linker sections.
(e.g. I see that __bss_start points to 0x1201027e, with the variable
this moves to 0x12010280)

However with the current linaro compilers this does not happen so I
don't propose a patch for this issue.
Linaro GCC 5.4-2017.05 5.4.1 20170404
Linaro GCC 6.3-2017.05 6.3.1 20170404
Linaro GCC 7.1-2017.05 7.1.1 20170510

This series is available at 
http://git.toradex.com/cgit/u-boot-toradex.git/log/?h=for-next


Max Krummenacher (2):
  arm: use $loadaddr as the standalone entry point
  hello_world.c: fix entry point in case of arm thumb binary

 arch/arm/config.mk|  4 
 doc/README.standalone |  2 +-
 examples/standalone/hello_world.c | 15 +++
 3 files changed, 20 insertions(+), 1 deletion(-)

-- 
2.13.1

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot