On Thu, 14 Feb 2019 at 12:51, Peter Maydell <peter.mayd...@linaro.org> wrote:
>
> The Musca board puts its SRAM and flash behind TrustZone
> Memory Protection Controllers (MPCs). Each MPC sits between
> the CPU and the RAM/flash, and also has a set of memory mapped
> control registers. Wire up the MPCs, and the memory behind them.
> For the moment we implement the flash as simple ROM, which
> cannot be reprogrammed by the guest.
>
> Signed-off-by: Peter Maydell <peter.mayd...@linaro.org>
> ---
> hw/arm/musca.c | 155 ++++++++++++++++++++++++++++++++++++++++++++++---
> 1 file changed, 147 insertions(+), 8 deletions(-)
>
> diff --git a/hw/arm/musca.c b/hw/arm/musca.c
> index 8774e0b87b7..5fadac8c09b 100644
> --- a/hw/arm/musca.c
> +++ b/hw/arm/musca.c
> @@ -27,11 +27,15 @@
> #include "hw/arm/armsse.h"
> #include "hw/boards.h"
> #include "hw/core/split-irq.h"
> +#include "hw/misc/tz-mpc.h"
> #include "hw/misc/tz-ppc.h"
> #include "hw/misc/unimp.h"
>
> #define MUSCA_NUMIRQ_MAX 96
> #define MUSCA_PPC_MAX 3
> +#define MUSCA_MPC_MAX 5
> +
> +typedef struct MPCInfo MPCInfo;
[...]
> +typedef struct MPCInfo {
> + const char *name;
> + hwaddr addr;
> + hwaddr size;
> + MPCInfoType type;
> +} MPCInfo;
This should just be "struct MPCInfo { ... };" to avoid clang
complaining:
hw/arm/musca.c:165:3: error: redefinition of typedef 'MPCInfo' is a C11
feature [-Werror,-Wtypedef-redefinition]
Since it's a minor thing I'll just squash it in when I
apply this series, assuming I don't need to respin for
anything else.
thanks
-- PMM
