jjb/common: fix releng name for porter

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

Add agl-service-weather to CI

Change-Id: Icca26c1f11849b85436354d09007048f050ff949
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

jjb/common: lava-jobs-submit: fix device_type replacement

Fix "raspberrypi3-uboot" -> "raspberrypi3-uboot-uboot" type of
substitutions.

Ensure that the full string (up to EOL) is matched before trying to
replace.

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

jjb/common: run-test-short: Update releng-scripts usage

Update the calls to releng-scripts to use the latest options.

This allows to differentiate the build types, find the correct URL and
run the corresponding test plans.

Change-Id: I75762ec8d0a7c204b6ec898f936f59a7a9b5e488
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>

Create a link to the last generated docker file

Change-Id: I2a9babf0db60be4f9e6ee65d05c655b18d8603bb
Signed-off-by: Ronan Le Martret <ronan.lemartret@iot.bzh>

Merge "Streamline image output and add new board"

Streamline image output and add new board

Change-Id: If9f7f32d0c73bf4f964a205a18dc1fc720b38e3b
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

jjb/common: lava-labs-prepare.sh fix polling on reserved boards

The script would still wait for 60 seconds and retry on boards that are
not in the reserved state.

Fix this and poll for a change of status only if the board is reserved.

Fixes: 1ae347940236 ("jjb/common: lava-labs-prepare.sh")
Change-Id: I4ce71979f7cb2856768de748c1afb6a1cfb47ca2
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>

Merge "Apply new folder structure for application repository"

Apply new folder structure for application repository

Change-Id: I5937215900d8310947e5a7a1e6e247abda0703b8
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Merge "Add new repositories to application CI"

Add new repositories to application CI

Change-Id: I78d5ef464da02e87e3f6d78e4cb7e40fcf63a9b2
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

jjb/common: lava-labs-prepare.sh

When device status was reserved, the script would exit and the job fail.

If device status is reserved poll the device status a limited amount of
time. If the status within the defined polling time changes continue
otherwise exit.

Polling as been set to 10 retries on the minute.

JIRA: SPEC-1210
Change-Id: I7ee950777f0881a4a588c9c0dc8db12a7f6dda4a
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>

jjb/common: lava-labs-prepare.sh: disable error exit

Disable exit on error detection in lava-labs-prepare.sh, this script is
looking for error codes when needed and exits gracefully.

Change-Id: I0044466ef3f7ad7b606b1452d51676a4472e87d0
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>

Merge "jjb/common: if any, display lava error msg in jenkins log"

jjb/common: if any, display lava error msg in jenkins log

Do a second grep on the lava log to include lava error message if there
was any. This is usefull in case a job fails. The error can be read
directly out of the jenkins log.

JIRA: SPEC-1178

Change-Id: Id3333df9c56dd67f5d664ae0d234e33e3300c366
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>

jjb/common: Fix race condition when using grep and tr

when using grep + tr to find board availability, status, a race
condition happens sometimes. When it happens the job is stuck and
timeout fails.

Fix it by calling sequentially grep then tr and not piping.

Change-Id: Ib1322e280900b62abdd827992b42183024cd094c
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>

jjb/common: Fix query of lava log in jenkins log

Fix the query of the log in jenkins log. Use the lava website url and not the
lava API url.

JIRA: SPEC-1178

Change-Id: I9ddc48b13e91a91aa2e53c768aa02f2ae12e5a99
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>

jjb/common: run-test-prepare: fix unmapped boards

If a $MACHINE doesn't have a proper mapping for LAVA or
releng-scripts, fail gracefully (exit 0)

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

jjb/common: clean up device names to standardized ones.

Use the device_name table aliases earlier in the CI process. That way
all the scripts are using the correct device name. Avoiding name
mismatch resulting to CI failure.

Change-Id: Ia6f9f8beb06c43a24605abb24e676eaed68cc67e
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>

jjb/common: run-test-short-*: meaningful LAVA job names

The previous job name was always "AGL-gerrit" which is not very useful.
Change it to "AGL-gerrit-" + change_number + patchset_number

Change-Id: I161de8a3547f864d4a50e5d984561899e0401532
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>

jjb/common: remove hwtest.enable files v2

releng-scripts now supports LAVA job generation for all boards.

Remove the check in run-test-prepare.sh on whether or not to create LAVA
test jobs. Replace this test by several exports using releng-scripts
job-prereq tool. These exports will be used to rsync the files to the CI
upload server.
If the calls to job-prereq returns an error, make the script fail
gracefully.

Transition all platforms in run-test-short.sh to use releng-scripts.
Transition all platforms in run-test-short-release.sh to use releng scripts.

JIRA: SPEC-1144

Change-Id: I1e0e6d39b986770adcdfcf92c5d589a7fbe99a33
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>

Merge "Revert "jjb/common: remove hwtest.enable files""

Revert "jjb/common: run-test-short-*: meaningful LAVA job names"
Revert due to issues with jobs.
This reverts commit 7ec5cece04ee2af16f224214e7c8e23212c08c54.

Change-Id: I3e660f27485731b13bdf1959a9f441d81a3efaf9

Revert "jjb/common: remove hwtest.enable files"

Issues with the jobs.

This reverts commit f28a1005057b096a95ba73bc3bf3e029b835deaa.

Change-Id: Icf75e5b462b0d4aff3c8d2c5806b8ca9270ad22b

jjb/common: run-test-short-*: meaningful LAVA job names

The previous job name was always "AGL-gerrit" which is not very useful.
Change it to "AGL-gerrit-" + change_number + patchset_number

Change-Id: I2c929a2c0d0541e593be4e459766a836a1b57e4b
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>

jjb/common: remove hwtest.enable files

releng-scripts now supports LAVA job generation for all boards.

Remove the check in run-test-prepare.sh on whether or not to create LAVA
test jobs. Replace this test by several exports using releng-scripts
job-prereq tool. These exports will be used to rsync the files to the CI
upload server.

Transition all platforms in run-test-short.sh to use releng-scripts.
Transition all platforms in run-test-short-release.sh to use releng scripts.

JIRA: SPEC-1144

Change-Id: I558563704bbe9bf0bf554219f4f7ca8e086649bb
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>

Add eel branch for application CI

Change-Id: Id0a49e2ab2b3fc90968e93633f37015b9138910d
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Fail gracefully if SDK is not found

Change-Id: Ic21a7d3b9a38bc0eb7a86cc730534ec08b718116
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Merge "jjb/common: run-test-short: always use releng"

Merge "jjb/common/lava-labs-prepare: LAVA fail gracefully if device not found"

jjb/common/lava-labs-prepare: LAVA fail gracefully if device not found

When LAVA labs are offline, or otherwise unavailable,
"lava-tool devices-list" fails causing the whole job to fail.

Detect this error earlier when trying to add the authentication and skip
the lab if offline.
If the device was not found in any lab, exit with success.

JIRA: SPEC-1142

Change-Id: Ia2cdeea3da8ade7feb5962e0898f40e59ca9cfdb
Signed-off-by: lollivier <lollivier@baylibre.com>

jjb/common: run-test-short: always use releng

When present, use releng to generate LAVA jobs for all boards.

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

jjb/common: lava-labs-prepare: rename dra7

Rename dra7-evm to use upstream LAVA device-type conventions.

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

Add new applications to CI

This adds these app repos to CI:
      - agl-service-audio-4a
      - agl-service-audio-mpcd
      - agl-service-unicens
      - navigation
      - onscreenapp

Change-Id: Ia7f56d277999bfceb6d877b72a73cd630a28c937
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Add the cyclone5 board to CI

This adds the cyclone5 board and replaces the porter-nogfx.

Change-Id: I202262bc8b208c84dcaa5671c6e20c0a45bec65d
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

jjb/common: lava-labs-prepare: extend aliases

Extend the aliases table for all existing devices, even the one not using
releng. That way we can search for the correct device type in the lava
labs.

Change-Id: Iaaa5b66923d4ed1e302d874457af573e12384acf
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>

jjb/common: lava-labs-prepare: catching errors

When no device of the requested device-type is available, abort early
with a useful error message, otherwise job continues and fails in
confusing ways.

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

jjb/common: lava-jobs-submit

lava-tool submit-jobs go back to --block as wait-for-jobs isn't working
as expected. Revert the change until wait-for-jobs works.

But keep this version of the tool to have a link to the lava-job posted
in the log when using submit-job.

Change-Id: Ie7d045d198b00b19d198404fc67b2bcfe992e9a6
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>

jjb/common: lava-jobs: create device name aliases

The previous version would fail to find some boards if the device names
were different between lava, AGL, releng.
This version creates a device alias table. And uses this table with the
correct name for each component (AGL, lava, releng).

Change-Id: Ief064c5ff9edb2d824e80549148ee0ab40ebcddf
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>

jjb/common: lava-jobs-submit: remove timeout

Remove the timeout from lava submit jobs.
The previous version would timeout after 5 minutes but this is not enough
if the job is qeued. Let Jenkins handle the timeout and remove that one.

Change-Id: I501a539020174b2c9e030716717f1d3d0b06e539
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>

Fix lab URL using RPC2/

Change-Id: I695244c8d6880499db2ac611603b538efc093d3f
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Merge "jjb/common: lava-jobs-submit: fixup wait for job"

Add python-simplejson as dependency of lava-tool 0.19

Change-Id: Icfdfc454106b14c94f05da61d5e349d22ae1fbaa
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Disable src and sstate mirror on image

Change-Id: Ic5cbae9c7eda50110b23acc4a354528bfc02975a
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

packer/provision: agl_dependencies force lava-tool version 0.19

lava-tool version 0.14 from ubuntu xenial is almost deprecated.
Use the jessie-backeports version 0.19.

Change-Id: I4ddb6eb81757ffdb978c8d2edafc4931cf5a42a4
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>

jjb/common: lava-jobs-submit: fixup wait for job

REQUIRES: lava-tool version >= 0.19

The previous version using submit-job with "--block" would never
timeout and remain blocked even if there was an issue during the
LAVA job execution.

This patch fixes the issue by using wait-job-events with a tiemout.
Parsing has been updated to follow these commands as well.

Change-Id: If090d278cb58c1c1b902793bf0ca40daceb2fd97
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>

jjb/common: lava-jobs-submit: fixup device-list failure

The previous reordering of lava lab selection result in a failure
the second time through the devices list.

This patch fixes the failure enough to get things working again,
but the ordering needs to be fixed so that there is only one pass
through the devices-list.

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

CIBT Update the callback arg for releng.

Update callback arg for releng following releng changes:
https://gerrit.automotivelinux.org/gerrit/#/c/11751/

Change-Id: I2f63727e2357d84085c8c91f8bcbe8983489abc4
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>

CIBT improve lava lab searching and coding style

Fix coding style, indentation.

Improve the lab search, if a lab has the device do not search in others
as it won't be used.

Change-Id: I7471511ff7af3d2f6d8708943b42099a450ef770
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>

CIBT find the LAVA lab before calling releng

Add some logic to lava-labs-prepare:
- Search for the machine in the list of labs
- export the lab name if found

Call releng with the corresponding lab name in run-test-short

Send the job directly to the corresponding lab in lava-jobs-submit.
Keep the 'checking for device' in this script as well, just to make sure
the device is still available and can run the job. Otherwise the script
could be blocked there if the device status changes.

Change-Id: I50646b7fdb607cb9245a1d31e644eee45a2a2356
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>

Merge "Lava jobs update raspberrypi3 device_type name"

Fix agl-run-test-short

Remove machine typo from the releng script call and use the machine
argument instead.

Change-Id: I012204bf61f45f6c6d4ae56c579db84c9072278a
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>

Lava jobs update raspberrypi3 device_type name

For the raspberrypi3, update the device_type name to the LAVA
names for the rpi3 32bit:
- raspberrypi3-uboot
- bcm2837-rpi-3-b-32b

Change-Id: Icc654e47fd03b437165fb9c9aac5efdac490ef5c
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>

Typo fix in agl-run-test-short

Fix a typo in the if to call releng-scripts.

Change-Id: Idb2331b79d8a0947ae1d69f3bc73cd266c01b7ef
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>

Merge changes Ie341a1ed,Ibd6449c1

* changes:
  Update releng calls to use new args
  Use releng for rapsberrypi3

Fix comment triggers to be strict

Comment triggers stuck in a loop since
some jobs contain "remove-artifacts" in the
job name.  Made the matching strict.

Change-Id: Icaeb4a9db2a128ea00e0b3e775fe7c9fa52ff2a7
Signed-off-by: Jeremy Phelps <jphelps@linuxfoundation.org>

Update releng calls to use new args

Update the releng arguments:
- Use the new url scheme with CI, changeid and patchset
- Use machine argument

Change-Id: Ie341a1edf6f341796a7fe0647bf61f29172eedb4
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>

Use releng for rapsberrypi3

Add the rapsebrrypi3 to the list of boards using releng-scripts for
LAVA job definition generation.

Change-Id: Ibd6449c101aff829d2814d3b15a7c9feae6d4b52
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>

Fix task dependency bug in eel also in CI build

Change-Id: I6f61a2c15f0f9fcfe27ed66b688bf1a4e8921825
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Add hotfix to release build for YP bug 12070

This adds a hotfix for https://bugzilla.yoctoproject.org/show_bug.cgi?id=12070
until it is fixe upstream.

Change-Id: I9fe57d853f44ad57af115825b2d66e859e20d53e
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Add source and binary release jobs for EE

Change-Id: I75103b1943f7a2778d4fb1a7e06383e9515566b4
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Add include fragment for ptest

Change-Id: I963b015bb05b1d83ac88f69ff7921a39e16b1742
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Disable isafw report and change trigger

Change-Id: Ie9af0df97a76b4684201fe392153aa8cc680c4a0
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Add agl-service-mediaplayer

Change-Id: I9d66727c44c1db17f201f4025fec716923c0c69b
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Merge "jjb/common: run-test-short: use new releng opts"

Fix a few typos

Change-Id: Icf398847d42940fc79f9379a347808a5fe9fe3dc
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

jjb/common: run-test-short: use new releng opts

releng-scripts command-line args were updated to allow more
flexibility in defining rootfs image.  Use new options.

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

Add release build job template for eel

Change-Id: If6e448e00f01c4f9316fcd0545d1eb015f3d2221
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Add tests from qa-testdefinitions to dab-snapshot build

Change-Id: I63d012a25ecce656d1acc9090c8de278eccd2287
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Allow build of SDK for aarch64 and rsync after test

Change-Id: Ic3934c7a32314709f5bb376c57c180960ae0f0e9
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Enforce uImage on dab

Change-Id: I4ce0b0606f0e2990255334f1c0247f7241dd82b7
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Add CI for apps/agl-service-gstreamer

Change-Id: I3ad36050d01bac6688b83e05c696cbfee0d92c59
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Enforce the reinstallation of jinja2

to make sure we get the desired version ...

Change-Id: I56a8be583340b851e62773bb4d8ba5ac9dde692d
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

jjb/common: run-test-short: fix pip show

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

jjb/common: run-test-short: show jinja2 version

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

Merge "Increase timeout and skip test prepare"

Increase timeout and skip test prepare

Change-Id: I17457661b7b654463035078f2778ce86a021c99d
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

jjb/common: run-test-short: fix board-specific check for releng-scripts

The (broken) conditional used to check which boards to use with releng-scripts
was always returning true.  Fix it to use [[ ]] instead of (( ))

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

Merge "Do not abort the other CIBT jobs on failure"

Do not abort the other CIBT jobs on failure

Change-Id: Icbda1c48bd82df5178b3bb737f31ade3919608b8
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

jjb/common: run-test-short: add tests, unique version and callbacks

Use the gerrit changeset/patch numbers in combination with a SHA1 of
the repo manifest to create a unique build_version for the metadata.

Enable jobs to send results to lab-baylibre-legacy backend.

Send all tests (there are so few, might as well run them all.)

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

Merge "jjb/common: aglsetup: save pre-build env"

Merge "jjb/common: run-test-short: fix line-continuation typo"

packer: agl_dependencies: need latest python jinja2 lib

releng-scripts templating depends on the current stable (v2.9) version
of jinja2.

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

jjb/common: run-test-short: fix line-continuation typo

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

jjb/common: aglsetup: save pre-build env

Save some pre-build environment which can be saved/uploaded
and used to reproduce builds and share with downstream CI
infrastructure.

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

Fix issue with difference in buildid by using changeset number only

Change-Id: I852c93da0514dbdc224612b7d150a17b572e6021
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Add application repositories for agl-service-soundmanager-2017 and agl-service-windowmanager-2017

Change-Id: I613805d157182907b5b0d21c1714727736a7f99e
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Remove safety switch

Change-Id: Ia6320c3f78308acab6e9a33987c5a2bb90db41ea
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Fix job name and typo in script

Fix job name in ci-app yaml file and typo variable name in
common script.

Change-Id: I903b5135852b710efb187cb2194fe60c59e825b0
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

After merge, rebuild app

and upload it to the app repository folder on
the download server for consumption.

Change-Id: I3df70f0ce11f0d448999e9b54e637a53070e0439
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Exempt aarch64 on dab currently.

Change-Id: Iddf1b14e632d43e43567058ecbbdaa3bd59d4374
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Merge "Adapt trigger for app build"

Adapt trigger for app build

Change-Id: Ie32360bcdf4fcf0586494cdc4e70aa5e0e4c5f13
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Merge "Per branch app build"

Per branch app build

Thie change introduces app builds per apps/* repo branch.
E.g. master and dab branch are separately tracked and built.

Preparation of the app folder.

Change-Id: I5f15b68e072c6abc69dfd41197ad2052ed8aed4e
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Update snapshot build recipe for dab and add dragonboard as 'full' aarch64 target

Change-Id: Ia2ea7f485282941d22ef81bd2b0d73e7350fd90e
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

LAVA jobs: create jobs using releng

Switch to using releng-scripts "create-jobs" util for creating LAVA
jobs from new templating system.

For beta-testing, only use for QEMU and m3ulcb for now

(Yes, the if/else/endif without nesting is ugly, but makes the
 patch easier to read without all the whitespace change for
 indents.  Whitespace can be updated in a follow-up if this
 change is accepted.)

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

append needs extra space

Change-Id: Iebb770258d31cd87ec55427c96a408fb9e61c75e
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>