fs/squashfs: sqfs_probe: fix possible memory leak on error

If SquashFS magic number is invalid, there's a memory leak.

Reviewed-by: Joao Marcos Costa <jmcosta944@gmail.com>
Signed-off-by: Richard Genoud <richard.genoud@posteo.net>
[jh80.chung: cherry picked from mainline commit ccd4c08a452b3703ee16ba730a84b7caadcff97a]
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
Change-Id: I3be0a354f3407d13902c244580109eb0a9ff71e9

fs/squashfs: sqfs_read: fix memory leak on finfo.blk_sizes

finfo.blk_sizes may not be freed in case of error in the for loop
Setting it to null and freeing it at the end makes prevents that from
happening.

Reviewed-by: Joao Marcos Costa <jmcosta944@gmail.com>
Signed-off-by: Richard Genoud <richard.genoud@posteo.net>
[jh80.chung: cherry picked from mainline commit 571b67ee1dcb989dc50575f14e87bfdef6bfd8db]
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
Change-Id: I97086373caa5a031f3ba12aa4997de3eab10c530

fs/squashfs: sqfs_get_abs_path: fix possible memory leak on error

if  sqfs_tokenize(rel_tokens, rc, rel); fails, the function exits
without freeing the array base_tokens.

Reviewed-by: Joao Marcos Costa <jmcosta944@gmail.com>
Signed-off-by: Richard Genoud <richard.genoud@posteo.net>
[jh80.chung: cherry picked from mainline commit 33686804d29e676177d408f1ce047b8c35185ddb]
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
Change-Id: I377f5aeda3ed3f6b889ce9f9c506629c21f45d3d

fs/squashfs: sqfs_get_abs_path: fix error check

the return value of sqfs_tokenize(rel_tokens, rc, rel); wasn't checked.
(but "ret" value was !)
This is obviouly a typo.

Reviewed-by: Joao Marcos Costa <jmcosta944@gmail.com>
Signed-off-by: Richard Genoud <richard.genoud@posteo.net>
[jh80.chung: cherry picked from mainline commit 53db0e24a8f981c1e303a1c745b8f0e28544096f]
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
Change-Id: Ib9ff7ebded9de101a0d929414c8ecdf6137e8621

fs/squashfs: sqfs_frag_lookup: simplify error handling

For consistency with other functions.

Reviewed-by: Joao Marcos Costa <jmcosta944@gmail.com>
Signed-off-by: Richard Genoud <richard.genoud@posteo.net>
[jh80.chung: cherry picked from mainline commit c9b8e86f8bd238421b35ec15d88b4d3cac8a0992]
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
Change-Id: If6a955583f876fe4399982e58ade2784c2da1b30

fs/squashfs: sqfs_read: fix another memory leak

data_buffer was allocated in a loop and freed only once.

Signed-off-by: Richard Genoud <richard.genoud@posteo.net>
[jh80.chung: cherry picked from mainline commit 555459e7931f7ae34a4e10d1e21ce7aada7aa59b]
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
Change-Id: I69dc9bf407d07c7af28f7c3bb1b1b2966f1ee216

fs/squashfs: sqfs_read: fix memory leak

sqfs_closedir() should be called to free memory allocated by
sqfs_opendir()

Reviewed-by: Joao Marcos Costa <jmcosta944@gmail.com>
Signed-off-by: Richard Genoud <richard.genoud@posteo.net>
[jh80.chung: cherry picked from mainline commit 7ce9745cbeae7733a7068f19a7fc3ec8f21dc38c]
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
Change-Id: I200f764bb6ce2bb828e2217d1b0f949e1db14597

fs/squashfs: sqfs_read: remove useless sqfs_closedir()

as sqfs_opendir failed, there's no need to call sqfs_closedir

Signed-off-by: Richard Genoud <richard.genoud@posteo.net>
[jh80.chung: cherry picked from mainline commit b34949fe9baaae49003b2981213495bab3afef9b]
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
Change-Id: Id61d825b7976f151a77738dadf5c99ba0613d955

fs/squashfs: sqfs_read: fix dangling pointer dirs->entry

dirs->entry shouldn't be left dangling as it could be freed twice.

Signed-off-by: Richard Genoud <richard.genoud@posteo.net>
[jh80.chung: cherry picked from mainline commit d1d8d75f69f82f7ef7c55b58a7d9e73055956f3c]
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
Change-Id: Ie76becc6bfb30341d1964a34154ebe4014def678

fs/squashfs: sqfs_size: remove useless sqfs_closedir()

as sqfs_opendir failed, there's no need to call sqfs_closedir

Signed-off-by: Richard Genoud <richard.genoud@posteo.net>
[jh80.chung: cherry picked from mainline commit 35475f83a107901551c31f7bc70f5266862a33e4]
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
Change-Id: I809ae83d5949f86d547b52982cf318e7202a6868

fs/squashfs: sqfs_size: fix dangling pointer dirs->entry

dirs->entry shouldn't be left dangling as it could be freed twice.

Signed-off-by: Richard Genoud <richard.genoud@posteo.net>
[jh80.chung: cherry picked from mainline commit 508a9dc7f694df11c7de7460f888f508e40a2046]
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
Change-Id: I9f03d0d18e49f3ad0b5b7f1ca6f48625c00e46aa

fs/squashfs: sqfs_concat_tokens: check if malloc succeeds

memory allocation should always be checked

Reviewed-by: Joao Marcos Costa <jmcosta944@gmail.com>
Signed-off-by: Richard Genoud <richard.genoud@posteo.net>
[jh80.chung: cherry picked from mainline commit dc3312c43c82ff9335b2bf4ee4f72365e824adfd]
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
Change-Id: I160a321fe56fc63476d19adbbaa62129daa03f79

fs/squashfs: sqfs_read_inode_table: fix dangling pointer

inode_table should not be left dangling as it may be freed in sqfs_opendir

Signed-off-by: Richard Genoud <richard.genoud@posteo.net>
[jh80.chung: cherry picked from mainline commit 4c83d2755776d4dd5e15a930d48db1082fc0d1cc]
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
Change-Id: I0d63396f7885f68140e78623df8e06b8b00049ac

fs/squashfs: sqfs_search_dir: fix memory leaks

path, target, res, rem and sym_tokens were not free on error nor success.

Reviewed-by: Joao Marcos Costa <jmcosta944@gmail.com>
Signed-off-by: Richard Genoud <richard.genoud@posteo.net>
[jh80.chung: cherry picked from mainline commit cd54591afd42c547f43a7d509862f20e09264cde]
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
Change-Id: I9417cb0f010dc267eeef4d0ee1aa8ba45a6678bd

fs/squashfs: sqfs_search_dir: fix dangling pointer

dirs->entry shouldn't be left dangling as it could be freed twice.

Signed-off-by: Richard Genoud <richard.genoud@posteo.net>
[jh80.chung: cherry picked from mainline commit 01e71ec61ab84bcdd4df708f8930d5acf1c2674d]
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
Change-Id: I51db60ce9c428536c10a72a9ff4a6466629b6a4a

fs/squashfs: sqfs_read_directory_table: fix memory leak

pos_list wasn't freed on every error

Reviewed-by: Joao Marcos Costa <jmcosta944@gmail.com>
Signed-off-by: Richard Genoud <richard.genoud@posteo.net>
[jh80.chung: cherry picked from mainline commit 7d23b2c5fb093142400467d8b81ba84786ed6f2c]
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
Change-Id: I152e9c306a194261b7b98e408a02000830194ba1

fs/squashfs: sqfs_split_path: fix memory leak and dangling pointers

*file and *dir were not freed on error

Reviewed-by: Joao Marcos Costa <jmcosta944@gmail.com>
Signed-off-by: Richard Genoud <richard.genoud@posteo.net>
[jh80.chung: cherry picked from mainline commit 5487477802e0ad7bfad046af25f4a04cfd98cb7a]
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
Change-Id: I185e3bb66f52d16eb02a2b2076aa723aab5cb753

fs/squashfs: sqfs_closedir: fix memory leak

sqfs_dirs wasn't freed anywhere.

Reviewed-by: Joao Marcos Costa <jmcosta944@gmail.com>
Signed-off-by: Richard Genoud <richard.genoud@posteo.net>
[jh80.chung: cherry picked from mainline commit 87d11e08e4792b3d188143972ba3a1651a52beee]
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
Change-Id: Icd463416ec6b70ee90c912faf077d3795f055f29

fs/squashfs: sqfs_opendir: fix some memory leaks and dangling pointers

When trying to load an non-existing file, the cpu hangs!

Signed-off-by: Richard Genoud <richard.genoud@posteo.net>
[jh80.chung: cherry picked from mainline commit f268768d43bbf11c7107597abef57c6b86b6b229]
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
Change-Id: I8273a22d76a36e39cf4ca0f48afd9fd409280217

fs/squashfs: Fix index off by 1 for inode SQFS_LDIR_TYPE

I've created a squashfs file system with Yocto (it use squashfs-tools)
and u-boot command sqfsls give the error:'Error while searching inode:
unknown type.'
After some digging in the code I found that the index is off by 1.
This patch fix this issue and I can successful use the sqfsls command.
After search for the squashfs format I found a link talk about a
similar issue but this time in the documentation. The link is:
https://github.com/AgentD/squashfs-tools-ng/commit/e6588526838caece9529

Signed-off-by: Gerard Koskamp <gerard.koskamp@nedap.com>
Tested-by: Joao Marcos Costa <joaomarcos.costa@bootlin.com>
[jh80.chung: cherry picked from mainline commit c49b0eb034710f08fc68cbfa2c23811e10667ad9]
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
Change-Id: I02ba72c07ea7c57ebace7103357e2b81bcf48772

tizen: rpi: move bootscripts to tizen specific directory

Move bootscripts from board/raspberrypi/rpi to tizen/bootscript/.
And combine to one script, because those are duplicated almost codes.

Change-Id: I2ee929bbcd6cb68566d441a272a1ba3d84beb100
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

tizen: bootscript: amlogic: support recovery mode

Support recovery mode.

Change-Id: Idf6dca45c0e3636c0579b674223e4af7c59d767f
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

tizen: bootscript: amlogic: remove tizen specific kernel loading address

Remove tizen specific kernel loading address.
When it's entered to recovery mode, "Synchronous Abort" is occurred.

Change-Id: I6ff3afc52299b12714a25bb83fe114467c007573
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

tizen: bootscript: amlogic: remove voutmode in bootargs

Remove voutmode in bootargs.
If it's defined, it will be passed from boot_args.
- It's used in only odroid-n2/c4.

Change-Id: I112c4932c31b8e33cdb8c073712f443c5c88c1ac
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

tizen: script: ignore when no file in output directory

The build output directory can have no file. Ignore when no file
in the output directory.

Change-Id: Ia2ce4c25f7f386915c2606a36d4ea66ca46747e3
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>

tizen: move Function button check to separate env script

This will make CONFIG_PREBOOT a bit more readable and easier to
understand.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: I75d0e42a6b26c7e4c5d7d4818e9e0f450151a1b2

configs: amlogic: check a reboot reason to enter download mode

Check a reboot reason to enter download mode.
If reboot reason is download, it will be entered to download mode.

Change-Id: I9a2223962ad49a19a2b084711107dd23e6c17b95
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

ARM: meson: add a reboot reason about DOWNLOAD

Tizen is using "reboot download" as download mode.

Change-Id: Ia53d1605982957bc2388552e81c4d3cd99eac1e8
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

tizen: make version parsing insensitive to message tranlations

Localised versions of version messages present version number at different
positions. Matching numbers with grep(1) ensures that version number is
extracted properly regardles of the message format.

Change-Id: I6dd4bc476aaa43c1a736f269dc14910d8653b730
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>

ARM: dts: meson: switch TFLASH_VDD_EN pin to open drain on Odroid-C4

For the proper reboot Odroid-C4 board requires to switch TFLASH_VDD_EN
pin to the high impedance mode, otherwise the board is stuck in the
middle of loading early stages of the bootloader from SD card.

This can be achieved by using the OPEN_DRAIN flag instead if the
ACTIVE_HIGH, what will leave the pin in input to achieve high state (pin
has the pull-up) and solve the issue.

Suggested-by: Neil Armstrong <narmstrong@baylibre.com>
Change-Id: Id91d0c5e354ed58fcba3b2e8a5ebe7f2953f2142
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

tizen: vim3(l)_deconfig: enable THOR mode if Func button is pressed

Extend the preboot script with a check of the Func button state. If it is
pressed, enter the THOR download mode.

Change-Id: Ic026c345078adbd9030b65e250b18afcc171553b
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

cmd/button: return button status

To make the button command useful in a shell script it should return the
status of the button:

* 0 (true) - pressed, on
* 1 (false) - not pressed, off

The button command takes only one argument. Correct maxargs.

Adjust the Python unit test.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
Reviewed-by: Philippe Reynes <philippe.reynes@softathome.com>
[backport of mainline commit a6bfd71a96201127836d59736abcb54dc2d5e1a5]
Change-Id: Ic799b7e189e3fb6793083a92b2ceadb0f5884889
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

tizen: script: fix wrong indent for case

There are unmet indent for case statement. Fix the wrong indent.

Change-Id: Iafb7e9586c5b401f481cef5afbc879c8f12f90e2
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>

lib: tizen: fix build error with tizen_defconfig

Fix build error with tizen_defconfig.
tizen_defconfig doesn't support DM_VIDEO.

Change-Id: Ie86a45e97c8d8c4c70e13dfaba70f23b0bdccd93
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

samsung: tizen_amlogic: update loglevel from 4 to 7

Update loglevel from 4 to 7.
It's useful that can see more log message because of kernel isn't stable.
In future, it will be restore to 4.

Change-Id: I162b3a049acd1a6765b4c7f9e5ce610fce238e59
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

button: add a simple ADC-based button driver

Add a simple ADC-based button driver. This driver binds to the 'adc-keys'
device tree node.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: I6da7101eff3ce53766d899f49f5839d728d52fb3

adc: meson-saradc: skip hardware init only if ADC is enabled

The driver skips hardware initialization if it is already configured by
the earlier bootloader stage (BL30). Skip the initialization only if the
hardware is really initialized and enabled.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: I3e2e2d270ad3e7e7f38e2bc3ce2a924a47b164af

adc: meson-saradc: add G12A variant

Add support for the SARADC variant found on the G12A SoCs family.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: If519d333e9773d089f37a8c7b4ccb144be68925b

clk: meson: add minimal driver for g12a-ao clocks

Add minimal driver AO clocks on meson G12A family. Only ADC related clocks
are supported.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: I0c91848bc55493e19570db333e7da6020d687907

tizen: bootscript: amlogic: restore ${opts} support

After unification of the Amlogic based boards boot scripts, support for
appending ${opts} to kernel cmdline has been lost. Restore it to make the
kernel debugging easier.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: I531822612f31a473019d45c832cb8726aea51060

usb: dwc3-meson-g12a: always configure dr-mode

dwc3_meson_g12a_force_mode() sets the dr-mode of the USB PHY. However
it skips setting the mode if it matches the one done during driver probe
(stored in private structure). This fails if the mode has been changed
to other value and then back to initial one. Fix this by configuring the
dr-mode always, regadless of the one set at the driver probe).

This fixes operation of USB gadget based drivers when they are initialized
for the second time.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: I5188ca7cccf8e9b05a6cae9fd2b6f9d55fc75cb7

samsung: tizen_rpi: add hal.img in dfu information

Add hal.img in dfu information.
It's for just prepering to thor download.
- Avoid to fail thor protocol when hal image is created.

Change-Id: I78df372fc0bfa260d120c16c59a4b7bfd6fdddcf
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

meson: add TIZEN_AMLOGIC_USE_VENDOR_KERNEL config

Add TIZEN_AMLOGIC_USE_VENDOR_KERNEL config.
It's more claar than just using CONFIG_TIZEN.
If someone wants to use mainline kernel, just disable it.

Enable CONFIG_TIZEN_AMLOGIC_USE_VENDOR_KERNEL.
- odroid-{c4,n2}_defconfig
- khadas-{vim3,vim3l}_defconfig
Tizen is using each board's vendor kernel.

Change-Id: I2a11e3e08e58ced7ef286487044a1616ac3ac8f4
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

lib: tizen: Add support for thor status notifications on video console

This patch implements thor flashing progress bar and status notifications
on graphics console.

Change-Id: Iba4945ae26358936de1e1847d0095f97e6ab3498
Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>

thor: Add API for thor status notifications

This patch adds an empty by default thor_status_notify() function which
can be reimplemented as required and which allows to add notifications
of thor status and flashing progress, e.g. on external monitor.

Change-Id: Ic1eaad7f3f5adbca404162b745a9287d5a9ae247
Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>

amlogic: vim3: add code for reading ethernet MAC address from efuse

Add a simple code for reading built-in ethernet MAC address from efuse.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: Ic1c39da270bc58f3b8e89a0631494e779281d46d

packaging: u-boot-amlogic: add a spec file for all amlogic boards

Add a spec file for all amlogic boards.
- It will be provided u-boot-{n2,c4,vim3,vim3l} packages
- Remove each spec files

Change-Id: Ic4f8ab6f5608fbe8606fdc3cca591e53c7bdfe18
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

configs: put a space to sys prompt in odroid-{c4,n2}_defconfig and khadas-vim3(l)_defconfig

Like all other targets, put a space to sys prompt.

Change-Id: Ie106f61406b686dfc7ec98079cb6adf96a7d6fff
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>

common: splash_source: fix -Wint-to-pointer-cast warning

Fix -Wint-to-pointer-cast warning

common/splash_source.c: In function 'splash_load_raw':
common/splash_source.c:100:12: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
  100 |  bmp_hdr = (struct bmp_header *)bmp_load_addr;
      |            ^
common/splash_source.c: In function 'splash_sf_read_raw':
common/splash_source.c:39:47: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
   39 |  return spi_flash_read(sf, offset, read_size, (void *)bmp_load_addr);
      |                                               ^

Change-Id: I3ab5875a7c33c0b22b2824dc8534dafcf70abd26
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

configs: Enable options for splash image in odroid-{c4,n2}_defconfig

These options are needed for the splash image loading.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: I892b493a1f1d2509649a8cca59d25c62b3613332

tizen: amlogic: Add hdmimode env variable

The hdmimode variable set to "1080p60hz" value allows the HDMI driver
to switch to default full HD timings for QHD/UHD monitors and such
monitors can still be used in lower resolution mode.
This a workaround until proper support for QHD/UHD is implemented
in the VPU driver.
The new variable is also used in kernel bootargs.

Change-Id: Ib1b58ee9173fa380c1c79bf819b0ffaca1a6aab9
Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>

video: meson: Add fallback to default timings for QHD/UHD monitors

The HDMI driver currently supports only up to 1920x1200 resolutions
and QHD/UHD monitors might not specify detailed timings in EDID other
than those for their maximum resolution, even though they work with
lower resolutions like full HD. In such cases display_read_timings()
will fail breaking the video output driver initialization.

As a workaround fallback to full HD timings is added for QHD or UHD
monitors. The default timing will only be used when "hdmimode"
environment variable is set to "1080p60hz", otherwise the driver
switches to CVBS mode as before, when display timings are
unsupported/could not be read.

The proper solution would be to implement support for 2K/4K HDMI
modes, which are also not supported yet by driver in the kernel.

Change-Id: I407228392a6b1b6e8b0c55a6962235111df0c69c
Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>

tizen: Add splash image file to the bootloader tar image

Change-Id: Ie3e1f76a663eddb65887ec42c651dedbd6b07f81
Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>

configs: Enable options for splash image in khadas-vim3(l)_defconfig

These options are needed for the splash image loading.

Change-Id: Ieeddd2e2d9ff6872297b9eb3f3d899702ee717a7
Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>

tizen: amlogic: add CONFIG entries for splash image loading

These options are needed for decompressing the logo image file
(gzipped BMP).

Change-Id: Ieb3c35adf5007e2f795f2cecc3cf365e7a40b629
Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>

configs: tizen_amlogic: Add environment variables for splash image

Set the splash file location, bmp image load address and logo position
on the display.

The compressed bitmap file should be installed on first mmc partition.

Change-Id: Ic1ce517460beaf318821f53ed3adbe6c41e091e8
Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>

packaging: u-boot-vim3(l): Add tizen logo bitmap file

This image will be used as u-boot splash image.

Change-Id: I262cc2219435afc7c8f208ff39b2bf7cce6a636c
Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>

tizen: enable support for formatting MBR partition layout during flashing

This is ensured by the following steps:
1. enable 'mbr' command and provide default partiton layout,
2. add support for formating block device from THOR downloaded script,
3. check MBR partition layout on boot and enter THOR mode if invalid
4. provide a script for formating MBR for THOR download

MBR partition layout is checked on boot if it mathes the one defined
during u-boot compilation. If not, set dfu_alt_info to the minimal safe
list and enter THOR download mode to flash the new, proper partition
layout and the complete Tizen image.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: I490e5b3314513012259f3981feae047f95649c3b

thor: add support for dfu_alt_info reintialization from the downloaded script

Reinitialize dfu_env_entities after flashing the 'SCRIPT' entity to
ensure that the potential changes to the 'dfu_alt_info' environment
variable are applied.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: Iad98f348d25ee37fbb7493d4e567d1a4de55dc2c

dfu: add support for dfu_alt_info reintialization from the flashed script

Reinitialize DFU USB gadget after flashing the 'SCRIPT' entity to ensure
that the potential changes to the 'dfu_alt_info' environment variable are
applied.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: Ia5025c765c83defc96da17b8c059dff833ec257b

dfu: add 'SCRIPT' entity support

Define a new 'SCRIPT' type for DFU entities. The downloaded data are
treated as simple u-boot's scripts and executed with run_command_list()
function.

Flashing the 'SCRIPT' entity might result in changing the 'dfu_alt_info'
from that script. Add a global variable for tracking the potential need
to reinitialize the dfu_alt_info structures.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: I8b43528ba82c4aecf5e84bbcb6ce1ca7b7621075

cmd: Add MBR partition layout control utility

Add a 'mbr' command to let user create or verify MBR partition layout
based on the provided text description. The partition layout is
altearnatively read from 'mbr_parts' environment variable. This can be
used in scripts to help system image flashing tools to ensure proper
partition layout.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: I8fd4d79a630296d41567e6288b6b4e4d083f7648

part: dos: add code for creating MBR partition layout

Add code for creating and writing MBR partition layout. The code generates
similar layout of EBRs (Exteneded Block Records) and logical volumes as
Linux's fdisk utility.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: I52724de45f32cdde43d900e586d30831906f9280

disk: dos: make some functions static

Make function not used outside this file static.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: I37039841c3a1530d5a81e5917f2f497b3bedcbbc

disk: dos: use generic macro for unaligned le32 access

Use a generic helper for reading LE32 integers.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: Iafa317eec51629733b1fac645b96a749845edd08

disk: dos: add some defines for the hardcoded numbers

Add handy defines for some hardcoded magic numbers related to extended
partition handling.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: Ibcd229cc2782d2ca37601ca035afc467c589850e

disk: dos: rename write_mbr_partition to write_mbr_sector

write_mbr_partition() function name is a bit misleading, so rename it to
write_mbr_sector(). This is a preparation for adding code for writing a
complete MBR partition layout.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: Ifd90eb8c2584f7474b9a19e917eac24ad1b03013

tizen: build-amlogic: add support for building images for all boards

Add 'all' target for build-amlogic.sh to build images for all supported
boards and pack them to the common amlogic_bootloader.tar.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: I691672d454850e50e04c64a20c2f960ca83c709e

tizen: bootscript: amlogic: use proper string comparison operator

The proper string comparison operator for u-boot's test is '='.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: Ic7a3f5f6b456631473df94e311694774651c4144

tizen: limit boot to MMC0 and MMC1 devices

Other types of boot are not supported on Tizen, so disable generating
scripts and envs for them.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: Ibe41e9ff8e60aa6949f4a082276083adde33e187

tizen: avoid doubled entries in params.bin

Disable generating entries (re)defined in tizen_amlogic.h to avoid
confusing user, which one is valid.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: I123c58b8fb71320e20815bbb0eebb3cb3b5c5a9e

tools: image-host.c: use correct output format

When building on a 32bit host the following warning occurs:

tools/image-host.c: In function ‘fit_image_read_data’:
tools/image-host.c:296:56: warning: format ‘%ld’ expects argument of
type ‘long int’, but argument 3 has type ‘__off64_t’
{aka ‘long long int’} [-Wformat=]
   printf("File %s don't have the expected size (size=%ld, expected=%d)\n",
                                                      ~~^
                                                      %lld
          filename, sbuf.st_size, expected_size);
                    ~~~~~~~~~~~~
tools/image-host.c:311:62: warning: format ‘%ld’ expects argument of
type ‘long int’, but argument 4 has type ‘__off64_t’
{aka ‘long long int’} [-Wformat=]
   printf("Can't read all file %s (read %zd bytes, expexted %ld)\n",
                                                            ~~^
                                                            %lld
          filename, n, sbuf.st_size);
                       ~~~~~~~~~~~~

Fix the format strings.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
Reviewed-by: Simon Glass <sjg@chromium.org>
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
[jh80.chung: cherry picked from mainline commit 3311eda658e0]
Change-Id: I5ed82a5c3d876a56d6b9ac75598eeed2d2b85d2a

tizen: remove generic params.bin and u-boot.bin

The generic entity params.bin is no longer functional, so remove it. Also
remove generic entity for the u-boot.bin to force user to use proper
board specific image.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: I6820c9b7fad7259525ae5ec59cae667e0482d680

tizen: sanitize file names for VIM3(l) boards

Use only the board name (vim3/vim3l) in file names (i.e. u-boot-vim3.bin)
like it is done for odroid boards (i.e. u-boot-c4.bin).

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: Ia8e4cca3bc96cd1408b75c2bae4d2a29ccd41160

video: Fix line padding calculation for 16 and 24 bpp bitmaps

The line size in the bitmap file is multiple of 4 bytes. In current code
the complement of row size to a multiple of 4 bytes is further unnecessarily
multiplied by the pixel size. This result in incorrect displaying of bitmaps
with image width which is not multiple of 4 pixels. Fix this by removing
the unnecessary multiplication.

Tested with 24BPP bitmap and XRGB32 display.

Change-Id: I9bd307780e388a70f47823444fe3ecdf910d08fb
Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>

tizen: unify build script for all boards

Use the same build script for all Amlogic boards.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: I69565c89e3960bbe353545df217f5e81d00465a4

tizen: amlogic: change a params name to postfix of each board's name

Chagne a params.bin name to postfix of each board's name and location
to root directory.
Also changed the kernel directory from each boardname to kvim/odroid.

Change-Id: I0468f0894d7220cc55b64885c8c411f2c5340606
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

packaging: amlogic: use one boot.scr.uimg script

Use one boot.scr.uimg instead of each scsripts.
To prevent conflict, move to boot.scr.uimg after installed each script.
Also change the place of boot.scr.uimg in build script.

Change-Id: Ia7d0746062f6d548ae1fb36a7111d55d353a1545
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

tizen: bootscript: amlogic: calculate tizen_kernel_add_r with ramdisk's size

Calculate tizen_kernel_addr_r with ramdisk's size.
It cat be prevented to overwrite the images.

Change-Id: If766b514e40865c3a531c3ebbe75f6b7a6898b2a
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

configs: enable CONFIG_SETEXPR config

Eanble CONFIG_SETEXPR configuration.

Change-Id: If1255fabae584b17e176fb7d8cbc0324ac2b3902
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

packaging: u-boot-kvim3l: add kvim3l spec file

Add kvim3l spec file for khadas-vim3l.

Change-Id: Ie7ddaaab70a3218a81927ccef9953bdb41e4eb52
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

tizen: bootscript: create one bootscript for amlogic boards

Create one bootscript for amlogic boards.
There is a common variable to use about each boards. So it can be
reused.
Remove each bootscript files. Instead, use one script file with
board_name.

Change-Id: If50529fc1f34f3ac274ce8a0955499f3e3e12d22
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

tizen: bootscript: remove vim3l boot script

Remove vim3l boot sript.
Instead, use tizen-boot-khadas-vim3-64b.scr.
In future, it can be used one image after combined other amlogic script.

Change-Id: Ifa16622ca0507db6b61720d7cddd3bd66bfbb07f
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

tizen: bootscript: remove fdtfile in boot script

Remove fdtfile in boot script file.
Move fdtfile variable from bootscript to tizen_amlogic header.

Change-Id: I7c87253eab09332e6ab73e5174bf93a95fde60f7
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

tizen: bootscript: move boot scripts under bootscript directory

Move each bootscript files to tizen/bootscript directory.
It's more easier to maintain which boot script is used for tizen than now

Change-Id: I435fdf8352195091763226fdbb05b844219f5842
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

build: kvim3: add vim3l in build-kvim3 script

There is no reason not to reuse build-kvim3 script.
If one wants to build vim3 or vim3l, pass the argument as target name.
- ./build-kvim3.sh vim3 or vim3l

Change-Id: I645e52f8dc97269e2048c0a2357755029dcfe7d9
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

amlogic: w400: add tizen boot script for vim3l

Add tizen boot script for vim3l.

Change-Id: I77503992af9bc3955ddd4fefd71ded12351f20d6
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

samsung: tizen_amlogic: add khadas vim3l environment value

Add khadas vim3l environment value to use thor download.

Change-Id: Ie97abfee1c48ad43c29c7539a1835593e24ecc5b
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

ARM: meson: Add TIZEN_KHADAS_VIM3L config

Add TIZEN_KHADAS_VIM3L config and enable it.

Change-Id: I20c431b4cc5f54f1ea8f24dbc8e94290e6047f96
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

configs: khadas: enable configurations relevant to Tizen

Enable configurations relevant to Tizen.

Change-Id: I66dc227aa94ec7626558df611790a829b4bef165
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

fip: kvim3l: add the prebuilt images to make u-boot image

Add the prebuilt images to make u-boot image.
It's from khadas opensource github.
https://github.com/khadas/u-boot.gi -b khadas-vims-v2015.01

Refer to doc/board/amlogic/khadas-vim3l.rst

Change-Id: I579d9375a682c5e737080950ae25009032c7584e
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

tizen: sd_fusing_amlogic: sync with recent script for old tools

The script was not fully sync with recent sd_fusing script change
and it cayses old sfdisk size issue because not enough EXTEND_SZ
to align block offset in extended partition. Update the parts with
recent rpi fusing script and also remove duplicated data size
variable.

Change-Id: I5bd939f9a29093d9acaf76e2b4cae7b1c8ae2de0
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>

tizen: amlogic: remove unused environment values

Remove unused environment values.
It doesn't need to enter the flash mode with inform file.
If it needs to consider the flash mode, then we can use Amlogic's
scheme with secure monitor command.

Change-Id: Ia5374f7b2f411c3a48c0d469c837541875f0f84a
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

samsung: tizen_amlogic: fix wrong defined name

Fix wrong defined name from KVIM3 to VIM3.
It's occurred a build error on gbs system.

Change-Id: Icff90e97d3e5b3c655ac0685780281b9ad414859
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

samsung: tizen_amlogic: increase u-boot size for raw write

Increase u-boot size from 0x850 to 0x1000.
- From 1M to 2M.
If u-boot image is over 1064K (0x850), then it will be broken.

Change-Id: I95b8a0b074d5c4b2877b98c2e5cd4057941fda64
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

tizen: amlogic: make 'dfu_alt_info' static

The data layout on the storage device is the same for both eMMC and SD
card, so after switching to the default boot MMC device, the 'dfu_alt_info'
string is static, fully known during the compile time. Use this simple
approach, which is a bit more error proof for future changes. It also
obsoletes the custom code added to Amlogic boards.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: I37fc03fe245bfdf5229922ecfd5f0d00ef00b6e6

tizen: amlogic: move trailing '\0' to the common place

To make the env setup a bit more error proof, move the required trailing
'\0' character for the 'boot_prefixes' entry to the common place.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: I535afa0a870b07a9d8637add8cccb8d55bcca5a0

tizen: amlogic: fix separators in env entries setup

The proper separator for environment entries is '\0' not '\n'.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: I409ee8d10ffbfd7f235cbf9ced5547b2d8dd7a06

tizen: amlogic: automatically set "dfu_device" env

Automatically set "dfu_device" to the current boot MMC device number. This
allows to call dfu/thor/ums commands without any parameters for the default
storage device.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: I4378161c3ebf5377d50e1e28dd57d3d1511f3648

dfu: mmc: use the default MMC device if entity specifies it as -1

Use the default MMC device set in the command line if entity specifies it
as -1. This allows to use the same dfu_alt_info string for different MMC
devices (like embedded eMMC and external SD card if data layout is the
same).

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: If076f723f60ab30f7f82c7cc9cffb484a854e29b