[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [PATCH v2 1/3] arm: Add Nordic Semiconductor nRF51 SoC
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-arm] [PATCH v2 1/3] arm: Add Nordic Semiconductor nRF51 SoC |
Date: |
Mon, 2 Jul 2018 13:25:10 +0100 |
User-agent: |
Mutt/1.10.0 (2018-05-17) |
On Thu, Jun 28, 2018 at 12:08:13AM +0930, Joel Stanley wrote:
> +/* TODO: Flash size should be defined by the board. Microbit uses 256KB */
> +#define FLASH_BASE 0x00000000
> +#define FLASH_SIZE (256 * 1024)
> +
> +/* TODO: Flash size should be defined by the board. Microbit uses 16KB */
> +#define SRAM_BASE 0x20000000
> +#define SRAM_SIZE (16 * 1024)
Making FLASH_SIZE and SRAM_SIZE properties of the NRF51State object.
Then microbit.c can set them.
> +static void nrf51_soc_init(Object *obj)
> +{
> + NRF51State *s = NRF51_SOC(obj);
> +
> + memory_region_init(&s->container, obj, "nrf51-container", UINT64_MAX);
> +
> + /* TODO: implement a cortex m0 and update this */
Please see "[PATCH v2 0/3] arm: add skeleton Cortex M0 CPU model". It
should work with TYPE_ARMV7M -> TYPE_ARMV6M and "cortex-m3" ->
"cortex-m0".
Julia is currently completing the Cortex M0 emulation but at this point
guest software may already run successfully. If not, then the patches
she sends over the next week or two ought to do the trick.
> +typedef struct NRF51State {
> + /*< private >*/
> + SysBusDevice parent_obj;
> +
> + /*< public >*/
> + char *kernel_filename;
Is this used by anything?
signature.asc
Description: PGP signature
- Re: [Qemu-arm] [PATCH v2 1/3] arm: Add Nordic Semiconductor nRF51 SoC,
Stefan Hajnoczi <=