u-boot.git/tools, branch v2017.09-rc2

rockchip: rk3368: mkimage: add support for the RK3368

2017-08-13T15:12:22+00:00

This commit adds support for RK3368 SoC in mkimage.

Signed-off-by: Philipp Tomsich 
Reviewed-by: Simon Glass

mips-relocs: Fix warning from gcc 6.3

2017-07-25T18:44:38+00:00

It seems that gcc 6.3 at least is smart enough to warn about the _val
variable being unassigned in the default case in the set_hdr_field()
macro, but not smart enough to figure out that the default case is never
taken. This results in warnings such as the following:

  pfx##hdr32[idx].field = _val;   \
                          ^
 ../tools/mips-relocs.c:51:11: note: _val was declared here
   uint64_t _val;      \
            ^
 ../tools/mips-relocs.c:88:2: note: in expansion of macro set_hdr_field
   set_hdr_field(p, idx, field, val)
   ^~~~~~~~~~~~~
 ../tools/mips-relocs.c:408:3: note: in expansion of macro set_phdr_field
    set_phdr_field(i, p_filesz, load_sz);
    ^~~~~~~~~~~~~~
 ../tools/mips-relocs.c: In function main:
 ../tools/mips-relocs.c:77:25: warning: _val may be used uninitialized
    in this function [-Wmaybe-uninitialized]

Avoid this by assigning _val = 0 in the default case, and asserting that
we didn't actually hit it for good measure.

For reference gcc 7.1.1 seems to be smart enough to not hit the above
warning without this patch.

Signed-off-by: Paul Burton 
Fixes: 011dd93ca97a ("MIPS: Stop building position independent code")
Cc: Daniel Schwierzeck 
Cc: Tom Rini 
Cc: u-boot@lists.denx.de

MIPS: Stop building position independent code

2017-07-25T18:44:00+00:00

U-Boot has up until now built with -fpic for the MIPS architecture,
producing position independent code which uses indirection through a
global offset table, making relocation fairly straightforward as it
simply involves patching up GOT entries.

Using -fpic does however have some downsides. The biggest of these is
that generated code is bloated in various ways. For example, function
calls are indirected through the GOT & the t9 register:

  8f998064   lw     t9,-32668(gp)
  0320f809   jalr   t9

Without -fpic the call is simply:

  0f803f01   jal    be00fc04 

This is more compact & faster (due to the lack of the load & the
dependency the jump has on its result). It is also easier to read &
debug because the disassembly shows what function is being called,
rather than just an offset from gp which would then have to be looked up
in the ELF to discover the target function.

Another disadvantage of -fpic is that each function begins with a
sequence to calculate the value of the gp register, for example:

  3c1c0004   lui    gp,0x4
  279c3384   addiu  gp,gp,13188
  0399e021   addu   gp,gp,t9

Without using -fpic this sequence no longer appears at the start of each
function, reducing code size considerably.

This patch switches U-Boot from building with -fpic to building with
-fno-pic, in order to gain the benefits described above. The cost of
this is an extra step during the build process to extract relocation
data from the ELF & write it into a new .rel section in a compact
format, plus the added complexity of dealing with multiple types of
relocation rather than the single type that applied to the GOT. The
benefit is smaller, cleaner, more debuggable code. The relocate_code()
function is reimplemented in C to handle the new relocation scheme,
which also makes it easier to read & debug.

Taking maltael_defconfig as an example the size of u-boot.bin built
using the Codescape MIPS 2016.05-06 toolchain (gcc 4.9.2, binutils
2.24.90) shrinks from 254KiB to 224KiB.

Signed-off-by: Paul Burton 
Cc: Daniel Schwierzeck 
Cc: u-boot@lists.denx.de
Reviewed-by: Daniel Schwierzeck 
Tested-by: Daniel Schwierzeck

RFC: moveconfig: Use toolchains from buildman

2017-07-23T02:22:46+00:00

It is annoying to have to set up and maintain two sets of toolchains, one
for buildman and one for moveconfig.

Adjust moveconfig to make use to buildman's toolchains. This should make
things easier.

One missing feature is the ability to specify the toolchain on the command
line with a special environment variable, e.g. CROSS_COMPILE_ARM. I'm not
sure if that is useful, but if it is it could be implemented in buildman.

Signed-off-by: Simon Glass 
Reviewed-by: Philipp Tomsich 
Tested-by: Philipp Tomsich

moveconfig: Tidy up imply flag parsing

2017-07-23T02:22:46+00:00

Add an option to specify 'all' to enable all flags. Also print an error
if an unrecognised flag is used. At present it just prints usage
information which is not very helpful.

Signed-off-by: Simon Glass

dtc: mkimage: Add the possibility to specify DTC

2017-07-22T19:36:19+00:00

FreeBSD recently switch to it's BSDL dtc. While it support most of the
features of the GPL one it still lacks the incbin directive.
Add the possibility to specify which dtc we want to use for compiling dts
and generating fit image.

Signed-off-by: Emmanuel Vadot 
Acked-by: Simon Glass

tools/fw_env: use fsync to ensure that data is physically stored

2017-07-22T19:36:16+00:00

Closing a file descriptor does not guarantee that the data has been
successfully saved to disk, as the kernel might defer the write.

Signed-off-by: Michael Heimpold

Merge branch 'master' of git://www.denx.de/git/u-boot-imx

2017-07-18T12:42:48+00:00

imx: reorganize IMX code as other SOCs

2017-07-12T08:17:44+00:00

Change is consistent with other SOCs and it is in preparation
for adding SOMs. SOC's related files are moved from cpu/ to
mach-imx/.

This change is also coherent with the structure in kernel.

Signed-off-by: Stefano Babic 

CC: Fabio Estevam 
CC: Akshay Bhat 
CC: Ken Lin 
CC: Marek Vasut 
CC: Heiko Schocher 
CC: "Sébastien Szymanski" 
CC: Christian Gmeiner 
CC: Stefan Roese 
CC: Patrick Bruenn 
CC: Troy Kisky 
CC: Nikita Kiryanov 
CC: Otavio Salvador 
CC: "Eric Bénard" 
CC: Jagan Teki 
CC: Ye Li 
CC: Peng Fan 
CC: Adrian Alonso 
CC: Alison Wang 
CC: Tim Harvey 
CC: Martin Donnelly 
CC: Marcin Niestroj 
CC: Lukasz Majewski 
CC: Adam Ford 
CC: "Albert ARIBAUD (3ADEV)" 
CC: Boris Brezillon 
CC: Soeren Moch 
CC: Richard Hu 
CC: Wig Cheng 
CC: Vanessa Maegima 
CC: Max Krummenacher 
CC: Stefan Agner 
CC: Markus Niebel 
CC: Breno Lima 
CC: Francesco Montefoschi 
CC: Jaehoon Chung 
CC: Scott Wood 
CC: Joe Hershberger 
CC: Anatolij Gustschin 
CC: Simon Glass 
CC: "Andrew F. Davis" 
CC: "Łukasz Majewski" 
CC: Patrice Chotard 
CC: Nobuhiro Iwamatsu 
CC: Hans de Goede 
CC: Masahiro Yamada 
CC: Stephen Warren 
CC: Andre Przywara 
CC: "Álvaro Fernández Rojas" 
CC: York Sun 
CC: Xiaoliang Yang 
CC: Chen-Yu Tsai 
CC: George McCollister 
CC: Sven Ebenfeld 
CC: Filip Brozovic 
CC: Petr Kulhavy 
CC: Eric Nelson 
CC: Bai Ping 
CC: Anson Huang 
CC: Sanchayan Maity 
CC: Lokesh Vutla 
CC: Patrick Delaunay 
CC: Gary Bisson 
CC: Alexander Graf 
CC: u-boot@lists.denx.de
Reviewed-by: Fabio Estevam 
Reviewed-by: Christian Gmeiner

tools/kwbimage: fix v1 header verification

2017-07-12T04:57:55+00:00

The verify_header callback in kwbimage.c only verifies v0 headers checksum.
Running 'mkimage -l' on a v1 image gives the following misleading output:

GP Header: Size ae000000 LoadAddr 34160600

Implement support for v1 headers. For that, factor out the header checksum code
to a separate main_hdr_checksum_ok() routine. This routine relies on the fact
that the checksum field offset is the same in both v0 and v1 headers. With this
patch applied 'mkimage -l' correctly identifies the image:

Image Type:   MVEBU Boot from sdio Image
Image version:1
Data Size:    398904 Bytes = 389.55 KiB = 0.38 MiB
Load Address: 007fffc0
Entry Point:  00800000

Signed-off-by: Baruch Siach 
Signed-off-by: Stefan Roese