Hi all,

We've started looking at the TF-A qemu_sbsa port a bit, and found some things that were carried over from the mach-virt port for no good reason:

Looking at plat/qemu/qemu_sbsa/platform.mk: - lib/cpus/aarch64/aem_generic.S is for fast models/foundation model. - although we may want to add a "generic armv8 cpu" to TF-A at some point, to reduce synchronization needs when bumping the cpu used by sbsa-ref in upstream QEMU. - lib/cpus/aarch64/cortex_a53.S is not supported by sbsa-ref, nor does the A53 support enough physical address bits to be able to control this platform. - drivers/io/io_semihosting.c, lib/semihosting/semihosting.c, and lib/semihosting/${ARCH}/semihosting_call.S are not actually used. But there *are* some other bits of qemu/common code which needs to be cinditionalized-out if these files aren't included. - plat/common/plat_psci_common.c - all of the ${ARCH} stanzas should just be 'aarch64', this platform cannot be booted in aarch32. - plat/common/plat_psci_common.c isn't usefully hooked up to anything at the moment, but we can fix that when we get to actually implementing useful PSCI. That file may simply not belong in 'common'.

/ Leif