dtoc: Correct dtoc output when testing
authorSimon Glass <sjg@chromium.org>
Mon, 26 Apr 2021 20:19:48 +0000 (08:19 +1200)
committerSimon Glass <sjg@chromium.org>
Thu, 29 Apr 2021 10:23:37 +0000 (03:23 -0700)
commit170732523bad74cd3a19c3993ba06c789c843ccf
tree401de0544216ee9f24ab84eff43e07b192d13efe
parent0fe44dc676855ae195e6532e0bed56f53bd3346c
dtoc: Correct dtoc output when testing

At present each invocation of run_steps() updates OUTPUT_FILES_COMMON,
since it does not make a copy of the dict. This is fine for a single
invocation, but for tests, run_steps() is invoked many times.

As a result it may include unwanted items from the previous run, if it
happens that a test runs twice on the same CPU. The problem has not been
noticied previously, as there are few enough tests and enough CPUs that
is is rare for the 'wrong' combination of tests to run together.

Fix this by making a copy of the dict, before updating it. Update the
tests to suit, taking account of the files that are no-longer generated.

With this fix, we no-longer generate files which are not needed for a
particular state of OF_PLATDATA_INST, so the check_instantiate() function
is not needed anymore. It has become dead code and so fails the
code-coverage test (dtoc -T). Remove it.

Signed-off-by: Simon Glass <sjg@chromium.org>
tools/dtoc/dtb_platdata.py
tools/dtoc/test_dtoc.py