[PATCH v2 0/3] target/arm: disable-tcg and without-default-devices fixes

Fabiano Rosas posted 3 patches 12 months ago
Failed in applying to current master (apply log)
There is a newer version of this series
target/arm/Kconfig       | 9 ++-------
tests/qtest/cdrom-test.c | 5 +++++
2 files changed, 7 insertions(+), 7 deletions(-)
[PATCH v2 0/3] target/arm: disable-tcg and without-default-devices fixes
Posted by Fabiano Rosas 12 months ago
Since v1:

Used the 'select if TCG' pattern for both build issues.

We don't want to use imply for semihosting because we'd need to
unpoison* CONFIG_SEMIHOSTING and include CONFIG_DEVICES from helper.c
when building with '--enable-tcg --without-default-devices'.

*- I see the config at build/config-poison.h but the compiler does not
   complain about the usage. It just gets ignored. Any idea why?

CI run: https://gitlab.com/farosas/qemu/-/pipelines/857704454

v1:
https://lore.kernel.org/r/20230503193833.29047-1-farosas@suse.de

Here's the fix for the cdrom test failure that we discussed in the
list, plus 2 fixes for the ---without-default-devices build.

When I moved the boards CONFIGs from default.mak to Kconfig, it became
possible (due to --without-default-devices) to disable the CONFIGs for
all the boards that require ARM_V7M. That breaks the build because
ARM_V7M is required to be always set.

Fabiano Rosas (3):
  target/arm: Select SEMIHOSTING when using TCG
  target/arm: Select CONFIG_ARM_V7M when TCG is enabled
  tests/qtest: Don't run cdrom tests if no accelerator is present

 target/arm/Kconfig       | 9 ++-------
 tests/qtest/cdrom-test.c | 5 +++++
 2 files changed, 7 insertions(+), 7 deletions(-)

-- 
2.35.3