Merge pull request #106 from amouiche/master

Add the capability to change the storage (eMMC, SD, SPINOR...).

Merge branch 'quanghd96-master'

Merge branch 'master' of github.com:quanghd96/rkdeveloptool into quanghd96-master

Signed-off-by: shineseth-rk <yi.liu@rock-chips.com>

Merge pull request #57 from sharkcz/fixes

use a safe way to copy the chip string

Merge pull request #56 from WinterMute/mingw-w64-build-fixes

fix building with mingw-w64

Add the capability to change the storage (eMMC, SD, SPINOR...).

Implemented by reverse engineering of windows RKDevTool USB transactions.
The naming of operations (eg. READ_STORAGE, CHANGE_STORAGE)

Add the capability to change the storage (eMMC, SD, SPINOR...).

Implemented by reverse engineering of windows RKDevTool USB transactions.
The naming of operations (eg. READ_STORAGE, CHANGE_STORAGE) are
pure speculation, but the result is effective.
Also, the numbering of storage device (eg. 1=eMMC, 2=SD, 9=SPINOR) may be
dependent of the SOC.

Developed and tested on rk3588 (Radxa Rock 5 ITX)

Example of usage:
rkdeveloptool db rk3588_spl_loader_v1.15.113.bin
# select the SPINOR
rkdeveloptool cs 9
# clear the whole SPINOR device
rkdeveloptool ef

# write an image starting at block 0
rkdeveloptool wl 0 /tmp/spinor_image.bin

show more ...

change sprintf to snprintf

use a safe way to copy the chip string

based on https://developers.redhat.com/blog/2019/08/12/efficient-string-copying-and-concatenation-in-c

Fixes: https://github.com/rockchip-linux/rkdeveloptool/

use a safe way to copy the chip string

based on https://developers.redhat.com/blog/2019/08/12/efficient-string-copying-and-concatenation-in-c

Fixes: https://github.com/rockchip-linux/rkdeveloptool/issues/55

show more ...

fix building with mingw-w64

v1.32:
1.support to upgrade 356x loader

Signed-off-by: liuyi <liuyi@rock-chips.com>

1.fix erase ubi bug
2.fix download ubi bug

Signed-off-by: liuyi <liuyi@rock-chips.com>

1.support erase block forcely
2.fix create gpt bug

Signed-off-by: liuyi <liuyi@rock-chips.com>

Merge branch 'master' of https://github.com/rockchip-linux/rkdeveloptool

1.support to write ubi image
2.cut off iconv reference

Signed-off-by: liuyi <liuyi@rock-chips.com>

1.support to write ubi image
2.cut off iconv reference

Signed-off-by: liuyi <liuyi@rock-chips.com>

Merge pull request #42 from nullr0ute/gcc9-fixes

Use signed char for variable

Use signed char for variable

Architectures other than x86 don't by default treat char as signed so
be explicit so the build doesn't fail when building on other arches
such as Arm.

Signed-off-by: Pe

Use signed char for variable

Architectures other than x86 don't by default treat char as signed so
be explicit so the build doesn't fail when building on other arches
such as Arm.

Signed-off-by: Peter Robinson <pbrobinson@gmail.com>

show more ...

fix array boundry error

g++ -DHAVE_CONFIG_H -I. -I./cfg -Wall -Werror -Wextra -Wreturn-type
-fno-strict-aliasing -D_FILE_OFFSET_BITS=64 -D_LARGE_FILE
-I/usr/include/libusb-1.0 -O2 -g -pipe -Wall

fix array boundry error

g++ -DHAVE_CONFIG_H -I. -I./cfg -Wall -Werror -Wextra -Wreturn-type
-fno-strict-aliasing -D_FILE_OFFSET_BITS=64 -D_LARGE_FILE
-I/usr/include/libusb-1.0 -O2 -g -pipe -Wall -Werror=format-security
-Wp,-D_FORTIFY_SOURCE=2 -Wp,-D_GLIBCXX_ASSERTIONS -fexceptions
-fstack-protector-strong -grecord-gcc-switches
-specs=/usr/lib/rpm/redhat/redhat-hardened-cc1
-specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -m64 -mtune=generic
-fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -c
-o main.o main.cpp
main.cpp: In function 'bool unpackBoot(char*)':
main.cpp:1734:11: error: array subscript 20 is outside array bounds of
'char [20]' [-Werror=array-bounds]1734 | str[len] = 0;| ~~~~~~~~~^~~
main.cpp:1772:7: note: while referencing 'name'1772 | char name[MAX_NAME_LEN];| ^~~~
cc1plus: all warnings being treated as errors
make[1]: *** [Makefile:475: main.o] Error 1

Reported-by: Peter Robinson <pbrobinson@gmail.com>
Signed-off-by: Kever Yang <kever.yang@rock-chips.com>

show more ...

Merge pull request #39 from cmuellner/compilation

Fix compilation issue in split_item().

Revert "Fixed Build Failed - gcc 8.2.0"

This reverts commit c43947cfb1e6cbd1a4bdf163bec449cde8bdc777.

Merge pull request #38 from cmuellner/parser

parse_parameter: Continue parsing after 'mtdparts'.

Merge pull request #31 from KL-Yang/master

Fixed Build Failure - gcc 8.2.0

Fix compilation issue in split_item().

Recent compilers complain with the following message:

main.cpp: In function ‘void split_item(STRING_VECTOR&, char*)’:
main.cpp:2840:10: error: ‘char* strn

Fix compilation issue in split_item().

Recent compilers complain with the following message:

main.cpp: In function ‘void split_item(STRING_VECTOR&, char*)’:
main.cpp:2840:10: error: ‘char* strncpy(char*, const char*, size_t)’ output truncated before terminating nul copying as many bytes from a string as its length

This patch addresses this issue.

Signed-off-by: Christoph Muellner <christoph.muellner@theobroma-systems.com>

show more ...

parse_parameter: Continue parsing after 'mtdparts'.

The parse_paramter loop iterates over a parameter.txt file's contents
and parses that line-by-line. However, the current implementation
silently s

parse_parameter: Continue parsing after 'mtdparts'.

The parse_paramter loop iterates over a parameter.txt file's contents
and parses that line-by-line. However, the current implementation
silently stops after parsing 'mtdparts'. This is problematic if
a 'uuid' token is found in one of the following lines.

This patch addresses the problem, by simply continue to iterate
over the next lines after 'mtdparts'.

Signed-off-by: Christoph Muellner <christoph.muellner@theobroma-systems.com>

show more ...

Fixed Build Failed - gcc 8.2.0

ERROR:

main.cpp:2841:10: error: ‘char* strncpy(char*, const char*, size_t)’
output truncated before terminating nul copying as many bytes from
a string as its length

Fixed Build Failed - gcc 8.2.0

ERROR:

main.cpp:2841:10: error: ‘char* strncpy(char*, const char*, size_t)’
output truncated before terminating nul copying as many bytes from
a string as its length [-Werror=stringop-truncation]

Since the allocation is only 100*char, the reasonable fix would be
limit the strncpy to only (100-1)*char.

show more ...