Some specific string were transformed to something else.
Example: usb_otg_path = 2:11 was transfomed as 131 because
being interpreted as a time

UUU is NXP's flashing tool for IMX platforms:
https://github.com/NXPmicro/mfgtools

UUU allows to intercept IMX ROM code, and download images on
target by using SDP (Serial Download Protocol).
IMX platforms are booting in SDP mode based on dip switches,
but also falls back in SDP mode when no valid bootloader is
found in the selected boot media (eg: SD card, or eMMC).

Signed-off-by: Rémi Duraffort <remi.duraffort@linaro.org>

Commands used to ented DFU mode in u-boot were hardcoded. In order to
allow for customization this section is now defined as list of
variables. Each device-type which uses DFU from u-boot should now
specify at least:
{% set fastboot_deploy_dfu_commands = fastboot_deploy_dfu_commands|default(["dfu"]) %}

Signed-off-by: Milosz Wasilewski <milosz.wasilewski@linaro.org>

This will make it easier to work on #256

Signed-off-by: Rémi Duraffort <remi.duraffort@linaro.org>

Signed-off-by: Rémi Duraffort <remi.duraffort@linaro.org>

The base template should not force the variable value but only provide a default.

With this patch, a device based on ssh.jinja2 can now specify the user to use when connecting.

Signed-off-by: Rémi Duraffort <remi.duraffort@linaro.org>

This reverts commit 2dbb0e05

.

Signed-off-by: Rémi Duraffort <remi.duraffort@linaro.org>

On some boards, the hardware dfu serial number is set to 0000000.
If the boards provides two boot media (NOR and NAND for example), we can have a
recovery u-boot on one media that will be used to flash the second media using
dfu.

Signed-off-by: Rémi Duraffort <remi.duraffort@linaro.org>

This will make it easier to fix #256

Signed-off-by: Rémi Duraffort <remi.duraffort@linaro.org>

Signed-off-by: Rémi Duraffort <remi.duraffort@linaro.org>

Allow uboot_interrupt_newline to be available. Add documentation for
this option, as well as troubleshooting instructions.

The comment in base-uboot.jinja was incorrect (and unnecessary);
removed.

Signed-off-by: Dan Rue <dan.rue@linaro.org>
Reviewed-by: Neil Williams <neil.williams@linaro.org>
Tested-by: _Marge_Bot <https://git.lavasoftware.org/lava/lava/merge_requests/350>

Also refactor the correspoding templates and add missing colons.

Closes #199

Signed-off-by: Rémi Duraffort <remi.duraffort@linaro.org>

Signed-off-by: Milosz Wasilewski <milosz.wasilewski@linaro.org>

Closes: #188

Signed-off-by: Neil Williams <neil.williams@linaro.org>
Reviewed-by: Rémi Duraffort <remi.duraffort@linaro.org>
Tested-by: _Marge_Bot <https://git.lavasoftware.org/lava/lava/merge_requests/296>

Unlike tftp/nfs, NBD jobs does not handle uboot_set_mac.
This patch fix that.

Change-Id: I90fdb2f2e335fb05206cb9476f741ed08a9553e5
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

Unlike base-fastboot, base-uboot missed the quote marks
on fastboot and adb serial numbers. If the "number" happened
to be all digits, this causes valid_device to fail.
Closes: #118

Signed-off-by: Neil Williams <neil.williams@linaro.org>

device-type is solely for the scheduler to use, based on the
test job submission. The template to use is determined solely
by the {% extends %}

Change-Id: I6325d86e07694b8729a48ed8f47fdb874781ae5f

LAVA-1319 - Remove static YAML files and replace with device
configuration generated from current Jinja2 templates.

Change-Id: I2a078a6aa00d8385780515167630aa3d526f7fa8

Change-Id: Ieaf24d17b495778de063488d2634643a478f12ac

Change-Id: Ia8c1e291472fdf9359eae2f65a306f11af0de5dc

- warp7 uses uboot usb mass storage, update base-uboot to allow ums commands

Change-Id: I882d6f66b65dbb135522bc2f5d89951af33b64d0

Change-Id: Ie65a3c0acae27f086fe42a8267b3f37b9f02502f

Commands for loadkernel and loadinitrd can already be modified.
Allow devices which have renamed 'tftp' to also set the modified
version for fdt.

Change-Id: I6886ed4cc3159b5618d47d526648f25932f33c60

Certain devices (apq8016-sbc-uboot) boot directly to a prompt and dont need interrupting.

Change-Id: I74c30866b7292d3a6fdfe4ad884693d996951727

For uboot, instead of building and sending a singular large boot command,
execute each step individually to allow better monitoring of errors.
Also, make bootloader error messages per-bootloader.

Change-Id: I68e91cb3084318684461666fea36e2770af56a2e

Retain fastboot as the existing command in U-Boot.
Force ttyS2 for the console, supported by recent kernel builds

Change-Id: Iccea8ddfd685d5c07396635899685315c37c780f

The kernel bootargs passed for NBD boot aren't quite right and were also
missing proper quites.  Fixup.

Validated in the BayLibre LAVA lab running 2017.7 + NBD patch + this
patch.

Change-Id: I3a3929396e0e4bb8d2e1d4aa137a90e2e4dc4b39
Cc: "Jan-Simon Möller" <dl9pf@gmx.de>
Signed-off-by: Kevin Hilman <khilman@baylibre.com>

This is the companion to
https://review.linaro.org/#/c/17088/



We include these changes:
- Add a new protocol called 'lava-xnbd' to the schema.
  This will be used to expose a filesystem image via nbd
  (network block device, xnbd-server) to the DUT.
  This is required when the target uses extended filesystem
  attributes which are not supported through nfs.
- Add the initrd parameter.
  In contrast to the 'ramdisk' which is usually a cpio.gz,
  an 'initrd' is a full filesystem image e.g. ext4.
  This is needed when the system uses xattr e.g. for
  security labels also in the 'initrd' already.
  The 'initrd' needs to stay unmodified.
- Add documentation.

v4: make u-boot entries shorter and use double-quotes
v3: fix base-uboot.jinja2 (u-boot cmdline too long)
v2: squash into one and add documentation

Change-Id: Ib031d9beed6ed25fa262eeb4b9be5ca7cb793112
Signed-off-by: Jan-Simon Möller <dl9pf@gmx.de>

Change-Id: Idfd9ac02cccaf1da055865a3d80f8d2e43b78cd7

This prevent loading a useless DTB when 'append_dtb' is set, since it will
already be sent with the crafted 'kernel-dtb' artifact.

Change-Id: I293d11df402b21dc7b187edf45f61385c93398ee
Signed-off-by: Florent Jacquet <florent.jacquet@free-electrons.com>

Change-Id: Ibb20d57f3af0be3225789d942e97ffd305e933a4

It's allowed to put the name of the block after the end tag. See
http://jinja.pocoo.org/docs/2.9/templates/#named-block-end-tags

Change-Id: I6c8ca7d2d7b9cdc7758067f8ab8c3ee3b12ba998

A string passed to default() should be between '' or "" or it will be
interpreted as a variable name.

Change-Id: If747997ed5660df9edb38e35bb37c7f8019379ff

The u-boot command "boot" is an optional feature which is simply an
alias for "run bootcmd".  Several vendor uboot (Amlogic, Renesas) do
not have this command enabled, so using this command doesn't work.

To fix, just use "run bootcmd" directly, so the base-uboot device-type
can work on boards without this feature.

Also update the test case which checks for 'boot' in default args.

Change-Id: Iae28e22c0894d2567d93cbd5ee80b697883e40b0
Signed-off-by: Kevin Hilman <khilman@baylibre.com>

Change-Id: I241ef9c1adba9bd58bdfd4e1145624bc5b9f180b

This allow a LAVA v2 device-type configuration to have
{% set append_dtb = true %}
and
{% set use_xip = true %}

Change-Id: Ic16cc18490107269b2228e9c5f30e7c207f81a2f
Signed-off-by: Florent Jacquet <florent.jacquet@free-electrons.com>

When a block is defined in the base template, it will be present in the
generated file even if the child template does not reference it.

So adding "{% block foo %}{{ super() }}{% endblock %}" will duplicate the block
"foo" in the generated file.

This duplication is not really an issue due to the way the yaml parser is
handling duplicated dictionaries (the last one overrides the previous ones).

Change-Id: Ifb266e6135871ea96cc09e9d84fd4d70b52506a7

This allows to override the whole bootcmd command by setting either
uboot_bootcmd, uboot_tftp_bootcmd, or uboot_nfs_bootcmd in any child template of
base-uboot.jinja2.
This also keeps the default value if any of them is not set.

Change-Id: I86e463576e972e95360986a7c78748da1ddae713
Signed-off-by: Florent Jacquet <florent.jacquet@free-electrons.com>

This allows to override the tftp commands in U-boot, by setting the
uboot_tftp_commands variable in any child template of base-uboot.jinja2.
This keeps the default tftp commands if not set.

Change-Id: Ic50aedc9d52fe559361b54cb6ad15576f1192e97
Signed-off-by: Florent Jacquet <florent.jacquet@free-electrons.com>