[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v1 1/2] hw/tricore: fix inclusion of tricore_testboard
From: |
Peter Maydell |
Subject: |
Re: [PATCH v1 1/2] hw/tricore: fix inclusion of tricore_testboard |
Date: |
Mon, 19 Jul 2021 22:21:20 +0100 |
On Mon, 19 Jul 2021 at 20:52, Alex Bennée <alex.bennee@linaro.org> wrote:
>
> We inadvertently added a symbol clash causing the build not to include
> the testboard needed for check-tcg.
>
> Fixes: f4063f9c31 ("meson: Introduce target-specific Kconfig")
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
> configs/devices/tricore-softmmu/default.mak | 1 +
> hw/tricore/Kconfig | 3 +--
> hw/tricore/meson.build | 4 ++--
> 3 files changed, 4 insertions(+), 4 deletions(-)
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
as far as this fix goes (though maybe CONFIG_TRICORE_TESTBOARD would be better?)
But I still don't understand and would like to know:
(1) why doesn't CONFIG_TRICORE get set by Kconfig anyway, as
f4063f9c31 claims to be doing?
(2) what are the CONFIG_$ARCH flags for? Apart from this, we
don't seem to be using any of them, as demonstrated by the fact
that nothing else broke :-)
thanks
-- PMM
Re: [PATCH v1 1/2] hw/tricore: fix inclusion of tricore_testboard, Philippe Mathieu-Daudé, 2021/07/19
[PATCH v1 2/2] gitlab: enable a very minimal build with the tricore container, Alex Bennée, 2021/07/19